Github user kinow commented on a diff in the pull request:

    https://github.com/apache/jena/pull/114#discussion_r175998281
  
    --- Diff: jena-arq/Grammar/master.jj ---
    @@ -2176,6 +2251,19 @@ String String() : { Token t ; String lex ; }
         }
     }
     
    +#ifdef ARQ
    +Number Number() : { Token t ; Number number ; }
    +{
    +  (
    +    t = < INTEGER > { number = integerValue(t.image) ; }
    +  | t = < DECIMAL > { number = doubleValue(t.image) ; }
    --- End diff --
    
    Oh, no reason at all. Just forgot about it I think. Re-reading that method, 
I almost feel like removing the number parameter, and just replacing by `return 
integetValue(t.image);` etc. But will just add the `< DOUBLE >` for now :-)


---

Reply via email to