johnzon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Hendrik Dev <>
Subject Re: fix suggestion for JsonStreamParser.isIntegralNumber()
Date Sat, 28 Jun 2014 22:28:18 GMT
"boolean escaped" is never read, so i commented it out to open a
discussion about the purpose of this variable. Syntactically the
variable seems to be unnecessary.

Sorry for mixing things up, will do a separate commit in the future
for every discussion point or fix.
(In the above commit i also renamed a method and fixed a trivial typo)

Is it (in general) ok to post github commit proposals and discuss
these before i create jira tickets/patches?


On Sun, Jun 29, 2014 at 12:00 AM, Romain Manni-Bucau
<> wrote:
> Same as the other thread, do you care creating a patch and attaching to a jira?
> BTW you removed boolean escaped = false;, I guess cause of your other
> patch. Would be great to not mix them ;)
> Romain Manni-Bucau
> Twitter: @rmannibucau
> Blog:
> LinkedIn:
> Github:
> 2014-06-28 23:25 GMT+02:00 Hendrik Dev <>:
>> isIntegralNumber() also have to throw a IllegalStateException
>> isIntegralNumber() does now throw ise, little renaming, cleanup of
>> possibly unnecessary code
>> Thanks
>> Hendrik

View raw message