On 2010-08-02, Torsten Curdt wrote: >> class PosixAttributes { >> /* nine constants */ >> static final int OWNER_CAN_READ = ...
>> /* set of the constants that apply */ >> Set getPermissions(); >> /* could be mixed with permissions if we wanted to */ >> Boolean isSetUserId(); >> Boolean isSetGroupId(); >> Boolean hasStickyBit(); >> /* TAR, AR and CPIO know these as well */ >> Boolean isSymbolicLink(); >> Boolean isRegularFile(); ... > Hm ...lots of stuff in there. That's why you need more than a C short to cover everything 8-) > But looks reasonable. Should be all "boolean"s instead of some > "Boolean"s thought, right? Boolean was intentional since ZIP doesn't have any idea whether an entry is a regular file. OTOH it could simply say "false" in that case as well since the format simply doesn't support anything but directories and regular files. So yes, boolean likely makes more sense. Stefan --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org