PCDLoader does not handle errors correctly

Description of the problem

The PCDLoader doesn’t handle errors well. If there’s an error when parsing, the error handler passed in is not called and the resulting exception is only caught by a global process handler. This may apply to all loaders as if there’s an error in the setTimeout method used for loading there’s no way to catch that exception.

        var url = 'https://threejs.org/examples/textures/sprites/disc.png';
        var loader = new THREE.PCDLoader().load(
          url, 
          () => alert('something loaded'),
          () => {},
          () => alert('error handler - never gets called')
        );

Three.js version

• r91

Browser

• All of them
• Chrome
• Firefox
• Internet Explorer

OS

• All of them
• Windows
• macOS
• Linux
• Android
• iOS

Hardware Requirements (graphics card, VR Device, …)

any

[elcommie]

Sure. Why not? I’ll get something up later today

[Mugen87]

Maybe something like this would help to improve error handling:

try {

    onLoad( scope.parse( data, url ) );

} catch ( e ) {

    if ( onError ) onError( e );

}

The onError() callback of PCDLoader.load() would now be able to handle exceptions caused by an invalid format.