On 18/08/2009, Rahul Akolkar <[email protected]> wrote: > On Tue, Aug 18, 2009 at 8:44 AM, <[email protected]> wrote: > > Author: sebb > > Date: Tue Aug 18 12:44:22 2009 > > New Revision: 805391 > > > > URL: http://svn.apache.org/viewvc?rev=805391&view=rev > > Log: > > Remove some JDK 1.3 work-rounds > > > > Modified: > > > jakarta/jmeter/trunk/src/components/org/apache/jmeter/visualizers/LineGraph.java > > > jakarta/jmeter/trunk/src/jorphan/org/apache/jorphan/util/JMeterError.java > > > jakarta/jmeter/trunk/src/jorphan/org/apache/jorphan/util/JMeterException.java > > > jakarta/jmeter/trunk/src/jorphan/org/apache/jorphan/util/JOrphanUtils.java > > > jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParseError.java > > > jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/HTMLParseException.java > > jakarta/jmeter/trunk/test/src/org/apache/jorphan/test/AllTests.java > > > > Modified: > jakarta/jmeter/trunk/src/components/org/apache/jmeter/visualizers/LineGraph.java > > URL: > http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/components/org/apache/jmeter/visualizers/LineGraph.java?rev=805391&r1=805390&r2=805391&view=diff > > > ============================================================================== > > --- > jakarta/jmeter/trunk/src/components/org/apache/jmeter/visualizers/LineGraph.java > (original) > > +++ > jakarta/jmeter/trunk/src/components/org/apache/jmeter/visualizers/LineGraph.java > Tue Aug 18 12:44:22 2009 > > @@ -68,7 +68,7 @@ > > private static final Paint[] PAINT_ARRAY = {Color.black, > > Color.blue,Color.green,Color.magenta,Color.orange, > > > Color.red,Color.yellow,Color.darkGray,Color.gray,Color.lightGray, > > - Color.pink,Color.cyan};// Using lower-case for JDK 1.3 > compatibility > > + Color.pink,Color.cyan}; > <snip/> > > Nit - comment seems to imply they'd be changed to uppercase at some > point (such as now given JDK move).
I checked, and it seems both upper and lower are equally valid, so I just dropped the comment. > -Rahul > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
