Hello Thomas,
Thank for your comments. here are my answers :
Thomas DeWeese wrote:
XmlWriter changes right now I'm not going to accept due to some issues:
1) Your updates removed the use of bulk writes which was
specifically added due to performance issues.
Sorry, maybe I first modify this file before this change, and I was a
bit careless and copied / replaced too much code ...
2) The change to encode all chars >= 0x80 as hex is not friendly
for foreign language users (they might view the document in an
editor that really displays those things). This is one of the
intents of encoding to handle. At most this might be made a option
for when writing the document.
You are right.
SVGGraphics2D
Line 223 in patch:
What's with the null paint stuff? It looks like it isn't used.
I wrote this part some time (I think a long time) ago to convert an old
proprietary graphic format were there was a "non visible" color, but I
didn't choose the right way to do this, and later the code remained
there, harmless but not useful. So : this code is not really used and
can be removed.
Lines 272 & 292 in patch:
I'm not sure that the 'solution' of setting stroke to the
'textColor' is a good one - most text doesn't want to be stroked
at all. Can you describe exactly what the problem is?
Should we be setting stroke to 'none'?
I'm not able to see any more what was my problem at the moment (I'm
pretty sure I encountered one that dealed with definitions of text
characteristics higher in the SVG tree than the 'text' tag, but I can't
reproduce the problem for the moment, and beside it could have been my
fault ...). I think it better to get back to the original code for that
part. I will file a bug if I finally find a SVG file to reproduce the
original problem (an if there was really a problem at all).
Line 289:
In what case are we missing setting the font-size correctly?
Is this for cases where you are using 'getRoot' repeatedly?
If so then much more needs to be done I think...
Same as the above point.
Line 305:
You deleted the restoration of the regular transform after a
supplemental transform was needed to draw text (needed for very
small fonts).
I think that's for the same reason as for your first remark : careless
copy / replace of code.
Line 329:
While I appreciate the very good attempt at handling of attributed
Character iterators, I am concerned that the code only considers
the attributes on the first char.
I needed this to handle easily decorated text in WMF file. I began
simple, and I thought that I would handle the attributes on other chars
as well, but later I forgot about it...
These attributes can change
on a per char basis - requiring the creation of tspans. Perhaps
check code can be added to see if the attributes are constant
(see 'ACI.getRunLimit(Set attributes)' and use the improved code
in that case and use the outline path it in the more complex cases?
That's a good idea.
Line 430:
I'm not thrilled with the mapping of Justifications. Once again
I would lean towards mapping zero to 'start', one to 'end'
.5 to middle, any other value can either be done with 'dx' or
simply stroke it. This should handle the most common cases nicely
and not cause regressions for the other cases.
Yes, using 'dx' could be better. putting CSS_MIDDLE_VALUE for values
between 0.3 and 0.7 was not a so bright idea, after looking at it again...
Line 3936:
.nbattrs (Net beans something file) in wmf/tosvg needed?
No sorry, it is just a Netbeans file, don't now why it landed there...
Really annoy bits: ;)
You seem to have been a bit careless with the Copyright lines.
Most of the new classes say 2001, in a few cases you deleted
existing years in a Copyright.
Really I am unlucky and careless !! I picked up a file in Batik SVN
repository at random and I must have accidentally stumbled on classes
which have the old header (I should have checked after, I know...). It
turns out that some of the classes still have the old header. Not a
reason for me to add new ones ;)
Finally:
Can I add the 'test' classes to Batik? They have the Apache License
so I assume so but I just wanted to make sure. I will probably
add the WMF viewer to the 'contrib' directory and try and figure
out how to merge the test transcoder into the Batik test framework.
Yes, I put the licence header in those files so you could put it in some
Batik directory if you thought they could be useful.
If you want, I can modify the files according to your remarks and file
the new version of the patch attached to the RFE.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
