aspell-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [aspell-devel] Partial improvement for compress


From: Jose Da Silva
Subject: Re: [aspell-devel] Partial improvement for compress
Date: Thu, 2 Sep 2004 14:43:39 -0700
User-agent: KMail/1.6.1

Please accept this update for compress.c
It has 2 bug fixes for compress.c

1st bug-fix is to remove the "static" of function get_word() which will cause 
problems for multiple instances of compress running at the same time.

2nd bug-fix is for words longer than 255 characters in length, so compress 
exits if longer than 255 characters rather than silently continue with 
corrupt results (to test it out, there is long.txt included).

The diff file is against the 0.60 release and includes a few more refinements 
relative to the last update (for example there is no point in passing 
BUFSIZE to get_word() if everything is hardcoded to BUFSIZE in size).

To test the long-word problem, test with:
compress -c <long.txt >long.cwl

To remove the long word fault, remove the "5" at the end of the very long 
string so that the word length is now only 255 chars instead of 256 chars.
At this point then, long.ttt should be equal to long.txt if following steps 
are followed:
compress -c <long.txt >long.cwl
compress -d <long.cwl >long.ttt
...which to me, I think appears working okay then.

Note: if you want to include versions, please replace version X with an 
appropriate number, otherwise remove the if statement for version.

Sincerely,
Jose Da Silva

Attachment: compress.c
Description: Text Data

Attachment: long.txt
Description: Text document

Attachment: compress_diff_3.txt
Description: Text document


reply via email to

[Prev in Thread] Current Thread [Next in Thread]