Remove unnecessary null pointer checks about island

[tgfrerer]

Oh, true, good point!

The check should not be for the gpu_mesh field but for the self, as we want to protect against accessing a field in an object that might have been already deleted...

e.g.

if ( self->gpu_mesh ) {
	delete self->gpu_mesh;
}

should be:

if ( self) {
	delete self->gpu_mesh;
}

(we still need to check for self being valid before possibly accessing its gpu_mesh field)

Would you like to send a PR? :)

[tgfrerer]

update: this should be fixed now as of 32a012b

[elfring]

💭 Are you going to reconsider any more implementation details?