1 Sep 2005 01:36
Re: DO NOT REPLY [Bug 35918] - [PATCH] Shapes are slightly shifted with respect to lines on larger graphics
Thomas DeWeese <Thomas.DeWeese <at> Kodak.com>
2005-08-31 23:36:12 GMT
2005-08-31 23:36:12 GMT
Jeremias Maerki wrote:
> Thanks for the hint. Sounds like rewriting PDFNumber.doubleOut using
> DecimalFormat makes sense. May I ask what you used before switching to
> DecimalFormat?
Double.toString(double d),
I just took a look, whats with the PDFNumber.doubleOut function????
It rounds numbers n+0.05 > x > n-0.05 to n (where n is an integer)?
That seems really hacky ;)
Also I can almost assure you that NumberFormat will be _much_
faster than the current code.
> On 31.08.2005 14:32:01 Thomas DeWeese wrote:
>
>>>I did a few things on PDFGraphics2D to improve the situation:
>>>http://svn.apache.org/viewcvs?rev=240344&view=rev
>>
>> BTW Batik saw a significant performance increase when we went
>>to using a DecimalFormat object to write our strings.
>
>
>
>
> Jeremias Maerki
>
> However, with this ever growing number I wonder if we should put
> some more structure to it. There is a real chance that we get more and
> more duplication just because people wouldn't know which tests are
> doing what so one starts writing new ones which may already be covered.
I agree.
> I don't want to suggest some complex system with the associated
> management, setup and on-going compliance overhead. But what about
> some simple naming system along the following lines:
>
> Most current tests (not all) cover a particular feature and can be
> described by the fo they target, the property they exercise and the
> particular aspect of that combination they test. Therefore giving each
> test file a name constructed like <fo name>[-<property name>]?
> [-<feature>]?[<serial number>]?.xml., e.g.
>
> table-padding-relative.xml will test relative padding values on a
> fo:table element.
>
> Yes, this will give us some longer names but will make looking for a
> particular test much easier as simple directory search/sort/filter
RSS Feed