Re: svn commit: r687323 - in /xmlgraphics/fop/trunk/src/java/org/apache/fop/render: afp/AFPRenderer.java pcl/PCLRenderer.java
Dear FopDevs, I've added the patched version and submitted a feature request for retroweaver: https://sourceforge.net/tracker/index.php?func=detailaid=2063970group_id=104240atid=637383 Since the next release date is still a little bit away I hope this can be fixed properly by then. Max Jeremias Maerki schrieb: As long as we don't distribute the unreleased package in one of our own releases, it's fine. We need to apply the same policy as we do for other libraries. If the source distribution doesn't contain those build dependencies, it's ok to keep the unreleased package. On 20.08.2008 18:31:46 Max Berger wrote: Dear Fop-Devs, further insight on retroweaver: I've downloaded and patched retroweaver not to modify Boolean.valueOf, which is now correctly verified against 1.4. I could add the (patched) artifact to fops lib/build. I will try and discuss with the retroweaver maintainer about options to include that into the standard retroweaver distribution. What is the opinion about having patched and unreleased dependencies? Even if it is just for build, and purely optional? Max Max Berger schrieb: Adrian, Adrian Cumiskey schrieb: I don't think it is Max... looks like @since 1.4. you're right - looks like the other valueOf methods (for integer, etc.) where introduced in 1.5, and this one was indeed introduced in 1.4. I've reverted that change. Apparently retroweaver still modifies calls to Boolean.valueOf, according to the doc to be compatible with 1.3. Unfortunately this requires adding the retroweaver-runtime to the verification classpath, which then renders the verification process useless, as it is supposed to detect failures like Integer.valueOf(), which will now again slip through. I'll look deeper into retoweaver to see if I can find a suitable solution. Max Jeremias Maerki signature.asc Description: OpenPGP digital signature
Re: svn commit: r687323 - in /xmlgraphics/fop/trunk/src/java/org/apache/fop/render: afp/AFPRenderer.java pcl/PCLRenderer.java
I don't think it is Max... looks like @since 1.4.
/**
* Returns a ttBoolean/tt instance representing the specified
* ttboolean/tt value. If the specified ttboolean/tt value
* is tttrue/tt, this method returns ttBoolean.TRUE/tt;
* if it is ttfalse/tt, this method returns ttBoolean.FALSE/tt.
* If a new ttBoolean/tt instance is not required, this method
* should generally be used in preference to the constructor
* [EMAIL PROTECTED] #Boolean(boolean)}, as this method is likely to to
yield
* significantly better space and time performance.
*
* @param b a boolean value.
* @return a ttBoolean/tt instance representing ttb/tt.
* @since 1.4
*/
public static Boolean valueOf(boolean b) {
return (b ? TRUE : FALSE);
}
Adrian.
[EMAIL PROTECTED] wrote:
Author: maxberger
Date: Wed Aug 20 06:24:18 2008
New Revision: 687323
URL: http://svn.apache.org/viewvc?rev=687323view=rev
Log:
valueOf is Java 1.5+ only
Modified:
xmlgraphics/fop/trunk/src/java/org/apache/fop/render/afp/AFPRenderer.java
xmlgraphics/fop/trunk/src/java/org/apache/fop/render/pcl/PCLRenderer.java
Modified:
xmlgraphics/fop/trunk/src/java/org/apache/fop/render/afp/AFPRenderer.java
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/src/java/org/apache/fop/render/afp/AFPRenderer.java?rev=687323r1=687322r2=687323view=diff
==
--- xmlgraphics/fop/trunk/src/java/org/apache/fop/render/afp/AFPRenderer.java
(original)
+++ xmlgraphics/fop/trunk/src/java/org/apache/fop/render/afp/AFPRenderer.java
Wed Aug 20 06:24:18 2008
@@ -935,7 +935,7 @@
RendererContext context;
context = super.createRendererContext(x, y, width, height,
foreignAttributes);
context.setProperty(AFPRendererContextConstants.AFP_GRAYSCALE,
-Boolean.valueOf(!this.colorImages));
+new Boolean(!this.colorImages));
return context;
}
Modified:
xmlgraphics/fop/trunk/src/java/org/apache/fop/render/pcl/PCLRenderer.java
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/src/java/org/apache/fop/render/pcl/PCLRenderer.java?rev=687323r1=687322r2=687323view=diff
==
--- xmlgraphics/fop/trunk/src/java/org/apache/fop/render/pcl/PCLRenderer.java
(original)
+++ xmlgraphics/fop/trunk/src/java/org/apache/fop/render/pcl/PCLRenderer.java
Wed Aug 20 06:24:18 2008
@@ -1148,7 +1148,7 @@
RendererContext context = super.createRendererContext(
x, y, width, height, foreignAttributes);
context.setProperty(PCLRendererContextConstants.PCL_COLOR_CANVAS,
-Boolean.valueOf(this.useColorCanvas));
+new Boolean(this.useColorCanvas));
return context;
}
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
Re: svn commit: r687323 - in /xmlgraphics/fop/trunk/src/java/org/apache/fop/render: afp/AFPRenderer.java pcl/PCLRenderer.java
Adrian, Adrian Cumiskey schrieb: I don't think it is Max... looks like @since 1.4. you're right - looks like the other valueOf methods (for integer, etc.) where introduced in 1.5, and this one was indeed introduced in 1.4. I've reverted that change. Apparently retroweaver still modifies calls to Boolean.valueOf, according to the doc to be compatible with 1.3. Unfortunately this requires adding the retroweaver-runtime to the verification classpath, which then renders the verification process useless, as it is supposed to detect failures like Integer.valueOf(), which will now again slip through. I'll look deeper into retoweaver to see if I can find a suitable solution. Max signature.asc Description: OpenPGP digital signature
Re: svn commit: r687323 - in /xmlgraphics/fop/trunk/src/java/org/apache/fop/render: afp/AFPRenderer.java pcl/PCLRenderer.java
Dear Fop-Devs, further insight on retroweaver: I've downloaded and patched retroweaver not to modify Boolean.valueOf, which is now correctly verified against 1.4. I could add the (patched) artifact to fops lib/build. I will try and discuss with the retroweaver maintainer about options to include that into the standard retroweaver distribution. What is the opinion about having patched and unreleased dependencies? Even if it is just for build, and purely optional? Max Max Berger schrieb: Adrian, Adrian Cumiskey schrieb: I don't think it is Max... looks like @since 1.4. you're right - looks like the other valueOf methods (for integer, etc.) where introduced in 1.5, and this one was indeed introduced in 1.4. I've reverted that change. Apparently retroweaver still modifies calls to Boolean.valueOf, according to the doc to be compatible with 1.3. Unfortunately this requires adding the retroweaver-runtime to the verification classpath, which then renders the verification process useless, as it is supposed to detect failures like Integer.valueOf(), which will now again slip through. I'll look deeper into retoweaver to see if I can find a suitable solution. Max signature.asc Description: OpenPGP digital signature
Re: svn commit: r687323 - in /xmlgraphics/fop/trunk/src/java/org/apache/fop/render: afp/AFPRenderer.java pcl/PCLRenderer.java
Hi Max, Personally I'm fine with it until we take a future retroweaver release containing your patch. So +1 from me. Adrian. Max Berger wrote: Dear Fop-Devs, further insight on retroweaver: I've downloaded and patched retroweaver not to modify Boolean.valueOf, which is now correctly verified against 1.4. I could add the (patched) artifact to fops lib/build. I will try and discuss with the retroweaver maintainer about options to include that into the standard retroweaver distribution. What is the opinion about having patched and unreleased dependencies? Even if it is just for build, and purely optional? Max Max Berger schrieb: Adrian, Adrian Cumiskey schrieb: I don't think it is Max... looks like @since 1.4. you're right - looks like the other valueOf methods (for integer, etc.) where introduced in 1.5, and this one was indeed introduced in 1.4. I've reverted that change. Apparently retroweaver still modifies calls to Boolean.valueOf, according to the doc to be compatible with 1.3. Unfortunately this requires adding the retroweaver-runtime to the verification classpath, which then renders the verification process useless, as it is supposed to detect failures like Integer.valueOf(), which will now again slip through. I'll look deeper into retoweaver to see if I can find a suitable solution. Max
Re: svn commit: r687323 - in /xmlgraphics/fop/trunk/src/java/org/apache/fop/render: afp/AFPRenderer.java pcl/PCLRenderer.java
As long as we don't distribute the unreleased package in one of our own releases, it's fine. We need to apply the same policy as we do for other libraries. If the source distribution doesn't contain those build dependencies, it's ok to keep the unreleased package. On 20.08.2008 18:31:46 Max Berger wrote: Dear Fop-Devs, further insight on retroweaver: I've downloaded and patched retroweaver not to modify Boolean.valueOf, which is now correctly verified against 1.4. I could add the (patched) artifact to fops lib/build. I will try and discuss with the retroweaver maintainer about options to include that into the standard retroweaver distribution. What is the opinion about having patched and unreleased dependencies? Even if it is just for build, and purely optional? Max Max Berger schrieb: Adrian, Adrian Cumiskey schrieb: I don't think it is Max... looks like @since 1.4. you're right - looks like the other valueOf methods (for integer, etc.) where introduced in 1.5, and this one was indeed introduced in 1.4. I've reverted that change. Apparently retroweaver still modifies calls to Boolean.valueOf, according to the doc to be compatible with 1.3. Unfortunately this requires adding the retroweaver-runtime to the verification classpath, which then renders the verification process useless, as it is supposed to detect failures like Integer.valueOf(), which will now again slip through. I'll look deeper into retoweaver to see if I can find a suitable solution. Max Jeremias Maerki
