search

DhavalSPatel / googlechartwrapper

Automatically exported from code.google.com/p/googlechartwrapper
0 stars 0 forks source link

GridLine ignores blank line value and produces sub-optimal URL encoding #4

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?

plot.setGridLine(new GridLine.GridLineBuilder(30, 15).segment(12, 
2).offset(25, 25).build());

Produces:

chg=30.0,15.0,12.0,12.0,25.0,25.0

length blank segment value is 2 not 12.  Also, where a float is ".0" then 
there is no point having this extra load on the URL.

What is the expected output? What do you see instead?

Corrected and improved output:

chg=30,15,12,2,25,25

What version of the product are you using? On what operating system?

svn trunk

Please provide any additional information below.

Patch attached - I've corrected the length blank issue and ammended 
GridLine.CutSample to optimise the ".0" case.

Original issue reported on code.google.com by chris.lo...@gmail.com on 31 Jan 2010 at 12:49

Attachments:

GoogleCodeExporter commented 9 years ago 
i tried the patch
will be part of the next commitment

thanks a lot :)

Original comment by steffan....@outlook.com on 8 Apr 2010 at 12:27