[ checkstyle-Bugs-2088193 ] Still problems with array brackets in wildcard generic types

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[ checkstyle-Bugs-2088193 ] Still problems with array brackets in wildcard generic types

SourceForge.net
Bugs item #2088193, was opened at 2008-09-01 22:20
Message generated for change (Comment added) made by
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397078&aid=2088193&group_id=29721

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core Framework
Group: release 4.4
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Thorsten (schleinzer)
Assigned to: Nobody/Anonymous (nobody)
Summary: Still problems with array brackets in wildcard generic types

Initial Comment:
Related to bug: 1394014

In 4.4, we still have the problem, that the following code does not get parsed properly but Checkstyle still tells "Got an exception - expecting EOF, found '}'"


import java.util.regex.Matcher;
import java.util.regex.Pattern;

import org.kohsuke.args4j.CmdLineParser;
import org.kohsuke.args4j.OptionDef;
import org.kohsuke.args4j.spi.OneArgumentOptionHandler;
import org.kohsuke.args4j.spi.Setter;

public class PositionOptionHandler extends OneArgumentOptionHandler<float[]> {

        Pattern positionPattern = Pattern.compile("(\\d+),(\\d+)(?:,(\\d+))?");

        public PositionOptionHandler(CmdLineParser parser,OptionDef option,Setter<? super float[]> setter) {
                super(parser, option, setter);
        }
}

----------------------------------------------------------------------

Comment By: https://www.google.com/accounts ()
Date: 2011-11-08 23:23

Message:
It's still an issue with checkstlye 5.4. The bad thing about it is, that
your build will fail even if you use such syntax in code that is excluded
from checkstyle check (e.g. in Tests)

----------------------------------------------------------------------

Comment By: Nicolas  Dordet (ndox)
Date: 2008-12-01 08:24

Message:
I try it with checkstyle5-beta01, I've got the same problem.
If i put in comment " // super(parser, option, setter);", the problem
disappear...

----------------------------------------------------------------------

Comment By: Oliver Burn (oburn)
Date: 2008-09-01 22:28

Message:
Logged In: YES
user_id=218824
Originator: NO

Can you try with 5.0 beta 1   --- this version is better for Java 5 code.

----------------------------------------------------------------------

You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=397078&aid=2088193&group_id=29721

------------------------------------------------------------------------------
RSA(R) Conference 2012
Save $700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
_______________________________________________
Checkstyle-devel mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/checkstyle-devel