glfw: Eliminate `Error.InvalidValue` about mach HOT 2 CLOSED

I agree, trying to mimic GLFW would add too much maintenance for too little benefit; felt I should put it on the table anyhow. You do make a good point that perhaps we could propose upstream for GLFW to publicize their internal 'parseMappings' function. Put a pin in that.

I'll look at making the changes whenever I get time today or tomorrow, cheers.

from mach.

Thanks for raising this.

One option is to try and mimic GLFW's parsing to pre-emptively assert that the format is wrong, and perhaps deliver a useful error message to inform the caller of what's gone wrong

I think this would be hard to maintain in the long-term. If we did go that route, I'd prefer we instead add some validation logic to GLFW's API and then everyone who uses GLFW can benefit from it.

the other option is to simply leave the Error.InvalidValue code for this function, and allow it to be unique to it, and any other functions that may do parsing that we can't pre-empt in a maintainable fashion, which, while it doesn't allow us to eliminate the error altogether as hoped, would still be more or less made okay by error denormalization.

I think that this is the most reasonable approach. I would also say InvalidValue doesn't need to be part of the broader Error type. We could potentially rename it to ParseError too? an inline error{ ParseError } or similar

from mach.