[ 
https://issues.apache.org/jira/browse/COMPRESS-404?focusedWorklogId=431331&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-431331
 ]

ASF GitHub Bot logged work on COMPRESS-404:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 06/May/20 17:37
            Start Date: 06/May/20 17:37
    Worklog Time Spent: 10m 
      Work Description: garydgregory commented on a change in pull request #97:
URL: https://github.com/apache/commons-compress/pull/97#discussion_r420970569



##########
File path: 
src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveEntry.java
##########
@@ -384,12 +430,49 @@ public TarArchiveEntry(final File file, final String 
fileName) {
         } else {
             this.mode = DEFAULT_FILE_MODE;
             this.linkFlag = LF_NORMAL;
-            this.size = file.length();
+            try {
+                this.size = Files.size(file);
+            } catch (IOException e) {
+                e.printStackTrace();

Review comment:
       Let's not dump stacks to the console, very messy when used in a larger 
software stack.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 431331)
    Time Spent: 1h 40m  (was: 1.5h)

> Add Path equivalents to TarArchiveEntry, with methods using File delegating 
> to equivalents
> ------------------------------------------------------------------------------------------
>
>                 Key: COMPRESS-404
>                 URL: https://issues.apache.org/jira/browse/COMPRESS-404
>             Project: Commons Compress
>          Issue Type: Sub-task
>            Reporter: Simon Spero
>            Priority: Minor
>          Time Spent: 1h 40m
>  Remaining Estimate: 0h
>
> (This is a reasonable place to start, as Paths give better access to 
> tar-relevant metadata on unix system).
> Symlink info is easier to obtain
> Unix based Paths allow access to useful attributes under "unix:*"  
> - numeric uid and gid 
> - string owner and group names
> - mtime,ctime,atime 
> - numeric mode
> - numeric dev and inode 
> - numeric rdev 
> - Linux, Solaris and Windows allow access to extended attributes 
> (MacOS X xattrs aren't supported as of jdk 9)



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to