List Archive
Thread
-
Why should I return the size in ZIP_SOURCE_STAT?,
Andreas Falkenhahn
(2017/10/31 15:44:14)
-
Re: Why should I return the size in ZIP_SOURCE_STA,
Dieter Baron
(2017/11/01 09:10:13)
-
Re: Why should I return the size in ZIP_SOURCE_STA,
Andreas Falkenhahn
(2017/11/01 15:16:39)
- Re: Why should I return the size in ZIP_SOURCE_STA, Dieter Baron (2017/11/02 11:13:15)
- Re: Why should I return the size in ZIP_SOURCE_STA, Andreas Falkenhahn (2017/11/02 20:10:12)
-
Re: Why should I return the size in ZIP_SOURCE_STA,
Andreas Falkenhahn
(2017/11/01 15:16:39)
-
Re: Why should I return the size in ZIP_SOURCE_STA,
Dieter Baron
(2017/11/01 09:10:13)
Message
hi, > On 31.10.2017, at 16:44 , Andreas Falkenhahn > <andreas%falkenhahn.com@localhost> wrote: > > > I'm using a custom source to feed data into zip_file_add(). According to the > docs, I have to implement ZIP_SOURCE_STAT for that. I use ZIP_SOURCE_STAT > to tell zip_file_add() the datestamp for my file but I don't see why I > should ever set ZIP_STAT_SIZE? Isn't that completely gratuitous because > the size can be automatically computed from the number of bytes fed by > ZIP_SOURCE_READ? > > I've just done some tests and it seems that passing a size in ZIP_SOURCE_STAT > isn't necessary at all. Still, the docs sound like it needs to be set: > > Information only available after the source has been read (e.g. size) > can > be omitted in an earlier call. > > To me this sounds like I don't have to submit a size when ZIP_SOURCE_STAT > is first called but I should submit it in the ZIP_SOURCE_STAT call that > comes in after all data has been read. But apparently that isn't necessary. The only information you have to report accurately is ZIP_STAT_COMP_METHOD, and ZIP_STAT_ENCRYPTION_METHOD (and ZIP_STAT_SIZE and ZIP_STAT_CRC if your data is compressed or encrypted). libzip can’t figure those out on its own. All other data is optional, but libzip uses it if present. So you should generally report all information that is readily available, but needn’t go out of your way to obtain any of it. (For instance, zip_source_file reports ZIP_STAT_SIZE and ZIP_STAT_MTIME obtained via calling stat(3). > So I think the docs should be updated to mention that passing the size > in ZIP_SOURCE_STAT isn't really necessary... (unless I'm missing something > here) Yes, we’ll update the man page to make this clearer. thanks, dillo > > -- > Best regards, > Andreas Falkenhahn > mailto:andreas%falkenhahn.com@localhost
Made by MHonArc.