summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat (limited to '')
-rw-r--r--ChangeLog10
-rw-r--r--LzFind.c1
-rw-r--r--LzmaEnc.c6
-rw-r--r--Makefile.in5
-rw-r--r--NEWS16
-rw-r--r--README2
-rw-r--r--carg_parser.c29
-rw-r--r--carg_parser.h10
-rwxr-xr-xconfigure6
-rw-r--r--doc/pdlzip.17
-rw-r--r--main.c477
-rw-r--r--pdlzip.h72
-rwxr-xr-xtestsuite/check.sh8
-rw-r--r--testsuite/test.lz (renamed from testsuite/test1.lz)bin11540 -> 11540 bytes
-rw-r--r--testsuite/test.lzmabin0 -> 11535 bytes
-rw-r--r--testsuite/test.txt (renamed from testsuite/test1)0
16 files changed, 414 insertions, 235 deletions
diff --git a/ChangeLog b/ChangeLog
index cee7979..a489100 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2010-08-19 Antonio Diaz Diaz <ant_diaz@teleline.es>
+
+ * Version 1.1 released.
+ * main.c: Added support for decompression of lzma-alone files.
+ * main.c: Match length limit set by options -1 to -8 has been
+ reduced to extend range of use towards gzip. Lower numbers now
+ compress less but faster. (-1 now takes 43% less time for only
+ 20% larger compressed size).
+ * Code has been converted to "C89 + long long" from C99.
+
2010-04-05 Antonio Diaz Diaz <ant_diaz@teleline.es>
* Version 1.0 released.
diff --git a/LzFind.c b/LzFind.c
index bec8b3e..a2a6636 100644
--- a/LzFind.c
+++ b/LzFind.c
@@ -3,7 +3,6 @@
#define _FILE_OFFSET_BITS 64
-#include <stdbool.h>
#include <stdint.h>
#include <string.h>
diff --git a/LzmaEnc.c b/LzmaEnc.c
index dfba154..fba2349 100644
--- a/LzmaEnc.c
+++ b/LzmaEnc.c
@@ -3,7 +3,6 @@
#define _FILE_OFFSET_BITS 64
-#include <stdbool.h>
#include <stdint.h>
#include <stdio.h>
#include <string.h>
@@ -1562,18 +1561,17 @@ static UInt32 GetOptimumFast(CLzmaEnc *p, UInt32 *backRes)
static void LZe_full_flush(CLzmaEnc *p, UInt32 posState)
{
- UInt32 len;
+ const UInt32 len = LZMA_MATCH_LEN_MIN;
+ File_trailer trailer;
RangeEnc_EncodeBit(&p->rc, &p->isMatch[p->state][posState], 1);
RangeEnc_EncodeBit(&p->rc, &p->isRep[p->state], 0);
p->state = kMatchNextStates[p->state];
- len = LZMA_MATCH_LEN_MIN;
LenEnc_Encode2(&p->lenEnc, &p->rc, len - LZMA_MATCH_LEN_MIN, posState, !p->fastMode, p->ProbPrices);
RcTree_Encode(&p->rc, p->posSlotEncoder[GetLenToPosState(len)], kNumPosSlotBits, (1 << kNumPosSlotBits) - 1);
RangeEnc_EncodeDirectBits(&p->rc, (((UInt32)1 << 30) - 1) >> kNumAlignBits, 30 - kNumAlignBits);
RcTree_ReverseEncode(&p->rc, p->posAlignEncoder, kNumAlignBits, kAlignMask);
RangeEnc_FlushData(&p->rc);
RangeEnc_FlushStream(&p->rc);
- File_trailer trailer;
Ft_set_data_crc( trailer, p->matchFinderBase.crc ^ 0xFFFFFFFFU );
Ft_set_data_size( trailer, p->nowPos64 );
Ft_set_member_size( trailer, p->rc.processed + Fh_size + Ft_size );
diff --git a/Makefile.in b/Makefile.in
index 2bd411d..724badf 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -96,8 +96,9 @@ dist : doc
$(DISTNAME)/configure \
$(DISTNAME)/doc/$(progname).1 \
$(DISTNAME)/testsuite/check.sh \
- $(DISTNAME)/testsuite/test1 \
- $(DISTNAME)/testsuite/test1.lz \
+ $(DISTNAME)/testsuite/test.txt \
+ $(DISTNAME)/testsuite/test.lz \
+ $(DISTNAME)/testsuite/test.lzma \
$(DISTNAME)/*.h \
$(DISTNAME)/*.c
rm -f $(DISTNAME)
diff --git a/NEWS b/NEWS
index 327a372..f6d4638 100644
--- a/NEWS
+++ b/NEWS
@@ -1,5 +1,15 @@
-Changes in version 1.0:
+Changes in version 1.1:
-Initial release.
+Support for decompression of lzma-alone (.lzma) files has been added.
-Using LZMA SDK 9.10 from Igor Pavlov.
+Match length limit set by options -1 to -8 has been reduced to extend
+range of use towards gzip. Lower numbers now compress less but faster.
+(-1 now takes 43% less time for only 20% larger compressed size).
+
+(Note that the bidimensional parameter space of LZMA can't be mapped to
+a linear scale optimal for all files. If your files are large, very
+repetitive, etc, you may need to use the --match-length and
+--dictionary-size options directly to achieve optimal performance).
+
+Code has been converted to "C89 + long long". A C99 compiler is no more
+needed.
diff --git a/README b/README
index e9d196e..0b6427d 100644
--- a/README
+++ b/README
@@ -6,6 +6,8 @@ gzip or bzip2. Pdlzip decompresses almost as fast as gzip and compresses
better than bzip2, which makes it well suited for software distribution
and data archiving.
+Pdlzip is also able to decompress legacy lzma-alone (.lzma) files.
+
Pdlzip is a public domain version of the lzip data compressor, intended
for those who can't (or do not want) distribute GPL licensed Free
Software. Pdlzip is written in C.
diff --git a/carg_parser.c b/carg_parser.c
index 793907b..f797b75 100644
--- a/carg_parser.c
+++ b/carg_parser.c
@@ -77,16 +77,16 @@ static char parse_long_option( struct Arg_parser * const ap,
for( len = 0; opt[len+2] && opt[len+2] != '='; ++len ) ;
- // Test all long options for either exact match or abbreviated matches.
+ /* Test all long options for either exact match or abbreviated matches. */
for( i = 0; options[i].code != 0; ++i )
if( options[i].name && !strncmp( options[i].name, &opt[2], len ) )
{
- if( strlen( options[i].name ) == len ) // Exact match found
+ if( strlen( options[i].name ) == len ) /* Exact match found */
{ index = i; exact = 1; break; }
- else if( index < 0 ) index = i; // First nonexact match found
+ else if( index < 0 ) index = i; /* First nonexact match found */
else if( options[index].code != options[i].code ||
options[index].has_arg != options[i].has_arg )
- ambig = 1; // Second or later nonexact match found
+ ambig = 1; /* Second or later nonexact match found */
}
if( ambig && !exact )
@@ -96,7 +96,7 @@ static char parse_long_option( struct Arg_parser * const ap,
return 1;
}
- if( index < 0 ) // nothing found
+ if( index < 0 ) /* nothing found */
{
add_error( ap, "unrecognized option `" ); add_error( ap, opt );
add_error( ap, "'" );
@@ -105,7 +105,7 @@ static char parse_long_option( struct Arg_parser * const ap,
++*argindp;
- if( opt[len+2] ) // `--<long_option>=<argument>' syntax
+ if( opt[len+2] ) /* `--<long_option>=<argument>' syntax */
{
if( options[index].has_arg == ap_no )
{
@@ -143,14 +143,15 @@ static char parse_short_option( struct Arg_parser * const ap,
const struct ap_Option options[],
int * const argindp )
{
- int cind = 1; // character index in opt
+ int cind = 1; /* character index in opt */
while( cind > 0 )
{
int index = -1;
int i;
const unsigned char code = opt[cind];
- const char code_str[2] = { code, 0 };
+ char code_str[2];
+ code_str[0] = code; code_str[1] = 0;
if( code != 0 )
for( i = 0; options[i].code; ++i )
@@ -163,7 +164,7 @@ static char parse_short_option( struct Arg_parser * const ap,
return 1;
}
- if( opt[++cind] == 0 ) { ++*argindp; cind = 0; } // opt finished
+ if( opt[++cind] == 0 ) { ++*argindp; cind = 0; } /* opt finished */
if( options[index].has_arg != ap_no && cind > 0 && opt[cind] )
{
@@ -191,9 +192,9 @@ char ap_init( struct Arg_parser * const ap,
const int argc, const char * const argv[],
const struct ap_Option options[], const char in_order )
{
- const char ** non_options = 0; // skipped non-options
- int non_options_size = 0; // number of skipped non-options
- int argind = 1; // index in argv
+ const char ** non_options = 0; /* skipped non-options */
+ int non_options_size = 0; /* number of skipped non-options */
+ int argind = 1; /* index in argv */
int i;
ap->data = 0;
@@ -207,13 +208,13 @@ char ap_init( struct Arg_parser * const ap,
const unsigned char ch1 = argv[argind][0];
const unsigned char ch2 = ( ch1 ? argv[argind][1] : 0 );
- if( ch1 == '-' && ch2 ) // we found an option
+ if( ch1 == '-' && ch2 ) /* we found an option */
{
const char * const opt = argv[argind];
const char * const arg = (argind + 1 < argc) ? argv[argind+1] : 0;
if( ch2 == '-' )
{
- if( !argv[argind][2] ) { ++argind; break; } // we found "--"
+ if( !argv[argind][2] ) { ++argind; break; } /* we found "--" */
else if( !parse_long_option( ap, opt, arg, options, &argind ) ) return 0;
}
else if( !parse_short_option( ap, opt, arg, options, &argind ) ) return 0;
diff --git a/carg_parser.h b/carg_parser.h
index 951b787..2d5fd0f 100644
--- a/carg_parser.h
+++ b/carg_parser.h
@@ -41,8 +41,8 @@ enum ap_Has_arg { ap_no, ap_yes, ap_maybe };
struct ap_Option
{
- int code; // Short option letter or code ( code != 0 )
- const char * name; // Long option name (maybe null)
+ int code; /* Short option letter or code ( code != 0 ) */
+ const char * name; /* Long option name (maybe null) */
enum ap_Has_arg has_arg;
};
@@ -71,11 +71,11 @@ void ap_free( struct Arg_parser * const ap );
const char * ap_error( const struct Arg_parser * const ap );
- // The number of arguments parsed (may be different from argc)
+ /* The number of arguments parsed (may be different from argc) */
int ap_arguments( const struct Arg_parser * const ap );
- // If ap_code( i ) is 0, ap_argument( i ) is a non-option.
- // Else ap_argument( i ) is the option's argument (or empty).
+ /* If ap_code( i ) is 0, ap_argument( i ) is a non-option.
+ Else ap_argument( i ) is the option's argument (or empty). */
int ap_code( const struct Arg_parser * const ap, const int i );
const char * ap_argument( const struct Arg_parser * const ap, const int i );
diff --git a/configure b/configure
index 57a3f20..7d572f9 100755
--- a/configure
+++ b/configure
@@ -5,12 +5,12 @@
# This configure script is free software: you have unlimited permission
# to copy, distribute and modify it.
#
-# Date of this version: 2010-04-05
+# Date of this version: 2010-08-19
args=
no_create=
pkgname=pdlzip
-pkgversion=1.0
+pkgversion=1.1
progname=pdlzip
srctrigger=pdlzip.h
@@ -27,7 +27,7 @@ mandir='$(datadir)/man'
sysconfdir='$(prefix)/etc'
CC=
CPPFLAGS=
-CFLAGS='-Wall -W -O2 -std=c99'
+CFLAGS='-Wall -W -O2'
LDFLAGS=
# Loop over all args
diff --git a/doc/pdlzip.1 b/doc/pdlzip.1
index c4a785e..4809683 100644
--- a/doc/pdlzip.1
+++ b/doc/pdlzip.1
@@ -1,12 +1,13 @@
.\" DO NOT MODIFY THIS FILE! It was generated by help2man 1.37.1.
-.TH PDLZIP "1" "April 2010" "Pdlzip 1.0" "User Commands"
+.TH PDLZIP "1" "August 2010" "Pdlzip 1.1" "User Commands"
.SH NAME
Pdlzip \- data compressor based on the LZMA algorithm
.SH SYNOPSIS
.B pdlzip
[\fIoptions\fR] [\fIfile\fR]
.SH DESCRIPTION
-Pdlzip \- A data compressor based on the LZMA algorithm.
+Pdlzip \- A "public domain" version of the lzip data compressor
+able to decompress legacy lzma\-alone (.lzma) files.
.SH OPTIONS
.TP
\fB\-h\fR, \fB\-\-help\fR
@@ -22,7 +23,7 @@ send output to standard output
decompress
.TP
\fB\-m\fR, \fB\-\-match\-length=\fR<n>
-set match length limit in bytes [80]
+set match length limit in bytes [36]
.TP
\fB\-q\fR, \fB\-\-quiet\fR
suppress all messages
diff --git a/main.c b/main.c
index 06725d4..b5e35df 100644
--- a/main.c
+++ b/main.c
@@ -15,7 +15,6 @@
#include <errno.h>
#include <limits.h>
#include <signal.h>
-#include <stdbool.h>
#include <stdint.h>
#include <stdio.h>
#include <stdlib.h>
@@ -32,6 +31,18 @@
#error "Environments where CHAR_BIT != 8 are not supported."
#endif
+#ifndef LLONG_MAX
+#define LLONG_MAX 0x7FFFFFFFFFFFFFFFLL
+#endif
+#ifndef LLONG_MIN
+#define LLONG_MIN (-LLONG_MAX - 1LL)
+#endif
+#ifndef ULLONG_MAX
+#define ULLONG_MAX 0xFFFFFFFFFFFFFFFFULL
+#endif
+
+long long int llabs( long long int number );
+
static void *SzAlloc(void *p, size_t size) { p = p; return MyAlloc(size); }
static void SzFree(void *p, void *address) { p = p; MyFree(address); }
@@ -44,21 +55,22 @@ const char * const program_year = "2010";
struct { const char * from; const char * to; } const known_extensions[] = {
- { ".lz", "" },
- { ".tlz", ".tar" },
- { 0, 0 } };
+ { ".lz", "" },
+ { ".tlz", ".tar" },
+ { ".lzma", "" },
+ { 0, 0 } };
struct Lzma_options
{
- int dictionary_size; // 4KiB..512MiB
- int match_len_limit; // 5..273
+ int dictionary_size; /* 4KiB..512MiB */
+ int match_len_limit; /* 5..273 */
};
enum Mode { m_compress = 0, m_decompress, m_test };
char * output_filename = 0;
-// assure at least a minimum size for buffer `buf'
+/* assure at least a minimum size for buffer `buf' */
inline void * resize_buffer( void * buf, const int min_size )
{
if( buf ) buf = realloc( buf, min_size );
@@ -69,16 +81,17 @@ inline void * resize_buffer( void * buf, const int min_size )
static void show_help()
{
- printf( "%s - A data compressor based on the LZMA algorithm.\n", Program_name );
+ printf( "%s - A \"public domain\" version of the lzip data compressor\n", Program_name );
+ printf( "able to decompress legacy lzma-alone (.lzma) files.\n" );
printf( "\nUsage: %s [options] [file]\n", invocation_name );
printf( "\nOptions:\n" );
printf( " -h, --help display this help and exit\n" );
printf( " -V, --version output version information and exit\n" );
printf( " -c, --stdout send output to standard output\n" );
printf( " -d, --decompress decompress\n" );
-// printf( " -f, --force overwrite existing output files\n" );
-// printf( " -k, --keep keep (don't delete) input files\n" );
- printf( " -m, --match-length=<n> set match length limit in bytes [80]\n" );
+/* printf( " -f, --force overwrite existing output files\n" ); */
+/* printf( " -k, --keep keep (don't delete) input files\n" ); */
+ printf( " -m, --match-length=<n> set match length limit in bytes [36]\n" );
printf( " -q, --quiet suppress all messages\n" );
printf( " -s, --dictionary-size=<n> set dictionary size limit in bytes [8MiB]\n" );
printf( " -t, --test test compressed file integrity\n" );
@@ -91,7 +104,7 @@ static void show_help()
printf( "Numbers may be followed by a multiplier: k = kB = 10^3 = 1000,\n" );
printf( "Ki = KiB = 2^10 = 1024, M = 10^6, Mi = 2^20, G = 10^9, Gi = 2^30, etc...\n" );
printf( "\nReport bugs to lzip-bug@nongnu.org\n" );
-// printf( "Pdlzip home page: http://www.nongnu.org/lzip/pdlzip.html\n" );
+/* printf( "Pdlzip home page: http://www.nongnu.org/lzip/pdlzip.html\n" ); */
}
@@ -105,22 +118,51 @@ static void show_version()
}
+static const char * format_num( long long num, long long limit,
+ const int set_prefix )
+ {
+ const char * const si_prefix[8] =
+ { "k", "M", "G", "T", "P", "E", "Z", "Y" };
+ const char * const binary_prefix[8] =
+ { "Ki", "Mi", "Gi", "Ti", "Pi", "Ei", "Zi", "Yi" };
+ static bool si = false;
+ static char buf[16];
+
+ if( set_prefix ) si = ( set_prefix > 0 );
+ {
+ const int factor = ( si ) ? 1000 : 1024;
+ const char * const *prefix = ( si ) ? si_prefix : binary_prefix;
+ const char *p = "";
+ int i;
+ limit = max( 999LL, min( 999999LL, limit ) );
+
+ for( i = 0; i < 8 && ( llabs( num ) > limit ||
+ ( llabs( num ) >= factor && num % factor == 0 ) ); ++i )
+ { num /= factor; p = prefix[i]; }
+ snprintf( buf, sizeof buf, "%lld %s", num, p );
+ }
+ return buf;
+ }
+
+
static long long getnum( const char * const ptr, const int bs,
const long long llimit, const long long ulimit )
{
- errno = 0;
+ long long result;
char *tail;
- long long result = strtoll( ptr, &tail, 0 );
+
+ errno = 0;
+ result = strtoll( ptr, &tail, 0 );
if( tail == ptr )
{
- show_error( "bad or missing numerical argument", 0, true );
+ show_error( "Bad or missing numerical argument.", 0, true );
exit( 1 );
}
if( !errno && tail[0] )
{
int factor = ( tail[1] == 'i' ) ? 1024 : 1000;
- int exponent = 0;
+ int exponent = 0, i;
bool bad_multiplier = false;
switch( tail[0] )
{
@@ -143,10 +185,10 @@ static long long getnum( const char * const ptr, const int bs,
}
if( bad_multiplier )
{
- show_error( "bad multiplier in numerical argument", 0, true );
+ show_error( "Bad multiplier in numerical argument.", 0, true );
exit( 1 );
}
- for( int i = 0; i < exponent; ++i )
+ for( i = 0; i < exponent; ++i )
{
if( LLONG_MAX / factor >= llabs( result ) ) result *= factor;
else { errno = ERANGE; break; }
@@ -155,7 +197,7 @@ static long long getnum( const char * const ptr, const int bs,
if( !errno && ( result < llimit || result > ulimit ) ) errno = ERANGE;
if( errno )
{
- show_error( "numerical argument out of limits", 0, false );
+ show_error( "Numerical argument out of limits.", 0, false );
exit( 1 );
}
return result;
@@ -186,6 +228,7 @@ static void show_name( const char * const name )
static bool read_inbuf( ISeqInStream * const inStream, Byte inBuf[],
size_t * const inPos, size_t * const inSize )
{
+ size_t rest;
if( *inPos >= *inSize ) *inSize = 0;
else if( *inPos > 0 )
{
@@ -193,16 +236,80 @@ static bool read_inbuf( ISeqInStream * const inStream, Byte inBuf[],
*inSize -= *inPos;
}
*inPos = 0;
- size_t rest = IN_BUF_SIZE - *inSize;
+ rest = IN_BUF_SIZE - *inSize;
if( rest > 0 )
{
if( inStream->Read( inStream, inBuf + *inSize, &rest ) != 0 )
- { show_error( "read error", errno, false ); return false; }
+ { show_error( "Read error", errno, false ); return false; }
*inSize += rest;
}
return true;
}
+
+static int lzma_Decode2( UInt64 unpackSize, CLzmaDec *state,
+ ISeqOutStream *outStream, ISeqInStream *inStream,
+ Byte inBuf[], size_t * const inPos,
+ size_t * const inSize, const bool testing )
+ {
+ long long total_in = 13, total_out = 0;
+ Byte outBuf[OUT_BUF_SIZE];
+ size_t outPos = 0;
+ const bool thereIsSize = (unpackSize != (UInt64)(Int64)-1);
+ LzmaDec_Init(state);
+
+ for (;;)
+ {
+ SizeT inProcessed;
+ SizeT outProcessed = OUT_BUF_SIZE - outPos;
+ ELzmaFinishMode finishMode = LZMA_FINISH_ANY;
+ ELzmaStatus status;
+
+ if( *inPos == *inSize && !read_inbuf( inStream, inBuf, inPos, inSize ) )
+ return 1;
+ inProcessed = *inSize - *inPos;
+ if (thereIsSize && outProcessed > unpackSize)
+ {
+ outProcessed = (SizeT)unpackSize;
+ finishMode = LZMA_FINISH_END;
+ }
+
+ if( LzmaDec_DecodeToBuf( state, outBuf + outPos, &outProcessed,
+ inBuf + *inPos, &inProcessed, finishMode, &status ) != 0 )
+ { show_error( "Data error.", 0, false ); return 1; }
+ *inPos += inProcessed;
+ total_in += inProcessed;
+ outPos += outProcessed;
+ unpackSize -= outProcessed;
+
+ if (outStream)
+ if (outStream->Write(outStream, outBuf, outPos) != outPos)
+ { show_error( "Can not write output file", errno, false ); return 1; }
+
+ total_out += outPos;
+ outPos = 0;
+
+ if( ( inProcessed == 0 && outProcessed == 0 ) ||
+ ( thereIsSize && unpackSize == 0 ) )
+ {
+ if( ( thereIsSize && unpackSize != 0 ) ||
+ ( !thereIsSize && status != LZMA_STATUS_FINISHED_WITH_MARK ) )
+ { show_error( "Data error.", 0, false ); return 1; }
+ if( verbosity >= 2 )
+ fprintf( stderr, "lzma-alone, dictionary size %7sB. ",
+ format_num( state->prop.dicSize, 9999, 0 ) );
+ if( verbosity >= 3 )
+ fprintf( stderr, "uncompressed size %9lld, compressed size %8lld. ",
+ total_out, total_in );
+ if( verbosity >= 1 )
+ { if( testing ) fprintf( stderr, "(apparently) ok\n" );
+ else fprintf( stderr, "(apparently) done\n" ); }
+ return 0;
+ }
+ }
+ }
+
+
static int Decode2( CLzmaDec *state, ISeqOutStream *outStream,
ISeqInStream *inStream, Byte inBuf[], size_t * const inPos,
size_t * const inSize, const int member_version,
@@ -213,166 +320,193 @@ static int Decode2( CLzmaDec *state, ISeqOutStream *outStream,
size_t outPos = 0;
uint32_t crc = 0xFFFFFFFFU;
LzmaDec_Init(state);
+
for (;;)
{
+ SizeT inProcessed;
+ SizeT outProcessed = OUT_BUF_SIZE - outPos;
+ ELzmaFinishMode finishMode = LZMA_FINISH_ANY;
+ ELzmaStatus status;
+
if( *inPos == *inSize && !read_inbuf( inStream, inBuf, inPos, inSize ) )
return 1;
if( *inPos == *inSize )
- { show_error( "unexpected EOF", errno, false ); return 1; }
- else
+ { show_error( "Unexpected EOF.", 0, false ); return 1; }
+ inProcessed = *inSize - *inPos;
+
+ if( LzmaDec_DecodeToBuf( state, outBuf + outPos, &outProcessed,
+ inBuf + *inPos, &inProcessed, finishMode, &status ) != 0 )
+ { show_error( "Data error.", 0, false ); return 1; }
+ *inPos += inProcessed;
+ total_in += inProcessed;
+ outPos += outProcessed;
+
+ if (outStream)
+ if (outStream->Write(outStream, outBuf, outPos) != outPos)
+ { show_error( "Can not write output file", errno, false ); return 1; }
+
+ CRC32_update_buf( &crc, outBuf, outPos );
+ total_out += outPos;
+ outPos = 0;
+
+ if (inProcessed == 0 && outProcessed == 0)
{
- SizeT inProcessed = *inSize - *inPos;
- SizeT outProcessed = OUT_BUF_SIZE - outPos;
- ELzmaFinishMode finishMode = LZMA_FINISH_ANY;
- ELzmaStatus status;
-
- if( LzmaDec_DecodeToBuf( state, outBuf + outPos, &outProcessed,
- inBuf + *inPos, &inProcessed, finishMode, &status ) != 0 )
- { show_error( "data error", 0, false ); return 1; }
- *inPos += inProcessed;
- total_in += inProcessed;
- outPos += outProcessed;
-
- if (outStream)
- if (outStream->Write(outStream, outBuf, outPos) != outPos)
- { show_error( "can not write output file", errno, false ); return 1; }
-
- CRC32_update_buf( &crc, outBuf, outPos );
- total_out += outPos;
- outPos = 0;
-
- if (inProcessed == 0 && outProcessed == 0)
+ File_trailer trailer;
+ size_t i;
+ const size_t trailer_size = Ft_versioned_size( member_version );
+ bool error = false;
+
+ if( status != LZMA_STATUS_FINISHED_WITH_MARK )
+ { show_error( "Data error.", 0, false ); return 1; }
+ if( *inSize - *inPos < trailer_size &&
+ !read_inbuf( inStream, inBuf, inPos, inSize ) ) return 1;
+ if( *inSize - *inPos < trailer_size )
{
- if( status != LZMA_STATUS_FINISHED_WITH_MARK )
- { show_error( "data error", 0, false ); return 1; }
- bool error = false;
- File_trailer trailer;
- const size_t trailer_size = Ft_versioned_size( member_version );
- if( *inSize - *inPos < trailer_size &&
- !read_inbuf( inStream, inBuf, inPos, inSize ) ) return 1;
- if( *inSize - *inPos < trailer_size )
- {
- error = true;
- if( verbosity >= 0 )
- fprintf( stderr, "trailer truncated at trailer position %u;"
- " some checks may fail.\n", (unsigned int)(*inSize - *inPos) );
- for( size_t i = *inSize - *inPos; i < trailer_size; ++i )
- inBuf[*inPos+i] = 0;
- }
- for( size_t i = 0; i < trailer_size; ++i )
- trailer[i] = inBuf[(*inPos)++];
- total_in += trailer_size;
- if( member_version == 0 ) Ft_set_member_size( trailer, total_in );
- if( Ft_get_data_crc( trailer ) != ( crc ^ 0xFFFFFFFFU ) )
- {
- error = true;
- if( verbosity >= 0 )
- fprintf( stderr, "crc mismatch; trailer says %08X, data crc is %08X.\n",
- (unsigned int)Ft_get_data_crc( trailer ),
- (unsigned int)( crc ^ 0xFFFFFFFFU ) );
- }
- if( Ft_get_data_size( trailer ) != total_out )
- {
- error = true;
- if( verbosity >= 0 )
- fprintf( stderr, "data size mismatch; trailer says %lld, data size is %lld.\n",
- Ft_get_data_size( trailer ), total_out );
- }
- if( Ft_get_member_size( trailer ) != total_in )
- {
- error = true;
- if( verbosity >= 0 )
- fprintf( stderr, "member size mismatch; trailer says %lld, member size is %lld.\n",
- Ft_get_member_size( trailer ), total_in );
- }
- if( !error && verbosity >= 3 )
- fprintf( stderr, "data crc %08X, data size %9lld, member size %8lld. ",
+ error = true;
+ if( verbosity >= 0 )
+ fprintf( stderr, "trailer truncated at trailer position %u;"
+ " some checks may fail.\n", (unsigned int)(*inSize - *inPos) );
+ for( i = *inSize - *inPos; i < trailer_size; ++i )
+ inBuf[*inPos+i] = 0;
+ }
+ for( i = 0; i < trailer_size; ++i )
+ trailer[i] = inBuf[(*inPos)++];
+ total_in += trailer_size;
+ if( member_version == 0 ) Ft_set_member_size( trailer, total_in );
+ if( Ft_get_data_crc( trailer ) != ( crc ^ 0xFFFFFFFFU ) )
+ {
+ error = true;
+ if( verbosity >= 0 )
+ fprintf( stderr, "crc mismatch; trailer says %08X, data crc is %08X.\n",
(unsigned int)Ft_get_data_crc( trailer ),
- Ft_get_data_size( trailer ), Ft_get_member_size( trailer ) );
- if( !error && verbosity >= 1 )
- { if( testing ) fprintf( stderr, "ok\n" );
- else fprintf( stderr, "done\n" ); }
- if( error ) return 2;
- return 0;
+ (unsigned int)( crc ^ 0xFFFFFFFFU ) );
+ }
+ if( Ft_get_data_size( trailer ) != total_out )
+ {
+ error = true;
+ if( verbosity >= 0 )
+ fprintf( stderr, "data size mismatch; trailer says %lld, data size is %lld.\n",
+ Ft_get_data_size( trailer ), total_out );
}
+ if( Ft_get_member_size( trailer ) != total_in )
+ {
+ error = true;
+ if( verbosity >= 0 )
+ fprintf( stderr, "member size mismatch; trailer says %lld, member size is %lld.\n",
+ Ft_get_member_size( trailer ), total_in );
+ }
+ if( !error && verbosity >= 3 )
+ fprintf( stderr, "data crc %08X, data size %9lld, member size %8lld. ",
+ (unsigned int)Ft_get_data_crc( trailer ),
+ Ft_get_data_size( trailer ), Ft_get_member_size( trailer ) );
+ if( !error && verbosity >= 1 )
+ { if( testing ) fprintf( stderr, "ok\n" );
+ else fprintf( stderr, "done\n" ); }
+ if( error ) return 2;
+ return 0;
}
}
}
+
static int Decode( ISeqOutStream *outStream, ISeqInStream *inStream,
const char * const name, const bool testing )
{
+ UInt64 unpackSize = 0;
CLzmaDec state;
File_header header;
Byte inBuf[IN_BUF_SIZE];
size_t inPos = 0, inSize = 0;
int retval = 0;
+ bool lzma_mode = false;
+ bool first_member;
+ /* 5 bytes of LZMA properties and 8 bytes of uncompressed size */
+ unsigned char props[LZMA_PROPS_SIZE+8];
- for( bool first_member = true; ; first_member = false )
+ for( first_member = true; ; first_member = false )
{
+ int i;
if( inSize < Fh_size &&
!read_inbuf( inStream, inBuf, &inPos, &inSize ) ) return 1;
- if( inSize < Fh_size ) // End Of File
+ if( inSize < Fh_size ) /* End Of File */
{
if( !first_member ) break;
- show_error( "error reading member header", 0, false ); return 1;
+ show_error( "Error reading member header.", 0, false ); return 1;
}
- for( int i = 0; i < Fh_size; ++i )
- header[i] = inBuf[inPos++];
+ for( i = 0; i < Fh_size; ++i )
+ props[i] = header[i] = inBuf[inPos++];
if( !Fh_verify_magic( header ) )
{
- if( !first_member ) break; // trailing garbage
- show_error( "bad magic number (file not in lzip format)", 0, false );
- return 2;
+ if( !first_member ) break; /* trailing garbage */
+ if( inSize >= 13 - Fh_size ) /* try lzma-alone */
+ {
+ for( i = Fh_size; i < 13; ++i ) props[i] = inBuf[inPos++];
+ for( i = 0; i < 8; ++i )
+ unpackSize += (UInt64)props[LZMA_PROPS_SIZE+i] << (i * 8);
+ if( ( props[12] == 0 || props[12] == 0xFF ) && props[12] == props[11] )
+ lzma_mode = true;
+ }
+ if( !lzma_mode )
+ {
+ show_error( "Bad magic number (file not in lzip format).", 0, false );
+ return 2;
+ }
}
if( !first_member ) show_name( name );
- if( !Fh_verify_version( header ) )
- {
- if( verbosity >= 0 )
- fprintf( stderr, "version %d member format not supported, newer %s needed.\n",
- Fh_version( header ), program_name );
- return 2;
- }
- if( Fh_get_dictionary_size( header ) < min_dictionary_size ||
- Fh_get_dictionary_size( header ) > max_dictionary_size )
+ if( !lzma_mode )
{
- if( verbosity >= 0 )
- fprintf( stderr, "invalid dictionary size in member header.\n" );
- return 2;
- }
+ int ds, i;
+ if( !Fh_verify_version( header ) )
+ {
+ if( verbosity >= 0 )
+ fprintf( stderr, "version %d member format not supported, newer %s needed.\n",
+ Fh_version( header ), program_name );
+ return 2;
+ }
+ if( Fh_get_dictionary_size( header ) < min_dictionary_size ||
+ Fh_get_dictionary_size( header ) > max_dictionary_size )
+ {
+ if( verbosity >= 0 )
+ fprintf( stderr, "invalid dictionary size in member header.\n" );
+ return 2;
+ }
- if( verbosity >= 1 )
- {
if( verbosity >= 2 )
- fprintf( stderr, "version %d, dictionary size %7dB. ",
- Fh_version( header ), Fh_get_dictionary_size( header ) );
- }
+ fprintf( stderr, "version %d, dictionary size %7sB. ",
+ Fh_version( header ),
+ format_num( Fh_get_dictionary_size( header ), 9999, 0 ) );
- /* 5 bytes of LZMA properties */
- unsigned char props[LZMA_PROPS_SIZE];
- props[0] = 93; // (45 * 2) + (9 * 0) + 3
- int ds = Fh_get_dictionary_size( header );
- for( int i = 1; i <= 4; ++i ) { props[i] = ds & 0xFF; ds >>= 8; }
+ props[0] = 93; /* (45 * 2) + (9 * 0) + 3 */
+ ds = Fh_get_dictionary_size( header );
+ for( i = 1; i <= 4; ++i ) { props[i] = ds & 0xFF; ds >>= 8; }
+ }
LzmaDec_Construct(&state);
if( LzmaDec_Allocate( &state, props, LZMA_PROPS_SIZE, &g_Alloc ) != 0 )
- { show_error( "can not allocate memory", 0, false ); return 1; }
- retval = Decode2( &state, outStream, inStream, inBuf, &inPos,
- &inSize, Fh_version( header ), testing );
+ { show_error( "Can not allocate memory.", 0, false ); return 1; }
+ if( !lzma_mode )
+ retval = Decode2( &state, outStream, inStream, inBuf, &inPos,
+ &inSize, Fh_version( header ), testing );
+ else
+ retval = lzma_Decode2( unpackSize, &state, outStream, inStream,
+ inBuf, &inPos, &inSize, testing );
LzmaDec_Free(&state, &g_Alloc);
- if( retval != 0 ) break;
+ if( retval != 0 || lzma_mode ) break;
}
return retval;
}
+
static int Encode( ISeqOutStream *outStream, ISeqInStream *inStream,
const struct Lzma_options * const encoder_options )
{
CLzmaEncProps props;
+ int retval = 0;
+ File_header header;
CLzmaEncHandle enc = LzmaEnc_Create(&g_Alloc);
if(enc == 0)
- { show_error( "can not allocate memory", 0, false ); return 1; }
+ { show_error( "Can not allocate memory.", 0, false ); return 1; }
LzmaEncProps_Init(&props);
props.dictSize = encoder_options->dictionary_size;
@@ -385,8 +519,6 @@ static int Encode( ISeqOutStream *outStream, ISeqInStream *inStream,
props.mc = 16 + ( encoder_options->match_len_limit / 2 );
LzmaEnc_SetProps(enc, &props);
- int retval = 0;
- File_header header;
Fh_set_magic( header );
if( !Fh_set_dictionary_size( header, encoder_options->dictionary_size ) ||
encoder_options->match_len_limit < min_match_len_limit ||
@@ -394,14 +526,15 @@ static int Encode( ISeqOutStream *outStream, ISeqInStream *inStream,
internal_error( "invalid argument to encoder" );
if( outStream->Write( outStream, header, Fh_size ) != Fh_size )
- { show_error( "can not write output file", errno, false ); retval = 1; }
+ { show_error( "Can not write output file", errno, false ); retval = 1; }
else
if( LzmaEnc_Encode(enc, outStream, inStream, NULL, &g_Alloc, &g_Alloc) != 0 )
- { show_error( "data error", 0, false ); retval = 1; }
+ { show_error( "Data error.", 0, false ); retval = 1; }
LzmaEnc_Destroy(enc, &g_Alloc, &g_Alloc);
return retval;
}
+
int verbosity = 0;
@@ -409,13 +542,13 @@ void show_error( const char * const msg, const int errcode, const bool help )
{
if( verbosity >= 0 )
{
- if( msg && msg[0] != 0 )
+ if( msg && msg[0] )
{
fprintf( stderr, "%s: %s", program_name, msg );
if( errcode > 0 ) fprintf( stderr, ": %s", strerror( errcode ) );
fprintf( stderr, "\n" );
}
- if( help && invocation_name && invocation_name[0] != 0 )
+ if( help && invocation_name && invocation_name[0] )
fprintf( stderr, "Try `%s --help' for more information.\n", invocation_name );
}
}
@@ -423,19 +556,15 @@ void show_error( const char * const msg, const int errcode, const bool help )
void internal_error( const char * const msg )
{
- const char * const e = "internal error: ";
- char * s = resize_buffer( 0, strlen( e ) + strlen( msg ) + 1 );
- strcpy( s, e );
- strcat( s, msg );
- show_error( s, 0, false );
- free( s );
+ fprintf( stderr, "%s: internal error: %s.\n", program_name, msg );
exit( 3 );
}
static int extension_index( const char * const name )
{
- for( int i = 0; known_extensions[i].from; ++i )
+ int i;
+ for( i = 0; known_extensions[i].from; ++i )
{
const char * const ext = known_extensions[i].from;
if( strlen( name ) > strlen( ext ) &&
@@ -485,29 +614,30 @@ CRC32 crc32;
int main( const int argc, const char * const argv[] )
{
- // Mapping from gzip/bzip2 style 1..9 compression modes
- // to the corresponding LZMA compression modes.
+ /* Mapping from gzip/bzip2 style 1..9 compression modes
+ to the corresponding LZMA compression modes. */
const struct Lzma_options option_mapping[] =
{
- { 1 << 16, 5 }, // -0
- { 1 << 20, 10 }, // -1
- { 3 << 19, 12 }, // -2
- { 1 << 21, 17 }, // -3
- { 3 << 20, 26 }, // -4
- { 1 << 22, 44 }, // -5
- { 1 << 23, 80 }, // -6
- { 1 << 24, 108 }, // -7
- { 3 << 23, 163 }, // -8
- { 1 << 25, 273 } }; // -9
- struct Lzma_options encoder_options = option_mapping[6]; // default = "-6"
+ { 1 << 16, 5 }, /* -0 */
+ { 1 << 20, 5 }, /* -1 */
+ { 3 << 19, 6 }, /* -2 */
+ { 1 << 21, 8 }, /* -3 */
+ { 3 << 20, 12 }, /* -4 */
+ { 1 << 22, 20 }, /* -5 */
+ { 1 << 23, 36 }, /* -6 */
+ { 1 << 24, 68 }, /* -7 */
+ { 3 << 23, 132 }, /* -8 */
+ { 1 << 25, 273 } }; /* -9 */
+ struct Lzma_options encoder_options = option_mapping[6]; /* default = "-6" */
enum Mode program_mode = m_compress;
+ const char * input_filename = "";
+ CFileSeqInStream inStream;
+ CFileOutStream outStream;
+ int argind;
+ int retval;
bool force = false;
bool keep_input_files = false;
bool to_stdout = false;
- invocation_name = argv[0];
- CRC32_init();
- if (sizeof(UInt32) != 4 || sizeof(UInt64) != 8)
- internal_error( "incorrect UInt32 or UInt64" );
const struct ap_Option options[] =
{
@@ -529,7 +659,6 @@ int main( const int argc, const char * const argv[] )
{ 'h', "help", ap_no },
{ 'k', "keep", ap_no },
{ 'm', "match-length", ap_yes },
-// { 'o', "output", ap_yes },
{ 'q', "quiet", ap_no },
{ 's', "dictionary-size", ap_yes },
{ 'S', "volume-size", ap_yes },
@@ -539,17 +668,22 @@ int main( const int argc, const char * const argv[] )
{ 0 , 0, ap_no } };
struct Arg_parser parser;
+
+ invocation_name = argv[0];
+ CRC32_init();
+ if (sizeof(UInt32) != 4 || sizeof(UInt64) != 8)
+ internal_error( "incorrect UInt32 or UInt64" );
+
if( !ap_init( &parser, argc, argv, options, 0 ) )
- { show_error( "memory exhausted", 0, false ); return 1; }
- if( ap_error( &parser ) ) // bad option
+ { show_error( "Memory exhausted.", 0, false ); return 1; }
+ if( ap_error( &parser ) ) /* bad option */
{ show_error( ap_error( &parser ), 0, true ); return 1; }
- int argind = 0;
- for( ; argind < ap_arguments( &parser ); ++argind )
+ for( argind = 0; argind < ap_arguments( &parser ); ++argind )
{
const int code = ap_code( &parser, argind );
const char * const arg = ap_argument( &parser, argind );
- if( !code ) break; // no more options
+ if( !code ) break; /* no more options */
switch( code )
{
case '0': case '1': case '2': case '3': case '4':
@@ -575,12 +709,11 @@ int main( const int argc, const char * const argv[] )
}
}
- const char * input_filename = "";
if( ap_arguments( &parser ) > argind &&
strcmp( ap_argument( &parser, argind ), "-" ) )
input_filename = ap_argument( &parser, argind );
if( ap_arguments( &parser ) > argind + 1 )
- { show_error( "too many file names", 0, true ); return 1; }
+ { show_error( "Too many file names.", 0, true ); return 1; }
if( program_mode == m_test ) output_filename = "/dev/null";
else
@@ -593,9 +726,6 @@ int main( const int argc, const char * const argv[] )
}
}
- CFileSeqInStream inStream;
- CFileOutStream outStream;
-
FileSeqInStream_CreateVTable(&inStream);
File_Construct(&inStream.file);
@@ -603,13 +733,12 @@ int main( const int argc, const char * const argv[] )
File_Construct(&outStream.file);
if (InFile_Open(&inStream.file, input_filename) != 0)
- { show_error( "can not open input file", errno, false ); return 1; }
+ { show_error( "Can not open input file", errno, false ); return 1; }
if (OutFile_Open(&outStream.file, output_filename) != 0)
- { show_error( "can not open output file", errno, false ); return 1; }
+ { show_error( "Can not open output file", errno, false ); return 1; }
show_name( input_filename );
- int retval;
if( program_mode == m_compress )
retval = Encode( &outStream.s, &inStream.s, &encoder_options );
else
diff --git a/pdlzip.h b/pdlzip.h
index 5af155b..077ea9b 100644
--- a/pdlzip.h
+++ b/pdlzip.h
@@ -9,10 +9,22 @@
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
*/
+#ifndef __cplusplus
+enum Bool { false = 0, true = 1 };
+typedef enum Bool bool;
+#endif
+
+#ifndef max
+ #define max(x,y) ((x) >= (y) ? (x) : (y))
+#endif
+#ifndef min
+ #define min(x,y) ((x) <= (y) ? (x) : (y))
+#endif
+
enum {
min_dictionary_bits = 12,
min_dictionary_size = 1 << min_dictionary_bits,
- max_dictionary_bits = 27,
+ max_dictionary_bits = 26,
max_dictionary_size = 1 << max_dictionary_bits,
literal_context_bits = 3,
pos_state_bits = 2,
@@ -25,20 +37,22 @@ enum {
len_high_symbols = 1 << len_high_bits,
max_len_symbols = len_low_symbols + len_mid_symbols + len_high_symbols,
- min_match_len = 2, // must be 2
- max_match_len = min_match_len + max_len_symbols - 1, // 273
+ min_match_len = 2, /* must be 2 */
+ max_match_len = min_match_len + max_len_symbols - 1, /* 273 */
min_match_len_limit = 5 };
-typedef uint32_t CRC32[256]; // Table of CRCs of all 8-bit messages.
+typedef uint32_t CRC32[256]; /* Table of CRCs of all 8-bit messages. */
extern CRC32 crc32;
static inline void CRC32_init()
{
- for( unsigned int n = 0; n < 256; ++n )
+ unsigned int n;
+ for( n = 0; n < 256; ++n )
{
unsigned int c = n;
- for( int k = 0; k < 8; ++k )
+ int k;
+ for( k = 0; k < 8; ++k )
{ if( c & 1 ) c = 0xEDB88320U ^ ( c >> 1 ); else c >>= 1; }
crc32[n] = c;
}
@@ -49,26 +63,27 @@ static inline void CRC32_update_byte( uint32_t * crc, const uint8_t byte )
static inline void CRC32_update_buf( uint32_t * crc, const uint8_t * const buffer,
const int size )
{
- for( int i = 0; i < size; ++i )
+ int i;
+ for( i = 0; i < size; ++i )
*crc = crc32[(*crc^buffer[i])&0xFF] ^ ( *crc >> 8 );
}
-typedef uint8_t File_header[6]; // 0-3 magic bytes
- // 4 version
- // 5 coded_dict_size;
+static const uint8_t magic_string[4] = { 'L', 'Z', 'I', 'P' };
+
+typedef uint8_t File_header[6]; /* 0-3 magic bytes */
+ /* 4 version */
+ /* 5 coded_dict_size */
enum { Fh_size = 6 };
static inline void Fh_set_magic( File_header header )
{
- const uint8_t magic_string[4] = { 'L', 'Z', 'I', 'P' };
memcpy( header, magic_string, 4 );
header[4] = 1;
}
static inline bool Fh_verify_magic( const File_header header )
{
- const uint8_t magic_string[4] = { 'L', 'Z', 'I', 'P' };
return ( memcmp( header, magic_string, 4 ) == 0 );
}
@@ -80,8 +95,8 @@ static inline bool Fh_verify_version( const File_header header )
static inline int Fh_real_bits( const int value )
{
- int bits = 0;
- for( int i = 1, mask = 1; mask > 0; ++i, mask <<= 1 )
+ int bits = 0, i, mask;
+ for( i = 1, mask = 1; mask > 0; ++i, mask <<= 1 )
if( value & mask ) bits = i;
return bits;
}
@@ -103,7 +118,8 @@ static inline bool Fh_set_dictionary_size( File_header header, const int sz )
{
const int base_size = 1 << header[5];
const int wedge = base_size / 16;
- for( int i = 7; i >= 1; --i )
+ int i;
+ for( i = 7; i >= 1; --i )
if( base_size - ( i * wedge ) >= sz )
{ header[5] |= ( i << 5 ); break; }
}
@@ -114,9 +130,9 @@ static inline bool Fh_set_dictionary_size( File_header header, const int sz )
typedef uint8_t File_trailer[20];
- // 0-3 CRC32 of the uncompressed data
- // 4-11 size of the uncompressed data
- // 12-19 member size including header and trailer
+ /* 0-3 CRC32 of the uncompressed data */
+ /* 4-11 size of the uncompressed data */
+ /* 12-19 member size including header and trailer */
enum { Ft_size = 20 };
@@ -126,35 +142,43 @@ static inline int Ft_versioned_size( const int version )
static inline uint32_t Ft_get_data_crc( const File_trailer trailer )
{
uint32_t tmp = 0;
- for( int i = 3; i >= 0; --i ) { tmp <<= 8; tmp += trailer[i]; }
+ int i;
+ for( i = 3; i >= 0; --i ) { tmp <<= 8; tmp += trailer[i]; }
return tmp;
}
static inline void Ft_set_data_crc( File_trailer trailer, uint32_t crc )
- { for( int i = 0; i <= 3; ++i ) { trailer[i] = (uint8_t)crc; crc >>= 8; } }
+ {
+ int i;
+ for( i = 0; i <= 3; ++i ) { trailer[i] = (uint8_t)crc; crc >>= 8; }
+ }
static inline long long Ft_get_data_size( const File_trailer trailer )
{
long long tmp = 0;
- for( int i = 11; i >= 4; --i ) { tmp <<= 8; tmp += trailer[i]; }
+ int i;
+ for( i = 11; i >= 4; --i ) { tmp <<= 8; tmp += trailer[i]; }
return tmp;
}
static inline void Ft_set_data_size( File_trailer trailer, long long sz )
{
- for( int i = 4; i <= 11; ++i ) { trailer[i] = (uint8_t)sz; sz >>= 8; }
+ int i;
+ for( i = 4; i <= 11; ++i ) { trailer[i] = (uint8_t)sz; sz >>= 8; }
}
static inline long long Ft_get_member_size( const File_trailer trailer )
{
long long tmp = 0;
- for( int i = 19; i >= 12; --i ) { tmp <<= 8; tmp += trailer[i]; }
+ int i;
+ for( i = 19; i >= 12; --i ) { tmp <<= 8; tmp += trailer[i]; }
return tmp;
}
static inline void Ft_set_member_size( File_trailer trailer, long long sz )
{
- for( int i = 12; i <= 19; ++i ) { trailer[i] = (uint8_t)sz; sz >>= 8; }
+ int i;
+ for( i = 12; i <= 19; ++i ) { trailer[i] = (uint8_t)sz; sz >>= 8; }
}
diff --git a/testsuite/check.sh b/testsuite/check.sh
index 24e44df..c619507 100755
--- a/testsuite/check.sh
+++ b/testsuite/check.sh
@@ -22,10 +22,14 @@ mkdir tmp
printf "testing pdlzip..."
cd "${objdir}"/tmp
-cat "${testdir}"/test1 > in || framework_failure
+cat "${testdir}"/test.txt > in || framework_failure
fail=0
-"${LZIP}" -cd "${testdir}"/test1.lz > copy || fail=1
+"${LZIP}" -t "${testdir}"/test.lz || fail=1
+"${LZIP}" -cd "${testdir}"/test.lz > copy || fail=1
+cmp in copy || fail=1
+"${LZIP}" -t "${testdir}"/test.lzma || fail=1
+"${LZIP}" -cd "${testdir}"/test.lzma > copy || fail=1
cmp in copy || fail=1
for i in s4Ki 0 1 2 3 4 5 6 7 8s16 9s16 ; do
diff --git a/testsuite/test1.lz b/testsuite/test.lz
index a09b1e8..a09b1e8 100644
--- a/testsuite/test1.lz
+++ b/testsuite/test.lz
Binary files differ
diff --git a/testsuite/test.lzma b/testsuite/test.lzma
new file mode 100644
index 0000000..5f62956
--- /dev/null
+++ b/testsuite/test.lzma
Binary files differ
diff --git a/testsuite/test1 b/testsuite/test.txt
index 5b244d5..5b244d5 100644
--- a/testsuite/test1
+++ b/testsuite/test.txt