I did not try it on real code so you are probably right. I will try to
merge it on the branch tomorrow and see were it gets me and if I can
simplify the type variables and bounds.

--
Arnaud Bailly
FoldLabs Associate: http://foldlabs.com

I sell my house: http://bailly.me/


On Thu, Jun 27, 2013 at 9:39 PM, Xikilze <notifications@github.com> wrote:

> In src/main/java/org/codehaus/jparsec/LocatableParser2.java:
>
> > + * Licensed under the Apache License, Version 2.0 (the "License");           *
> > + * you may not use this file except in compliance with the License.          *
> > + * You may obtain a copy of the License at                                   *
> > + *                                                                           *
> > + * http://www.apache.org/licenses/LICENSE-2.0                                *
> > + *                                                                           *
> > + * Unless required by applicable law or agreed to in writing, software       *
> > + * distributed under the License is distributed on an "AS IS" BASIS,         *
> > + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.  *
> > + * See the License for the specific language governing permissions and       *
> > + * limitations under the License.                                            *
> > + *****************************************************************************/
> > +
> > +package org.codehaus.jparsec;
> > +
> > +public class LocatableParser2<T, U> extends Parser<T> {
>
> I think LocatableParser2 needs two parametrized types : T for the parser
> reference, U for the handler and the typecast of ctx.result.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/abailly/jparsec/pull/7/files#r4924767>
> .
>