Thanks Martin,

Committed at r1082475

I still wonder why we care about that ;o)

Jacques

From: "Martin Kreidenweis" <martin.kreidenw...@tngtech.com>
Hi,

Ignore the 'official'... :-)
It's just a workaround I found on several pages during my Google search.
I tested with Java 1.6.0_17 and 1.6.0_24.

Martin


On 17.03.2011 12:54, Jacques Le Roux wrote:
Compiles also with 6.0_14, sounds good
Still Why official?

Thks

Jacques

From: "Jacques Le Roux" <jacques.le.r...@les7arts.com>
Hi Martin,

Yes, it compiles just fine in 6.0_22
Why official? I guess you tested it with an older version, which one?

Thanks

Jacques

From: "Martin Kreidenweis" <martin.kreidenw...@tngtech.com>
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);









--
Martin Kreidenweis
Software Consultant
TNG Technology Consulting
martin.kreidenw...@tngtech.com - +49 176 10284831

TNG Technology Consulting GmbH, Betastr. 13a, 85774 Unterföhring
Geschäftsführer: Henrik Klagges, Gerhard Müller, Christoph Stock
Sitz: Unterföhring * Amtsgericht München * HRB 135082



Reply via email to