Page MenuHomeFreeBSD

tarfs: Silence warnings when no supported compression is enabled.
ClosedPublic

Authored by des on Feb 7 2023, 1:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
May 13 2024, 3:57 PM
Unknown Object (File)
Dec 23 2023, 12:12 AM
Unknown Object (File)
Nov 13 2023, 9:25 AM
Unknown Object (File)
Oct 10 2023, 8:18 AM
Unknown Object (File)
Jul 10 2023, 12:44 AM
Unknown Object (File)
Jul 10 2023, 12:43 AM
Unknown Object (File)
Jul 10 2023, 12:42 AM
Unknown Object (File)
Jul 5 2023, 1:36 PM
Subscribers

Details

Summary

Sponsored by: Juniper Networks, Inc.
Sponsored by: Klara, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des requested review of this revision.Feb 7 2023, 1:27 PM
sys/fs/tarfs/tarfs_io.c
43

Why not write #ifdef ZSTDIO, same as in other places?

And why do you need TARFS_ZIO, instead of reusing ZSTDIO?

des marked an inline comment as done.Feb 7 2023, 3:03 PM
des added inline comments.
sys/fs/tarfs/tarfs_io.c
43

Because I have branches with support for zlib and xz compression as well. The code which is #ifdef TARFS_ZIO is common to all three, not specific to zstd. I didn't include the other two in the initial code drop because they don't support seekable compression without upstream changes.

kib added inline comments.
sys/fs/tarfs/tarfs_io.c
43

Then at least add this explanation to the commit message. Otherwise it would cause the same question from other people, I am sure.

This revision is now accepted and ready to land.Feb 7 2023, 4:25 PM
This revision was automatically updated to reflect the committed changes.
des marked an inline comment as done.