What does this comment mean? What was "hidden" that is now not hidden. All I see is a variable renamed - and why the new name is better than the old escapes me.
Ralph Begin forwarded message: > From: ggreg...@apache.org > Subject: svn commit: r1620442 - > /logging/log4j/log4j2/trunk/log4j-api/src/main/java/org/apache/logging/log4j/message/StructuredDataMessage.java > Date: August 25, 2014 at 2:08:57 PM MST > To: comm...@logging.apache.org > Reply-To: d...@logging.apache.org > > Author: ggregory > Date: Mon Aug 25 21:08:57 2014 > New Revision: 1620442 > > URL: http://svn.apache.org/r1620442 > Log: > Don't hide ivar. > > Modified: > > logging/log4j/log4j2/trunk/log4j-api/src/main/java/org/apache/logging/log4j/message/StructuredDataMessage.java > > Modified: > logging/log4j/log4j2/trunk/log4j-api/src/main/java/org/apache/logging/log4j/message/StructuredDataMessage.java > URL: > http://svn.apache.org/viewvc/logging/log4j/log4j2/trunk/log4j-api/src/main/java/org/apache/logging/log4j/message/StructuredDataMessage.java?rev=1620442&r1=1620441&r2=1620442&view=diff > ============================================================================== > --- > logging/log4j/log4j2/trunk/log4j-api/src/main/java/org/apache/logging/log4j/message/StructuredDataMessage.java > (original) > +++ > logging/log4j/log4j2/trunk/log4j-api/src/main/java/org/apache/logging/log4j/message/StructuredDataMessage.java > Mon Aug 25 21:08:57 2014 > @@ -250,8 +250,8 @@ public class StructuredDataMessage exten > final StringBuilder sb = new StringBuilder(); > final boolean full = Format.FULL.equals(format); > if (full) { > - final String type = getType(); > - if (type == null) { > + final String myType = getType(); > + if (myType == null) { > return sb.toString(); > } > sb.append(getType()).append(' '); > >