2007/2/7, Alex Blewitt <[EMAIL PROTECTED]>:
On 06/02/07, Tim Ellison <[EMAIL PROTECTED]> wrote: > The pack200 code does have to be internationalized -- why would there be > an exception for that module?Mostly because it's a work-in-progress, and at present when I detect an error condition (or part of the parsing tree that I can't handle) I throw new Error("Not yet implemented"). In other words, it's a marker to let me know I need to go back there and fix it.
It is better to mark such methods by adding NotImplementedException to throws clause. japitools will catch such methods. SY, Alexey
These strings were internationalised into .properties files in an earlier change by others, and whilst 'real' messages should be internationalised when it's complete, they shouldn't be done so during development. There's also potentially some constants I've not marked with // NON-NLS at present, but like I said, it's a work in progress :-) Alex.
