On 2/19/07, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
Author: mbenson
Date: Mon Feb 19 15:24:43 2007
New Revision: 509378
URL: http://svn.apache.org/viewvc?view=rev&rev=509378
Log:
bulled through JXPath codebase to get a little better familiarity; tiny
refactorings and removals of unnecessary elses, etc. along the way
<snip/>
- if (parentContext != null) {
- return parentContext.getDecimalFormatSymbols(name);
- }
- return null;
+ return parentContext == null ? null :
parentContext.getDecimalFormatSymbols(name);
<snap/>
Unless in corner cases (such as parts of this commit humoring
checkstyle etc.), I am of the opinion that such refactorings shouldn't
be undertaken on released code.
I am not against the changes in this commit, but generally believe
that such changes are subjective, and could lead to vacuous round
trips if it became more widespread practice.
Thanks for your help with [jxpath].
-Rahul
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
