Don't assert that compression-level=0 produces larger file

than the default compression, since the actual result varies
depending on the version of liblzma.

SVN-Revision: 2811

diff --git a/libarchive/test/test_write_compress_lzma.c b/libarchive/test/test_write_compress_lzma.c
index cd2b9721d8d32681e003577258d70940db58faf3..b1718c1db9068b4ebdb3c5eacac4481d7982e012 100644 (file)
--- a/libarchive/test/test_write_compress_lzma.c
+++ b/libarchive/test/test_write_compress_lzma.c
@@ -186,10 +186,15 @@ DEFINE_TEST(test_write_compress_lzma)
        assertEqualIntA(a, ARCHIVE_OK, archive_write_close(a));
        assertEqualInt(ARCHIVE_OK, archive_write_free(a));
 
-       /* Level 0 really does result in larger data. */
+       /* It would be nice to assert that compression-level=0 produced
+        * consistently larger/smaller results than the default compression,
+        * but the results here vary a lot depending on the version of liblzma
+        * being used. */
+       /* 
        failure("Compression-level=0 wrote %d bytes; default wrote %d bytes",
            (int)used2, (int)used1);
        assert(used2 > used1);
+       */
 
        assert((a = archive_read_new()) != NULL);
        assertEqualIntA(a, ARCHIVE_OK, archive_read_support_format_all(a));

diff --git a/libarchive/test/test_write_compress_xz.c b/libarchive/test/test_write_compress_xz.c
index 2f32df27f2f37ce48970ccdb0437d554ec454e8d..c9da5ab08794de337c8719e10cdd60be00416cb2 100644 (file)
--- a/libarchive/test/test_write_compress_xz.c
+++ b/libarchive/test/test_write_compress_xz.c
@@ -193,10 +193,14 @@ DEFINE_TEST(test_write_compress_xz)
        assertEqualIntA(a, ARCHIVE_OK, archive_write_close(a));
        assertEqualInt(ARCHIVE_OK, archive_write_free(a));
 
-       /* Level 0 really does result in larger data. */
+       /* I would like to assert that compression-level=0 results in
+        * larger data than the default compression, but that's not true
+        * for all versions of liblzma. */
+       /*
        failure("Compression-level=0 wrote %d bytes; default wrote %d bytes",
            (int)used2, (int)used1);
        assert(used2 > used1);
+       */
 
        assert((a = archive_read_new()) != NULL);
        assertEqualIntA(a, ARCHIVE_OK, archive_read_support_format_all(a));