Re: [VOTE] Merge Temp_AFPGOCAResources branch into trunk

2008-11-25 Thread Jeremias Maerki
On 24.11.2008 16:50:48 Adrian Cumiskey wrote: Hi Clay, Yes with the refactoring and code reuse changes in the Temp_AFPGOCAResource branch, other renderers can now also take advantage of creating specialized concrete implementations of AbstractImageHandlerRegistry and ImageHandler.

Re: [VOTE] Merge Temp_AFPGOCAResources branch into trunk

2008-11-25 Thread Jeremias Maerki
On 24.11.2008 16:11:00 Adrian Cumiskey wrote: Hi Chris (and all), Chris Bowditch wrote: Adrian, you've put a lot of work into the GOCA branch and it has some great features, but There's one thing I don't like, which Jeremias points out in this thread:

DO NOT REPLY [Bug 42600] can not create page break in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42600 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --

DO NOT REPLY [Bug 42600] can not create page break in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42600 --- Comment #1 from [EMAIL PROTECTED] 2008-11-25 01:22:00 PST --- Created an attachment (id=22930) -- (https://issues.apache.org/bugzilla/attachment.cgi?id=22930) Patch proposal Tested with latest trunk version -- Configure

DO NOT REPLY [Bug 42600] can not create page break in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42600 --- Comment #2 from [EMAIL PROTECTED] 2008-11-25 01:23:24 PST --- Created an attachment (id=22931) -- (https://issues.apache.org/bugzilla/attachment.cgi?id=22931) Example fo for testing -- Configure bugmail:

DO NOT REPLY [Bug 42600] [PATCH] can not create page break in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42600 [EMAIL PROTECTED] changed: What|Removed |Added Keywords||PatchAvailable

DO NOT REPLY [Bug 42374] List label and bodyindentation incorrect in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42374 --- Comment #1 from [EMAIL PROTECTED] 2008-11-25 02:12:36 PST --- Created an attachment (id=22932) -- (https://issues.apache.org/bugzilla/attachment.cgi?id=22932) Patch proposal Changed several things, also in RtfListItem.java. The

DO NOT REPLY [Bug 42374] List label and bodyindentation incorrect in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42374 --- Comment #3 from [EMAIL PROTECTED] 2008-11-25 02:14:22 PST --- Created an attachment (id=22934) -- (https://issues.apache.org/bugzilla/attachment.cgi?id=22934) Another small test fo -- Configure bugmail:

DO NOT REPLY [Bug 42374] List label and bodyindentation incorrect in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42374 --- Comment #2 from [EMAIL PROTECTED] 2008-11-25 02:13:48 PST --- Created an attachment (id=22933) -- (https://issues.apache.org/bugzilla/attachment.cgi?id=22933) Result of the posted fo -- Configure bugmail:

DO NOT REPLY [Bug 42374] [PATCH] List label and bodyindentation incorrect in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42374 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED]

Re: [VOTE] Merge Temp_AFPGOCAResources branch into trunk

2008-11-25 Thread Jeremias Maerki
On 25.11.2008 09:28:18 Jeremias Maerki wrote: On 24.11.2008 16:11:00 Adrian Cumiskey wrote: Hi Chris (and all), Chris Bowditch wrote: Adrian, you've put a lot of work into the GOCA branch and it has some great features, but There's one thing I don't like, which

Re: [!! SPAM] Re: svn commit: r713747 [1/4] - in /xmlgraphics/fop/branches/Temp_AFPGOCAResources/src: documentation/content/xdocs/trunk/ java/META-INF/services/ java/org/apache/fop/afp/ java/org/apac

2008-11-25 Thread Chris Bowditch
Adrian Cumiskey wrote: Jeremias, Adrian, Jeremias Maerki wrote: I'm sure if such a person existed and they wrote their own FOP plug-in they could very easily fix a very small compatibility issue such as this. Again: it's not the fix that is the problem. It is about

Re: svn commit: r713747 [1/4] - in /xmlgraphics/fop/branches/Temp_AFPGOCAResources/src: documentation/content/xdocs/trunk/ java/META-INF/services/ java/org/apache/fop/afp/ java/org/apache/fop/afp/font

2008-11-25 Thread Adrian Cumiskey
Jeremias, Jeremias Maerki wrote: I'm sure if such a person existed and they wrote their own FOP plug-in they could very easily fix a very small compatibility issue such as this. Again: it's not the fix that is the problem. It is about backwards-compatibility and that's an issue mostly for

DO NOT REPLY [Bug 42374] [PATCH] List label and bodyindentation incorrect in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42374 Chris Bowditch [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED

Re: [VOTE] Merge Temp_AFPGOCAResources branch into trunk

2008-11-25 Thread Adrian Cumiskey
Jeremias, Jeremias Maerki wrote: We currently use the PDF plugin in our software and I prefer not to upgrade the plugin without knowing what the justification the interface change is? Providing support for native image embedding in AFP involves providing the image data (in whatever form it

Re: [VOTE] Merge Temp_AFPGOCAResources branch into trunk

2008-11-25 Thread Jeremias Maerki
On 25.11.2008 13:27:19 Adrian Cumiskey wrote: Jeremias, Jeremias Maerki wrote: We currently use the PDF plugin in our software and I prefer not to upgrade the plugin without knowing what the justification the interface change is? Providing support for native image embedding in AFP

DO NOT REPLY [Bug 43474] wrap-option=wrap doesn't work

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=43474 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --

DO NOT REPLY [Bug 43474] wrap-option=wrap doesn't work

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=43474 --- Comment #11 from [EMAIL PROTECTED] 2008-11-25 05:04:01 PST --- Created an attachment (id=22938) -- (https://issues.apache.org/bugzilla/attachment.cgi?id=22938) Patch proposal I created a patch working with the latest trunk

DO NOT REPLY [Bug 43474] wrap-option=wrap doesn't work

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=43474 --- Comment #12 from [EMAIL PROTECTED] 2008-11-25 05:05:00 PST --- Created an attachment (id=22939) -- (https://issues.apache.org/bugzilla/attachment.cgi?id=22939) Another test fo -- Configure bugmail:

DO NOT REPLY [Bug 43474] wrap-option=wrap doesn't work

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=43474 --- Comment #13 from [EMAIL PROTECTED] 2008-11-25 05:05:53 PST --- Created an attachment (id=22940) -- (https://issues.apache.org/bugzilla/attachment.cgi?id=22940) Result for the last fo -- Configure bugmail:

DO NOT REPLY [Bug 42374] [PATCH] List label and bodyindentation incorrect in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42374 --- Comment #5 from Chris Bowditch [EMAIL PROTECTED] 2008-11-25 05:21:19 PST --- Created an attachment (id=22941) -- (https://issues.apache.org/bugzilla/attachment.cgi?id=22941) RTF after patch applied Are you sure that the patch is

DO NOT REPLY [Bug 43474] [PATCH] wrap-option=wrap doesn't work

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=43474 Chris Bowditch [EMAIL PROTECTED] changed: What|Removed |Added Summary|wrap-option=wrap doesn't |[PATCH]

DO NOT REPLY [Bug 42374] [PATCH] List label and bodyindentation incorrect in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42374 --- Comment #6 from [EMAIL PROTECTED] 2008-11-25 05:40:07 PST --- Created an attachment (id=22942) -- (https://issues.apache.org/bugzilla/attachment.cgi?id=22942) Fo file Sorry for the guideline mistakes. To be generated correctly i

DO NOT REPLY [Bug 46292] New: [PATCH] PS Renderer does not handle a string value for a key in a dictionary.

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=46292 Summary: [PATCH] PS Renderer does not handle a string value for a key in a dictionary. Product: Fop Version: all Platform: All OS/Version: All Status: NEW

DO NOT REPLY [Bug 46294] New: start-indent on list-item is not correctly handled

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=46294 Summary: start-indent on list-item is not correctly handled Product: Fop Version: 0.95 Platform: PC OS/Version: Windows XP Status: NEW Severity: normal

DO NOT REPLY [Bug 42374] [PATCH] List label and bodyindentation incorrect in RTF

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=42374 --- Comment #7 from Chris Bowditch [EMAIL PROTECTED] 2008-11-25 09:04:24 PST --- Thanks for sending the FO. I thought I'd corrected the whitespace but I guess I must have missed something. The first test file then Renders as

DO NOT REPLY [Bug 46294] start-indent on list-item is not correctly handled

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=46294 --- Comment #1 from Chris Bowditch [EMAIL PROTECTED] 2008-11-25 09:10:54 PST --- Just spotted a typo in the bug description. The bug I was testing when I found the bug is actually bug 42374 not 42437 -- Configure bugmail:

Re: [VOTE] Merge Temp_AFPGOCAResources branch into trunk

2008-11-25 Thread Andreas Delmelle
On Nov 24, 2008, at 15:08, Adrian Cumiskey wrote: I would like to call for a merge of the AFP branch [1] back into trunk. snip / If the issue as discussed in the parallel thread is ironed out, I see no reason why not. +1 in that case. Adrian, just a small FWIW: Chris has drawn the

DO NOT REPLY [Bug 43474] [PATCH] wrap-option=wrap doesn't work

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=43474 --- Comment #14 from Andreas L. Delmelle [EMAIL PROTECTED] 2008-11-25 10:56:12 PST --- (In reply to comment #11) I created a patch working with the latest trunk version. It's similar to the last patch. But there were some further

DO NOT REPLY [Bug 46292] [PATCH] PS Renderer does not handle a string value for a key in a dictionary.

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=46292 Adrian Cumiskey [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |RESOLVED

Re: [VOTE] Merge Temp_AFPGOCAResources branch into trunk

2008-11-25 Thread Vincent Hennebert
Hi, Andreas Delmelle wrote: snip / OTOH, I feel compelled to mention that attempting to kill a raised, perfectly legitimate issue in an unreasonable way (by taking it to be a personal offense), may alienate some people... That said, I would myself probably take it as a personal offence too

Re: [VOTE] Merge Temp_AFPGOCAResources branch into trunk

2008-11-25 Thread Adrian Cumiskey
Hi Andreas, 2008/11/25 Andreas Delmelle [EMAIL PROTECTED] Adrian, just a small FWIW: Chris has drawn the right conclusion, I think. No need to take any of the questions/remarks personal (even though I understand this is difficult after the time you've spent on perfecting the code in that

DO NOT REPLY [Bug 45097] Questionable white-space-treatment behavior

2008-11-25 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=45097 --- Comment #14 from Andreas L. Delmelle [EMAIL PROTECTED] 2008-11-25 13:12:37 PST --- (In reply to comment #12) Sorry to chime in so late... Based on my novice analysis, it appears the various KnuthElements provide the following