mbien commented on PR #7938:
URL: https://github.com/apache/netbeans/pull/7938#issuecomment-2564864090
right. So usages of `load()` need to be called from a try/catch.
it might have been lost between so many messages but the timeline was like
this if i remember correctly:
- originally one of the calls had a try catch, i pointed that out and said
something in the sense of "we have to check what the failure mode of a failing
call is and might have to add it to both"
- we got the info that the load call should not throw anything, but only
return null, but it turned out that some exceptions slip through anyway
- -> so lets add try / catch to the `load` usages again
its off topic but IMO: anything what does IO and/or non-trivial parsing
should require checked exceptions, this automatically documents how to handle
failures etc, but its a minor detail here and we can't influence it anyway
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
