[RFR] 8204539: improve error messages in matchJavaTZ [windows]

Seán Coffey sean.coffey at oracle.com
Fri Jun 8 07:36:30 UTC 2018


Not sure if this has been tested.

Don't you need to escape the printing of the \0 character ?

errorMessage = "illegal character \0 found";

regards,
Sean.


On 08/06/2018 08:14, Langer, Christoph wrote:
>
> Hi Matthias,
>
> this looks good. Reviewed from my end.
>
> Best regards
>
> Christoph
>
> *From:*Baesken, Matthias
> *Sent:* Freitag, 8. Juni 2018 08:51
> *To:* Langer, Christoph <christoph.langer at sap.com>; 
> core-libs-dev at openjdk.java.net; 'sean.coffey at oracle.com' 
> <sean.coffey at oracle.com>
> *Cc:* Lindenmaier, Goetz <goetz.lindenmaier at sap.com>
> *Subject:* RE: [RFR] 8204539: improve error messages in matchJavaTZ 
> [windows]
>
> Hi,  looks like I uploaded a wrong version of the patch.
>
> Updated webrev is here  :
>
> http://cr.openjdk.java.net/~mbaesken/webrevs/8204539.1/ 
> <http://cr.openjdk.java.net/%7Embaesken/webrevs/8204539.1/>
>
> I followed the advice of Sean and  removed  the  mapFileName   from 
> the output because it is usually a static name .
>
> Updated webrev was going through our internal build/tests .
>
> Best regards, Matthias
>
> *From:*Langer, Christoph
> *Sent:* Donnerstag, 7. Juni 2018 13:52
> *To:* Baesken, Matthias <matthias.baesken at sap.com 
> <mailto:matthias.baesken at sap.com>>; core-libs-dev at openjdk.java.net 
> <mailto:core-libs-dev at openjdk.java.net>
> *Cc:* Lindenmaier, Goetz <goetz.lindenmaier at sap.com 
> <mailto:goetz.lindenmaier at sap.com>>
> *Subject:* RE: [RFR] 8204539: improve error messages in matchJavaTZ 
> [windows]
>
> Hi Matthias,
>
> in line 527, where the actual output is done, I think you would need 
> to replace variable ‘message’ with ‘outputMessage’, otherwise I guess 
> it won’t compile??
>
> Also, mapFileName can’t be used at this place, because it has already 
> been free’ed there.
>
> But in general the additions make sense and will make it easier to 
> find issues in the tzmappings file.
>
> Best regards
>
> Christoph
>
> *From:*Baesken, Matthias
> *Sent:* Donnerstag, 7. Juni 2018 13:35
> *To:* core-libs-dev at openjdk.java.net 
> <mailto:core-libs-dev at openjdk.java.net>
> *Cc:* Langer, Christoph <christoph.langer at sap.com 
> <mailto:christoph.langer at sap.com>>; Lindenmaier, Goetz 
> <goetz.lindenmaier at sap.com <mailto:goetz.lindenmaier at sap.com>>
> *Subject:* [RFR] 8204539: improve error messages in matchJavaTZ [windows]
>
> Hi, could you please  review this small  change that improves  the 
> error messages in matchJavaTZ .
>
> A reason of the failure is added to the message , and also  the offset 
> where the error happened .
>
> Bug :
>
> https://bugs.openjdk.java.net/browse/JDK-8204539
>
> Webrev :
>
> http://cr.openjdk.java.net/~mbaesken/webrevs/8204539/ 
> <http://cr.openjdk.java.net/%7Embaesken/webrevs/8204539/>
>
> Thanks, Matthias
>



More information about the core-libs-dev mailing list