Hi Jacques,
the 'official' workaround seems to be this:
public <STMNT extends SQLStatement<?>> SQLPlan<?> plan(STMNT statement) {
This also compiles in older Java versions and doesn't show a warning in Eclipse.
Martin
On 15.03.2011 20:19, jler...@apache.org wrote:
> Author: jleroux
> Date: Tue Mar 15 19:19:22 2011
> New Revision: 1081912
>
> URL: http://svn.apache.org/viewvc?rev=1081912&view=rev
> Log:
> Revert partially r1076852 (only
> http://svn.apache.org/viewvc/ofbiz/trunk/framework/sql/src/org/ofbiz/sql/Planner.java?view=diff&r1=1076852&r2=1076853&pathrev=1076853)
>
> This is preventing some java version to compile (So far at least 1.6.0_14 and
> 1.6.0_17 fail. It's ok with 1.6.0_23 and 1.6.0_24)
>
> Marco mades this change recently in his warnings effort. But I don't see
> anything in Eclipse nor while building. So I'm reverting this trivial change
>
>
> Modified:
> ofbiz/trunk/framework/sql/src/org/ofbiz/sql/Planner.java
>
> Modified: ofbiz/trunk/framework/sql/src/org/ofbiz/sql/Planner.java
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/framework/sql/src/org/ofbiz/sql/Planner.java?rev=1081912&r1=1081911&r2=1081912&view=diff
> ==============================================================================
> --- ofbiz/trunk/framework/sql/src/org/ofbiz/sql/Planner.java (original)
> +++ ofbiz/trunk/framework/sql/src/org/ofbiz/sql/Planner.java Tue Mar 15
> 19:19:22 2011
> @@ -37,7 +37,7 @@ public abstract class Planner<P extends
> }
> }
>
> - public SQLPlan<?> plan(SQLStatement<?> statement) {
> + public SQLPlan<?> plan(SQLStatement statement) {
> if (statement instanceof SQLDelete) return planDelete((SQLDelete)
> statement);
> if (statement instanceof SQLInsert) return planInsert((SQLInsert)
> statement);
> if (statement instanceof SQLSelect) return planSelect((SQLSelect)
> statement);
>
>
