RE: [RFR] 8205525 : Improve exception messages during manifest parsing of jar archives

Thu, 30 Aug 2018 07:52:10 -0700 


Hi  Max, probably   we should add  the  info about the MANIFEST.MF  , for 
example :
 change  getErrorPosition  to 

http://cr.openjdk.java.net/~mbaesken/webrevs/8205525.7/src/java.base/share/classes/java/util/jar/Attributes.java.udiff.html


    static String getErrorPosition(String filename, final int lineNumber) {
        if (filename == null || !jarPathInExceptionText) {
            return "META-INF/MANIFEST.MF  line:" + lineNumber;
       }

       final File file = new File(filename);
        return AccessController.doPrivileged(new PrivilegedAction<String>() {
            public String run() {
                return file.getAbsolutePath() + "!META-INF/MANIFEST.MF  line:" 
+ lineNumber;
         }
   .....


Best regards, Matthias



> -----Original Message-----
> From: Weijun Wang <[email protected]>
> Sent: Donnerstag, 30. August 2018 16:04
> To: Baesken, Matthias <[email protected]>
> Cc: Alan Bateman <[email protected]>; Sean Mullan
> <[email protected]>; Chris Hegarty <[email protected]>;
> [email protected]; [email protected]
> Subject: Re: [RFR] 8205525 : Improve exception messages during manifest
> parsing of jar archives
> 
> 
> 
> > On Aug 30, 2018, at 8:26 PM, Baesken, Matthias
> <[email protected]> wrote:
> >
> >> - What will the output look like? Is it "/tmp/x.jar:100"?
> >>
> >
> > Yes it look like this :
> >
> > line too long (/testdata/jars/file_with_long_line_1.jar:2)
> 
> Is this a little misleading? I think you mean
> 
>    /testdata/jars/file_with_long_line_1.jar!META-INF/MANIFEST.MF:2
> 
> Thanks
> Max





























					Reply via email to