https://issues.apache.org/bugzilla/show_bug.cgi?id=52061
--- Comment #4 from Sebb <[email protected]> --- Thanks for the patch. I've had a quick look, and there seem to be quite a few formatting changes and changes to import layout. Please only change the minimum needed to implement the feature. It's difficult to review the patch properly (and will be difficult to review the SVN diff mail later) if the patch contains spurious changes. One problem I did notice amongst the noise is that it renames a public constant [1] (and changes the value). That will break existing test plans. [1] public static final String USE_HDRS = "true"; // $NON-NLS-1$ -- You are receiving this mail because: You are the assignee for the bug.
