Handling for new warning?

Mike Duigou openjdk at duigou.org
Tue Sep 24 17:31:00 UTC 2019


I am working on a feature for javac that adds a new lint level warning 
for undeclared thrown RuntimeExceptions. The patch itself is small but 
since the compiler is built with all warnings enabled there are 
thousands of methods with found to be missing "throws" for 
RuntimeExceptions that they currently don't declare. Should I prepare 
the patch of this new feature to include all of the warnings fixes or 
omit them and explicitly disable the new warning in the build 
configuration? The former would be more correct and complete but masks 
the substance of the change in a huge volume of chaff. The later would 
be more convenient and results in a more succinct path but feels like a 
regression in not enabling all warnings.

Cheers,

Mike


More information about the compiler-dev mailing list