vapier / ncompress

a fast, simple LZW file compressor
https://vapier.github.io/ncompress/
The Unlicense
104 stars 18 forks source link

The error "filename too long to tack on .Z" #12

Closed praiskup closed 5 years ago

praiskup commented 6 years ago

https://github.com/vapier/ncompress/blob/4e663b112481344684d55768361d3b36517c3154/compress42.c#L1037

Can anyone please explain to me why that if () block is needed? I'm considering to fill a PR which will drop that entirely. Any objections?

vapier commented 5 years ago

the check is for old DOS/Windows systems with 8.3 filename limitations. on those systems, the check is necessary, but i don't think we need to care about them anymore. if anyone is still running such a system, they can patch the code themselves, or use a distro like FreeDOS to get compress support.