blob: 1af586c40dad177eae4b3e8e1a08dd66c6273126 [file] [log] [blame]
2003-06-12 Bruno Haible <bruno@clisp.org>
* gperf-3.0.1 released.
* src/version.cc: Bump version number to 3.0.1.
* doc/gperf.texi: Likewise.
* tests/c-parse.exp, tests/charsets.exp, tests/chill.exp,
tests/cplusplus.exp, tests/gpc.exp, tests/incomplete.exp,
tests/java.exp, tests/languages.exp, tests/modula2.exp,
tests/objc.exp, tests/permut2.exp, tests/permut3.exp,
tests/permutc2.exp, tests/test-4.exp: Update.
2003-05-31 Bruno Haible <bruno@clisp.org>
* doc/gperf.texi (User-supplied Struct): Mention the possibility of an
abbreviated struct declaration.
* src/input.cc (Input::read_input): Support struct declarations of the
form "struct foo;".
* tests/incomplete.gperf: New file.
* tests/incomplete.exp: New file.
* tests/Makefile.in (check-test): Check incomplete.gperf too.
Reported by Rob Leslie <rob@mars.org>.
2003-05-20 Bruno Haible <bruno@clisp.org>
* doc/Makefile.in (gperf.ps): Don't use $< in a target rule.
2003-05-27 Bruno Haible <bruno@clisp.org>
* Makefile.vms (CC): Correct value.
(getopt.obj, getopt1.obj, getline.obj, hash.obj): Don't set
HAVE_CONFIG_H.
2003-05-17 Bruno Haible <bruno@clisp.org>
* Makefile.msvc (DEBUGFLAGS): New variable.
(gperf.exe): Use it, and MFLAGS too.
2003-05-08 Bruno Haible <bruno@clisp.org>
* gperf-3.0 released.
2003-05-07 Bruno Haible <bruno@clisp.org>
* src/version.cc: Bump version number to 3.0.
* doc/gperf.texi: Likewise.
* tests/c-parse.exp, tests/charsets.exp, tests/chill.exp,
tests/cplusplus.exp, tests/gpc.exp, tests/java.exp,
tests/languages.exp, tests/modula2.exp, tests/objc.exp,
tests/permut2.exp, tests/permut3.exp, tests/permutc2.exp,
tests/test-4.exp: Update.
* src/configure.in: Fix AC_INIT argument.
* Makefile.devel (configure, lib/configure, src/configure,
tests/configure, doc/configure): Use the newest autoconf.
(src/config.h.in): Use the newest autoheader.
2003-05-03 Bruno Haible <bruno@clisp.org>
* doc/gperf.texi: Use two spaces as sentence separator, as recommended
by the texinfo manual.
2003-04-12 Bruno Haible <bruno@clisp.org>
* doc/configure.in (mandir): Change default value.
* doc/Makefile.in (docdir): Use datadir instead of prefix.
* Makefile.msvc (datadir): New variable.
(mandir, docdir): Use it instead of prefix.
(install, installdirs): Update.
* Makefile.vms (datadir): New variable.
(mandir, docdir): Use it instead of prefix.
(install, installdirs): Update.
2003-04-12 Bruno Haible <bruno@clisp.org>
* README.vms: New file.
* Makefile.vms: New file.
* Makefile.devel (src/config.h_vms): New rule.
(all): Depend on it.
2003-03-19 Bruno Haible <bruno@clisp.org>
* src/input.cc (Input::read_input): Ignore comments at the beginning
of the declarations section.
* doc/gperf.texi (Controls for GNU indent): New section.
Reported by Bruce Lilly <blilly@erols.com>.
2003-03-19 Bruno Haible <bruno@clisp.org>
* src/output.cc (Output::output_hash_function): Avoid lint warning if
not all arguments of the hash function are used. Avoid lint warning
for fallthrough in switch.
* tests/c-parse.exp, tests/charsets.exp, tests/chill.exp,
tests/cplusplus.exp, tests/java.exp, tests/languages.exp,
tests/modula2.exp, tests/objc.exp: All /*FALLTHROUGH*/ to expected
output.
Reported by Bruce Lilly <blilly@erols.com>.
2003-03-01 Bruno Haible <bruno@clisp.org>
* src/options.h (Options::set_initializer_suffix): New declaration.
* src/options.cc (Options::set_initializer_suffix): New method.
* src/input.cc (Input::read_input): Recognize %define
initializer-suffix.
* doc/gperf.texi (Gperf Declarations): Document %define
initializer-suffix.
* NEWS: Update.
2003-02-26 Bruno Haible <bruno@clisp.org>
* Makefile.msvc: New file.
* README.woe32: New file.
* Makefile.devel (all): Depend on src/config.h.msvc.
(src/config.h.msvc): New rule.
2003-01-07 Bruno Haible <bruno@clisp.org>
* src/input.h (Input::_charset_dependent): New field.
* src/input.cc (Input::read_input): Also set _charset_dependent.
* src/main.cc (main): Pass _charset_dependent from Input to Output.
* src/output.h (Output::Output): Add charset_dependent argument.
(Output::_charset_dependent): New field.
* src/output.cc (Output::Output): Add charset_dependent argument.
(Output::output): Provoke a compilation error if the execution
character set doesn't match the expectations.
* tests/c-parse.exp, tests/charsets.exp, tests/chill.exp,
tests/cplusplus.exp, tests/gpc.exp, tests/java.exp,
tests/languages.exp, tests/modula2.exp, tests/objc.exp,
tests/permut2.exp, tests/permut3.exp, tests/permutc2.exp,
tests/test-4.exp: Update.
* src/options.cc (Options::long_usage): Change bug report address to
<bug-gnu-gperf@gnu.org>.
* tests/test-6.exp: Update.
* src/output.cc (USE_DOWNCASE_TABLE): New macro.
(output_upperlower_table): New function.
(output_upperlower_strcmp, output_upperlower_strncmp,
output_upperlower_memcmp): Emit gperf_downcase array accesses.
(Output::output): Call output_upperlower_table.
* tests/permutc2.exp: Update.
* src/keyword-list.icc (KeywordExt_List::rest): Use a portable cast.
(Only in GCC a cast of an lvalue is an lvalue.)
2003-01-01 Bruno Haible <bruno@clisp.org>
* src/options.cc (Options::parse_options): Update copyright year.
* doc/gperf.texi (@author): Add me.
* src/options.h (NULLSTRINGS): New enum value.
(Options::get_stringpool_name, Options::set_stringpool_name): New
method declarations.
(Options::_stringpool_name): New field.
* src/options.icc (Options::get_stringpool_name): New method.
* src/options.cc (DEFAULT_STRINGPOOL_NAME): New variable.
(Options::long_usage): Document -Q and --null-strings.
(Options::Options): Initialize _stringpool_name.
(Options::~Options): Output _stringpool_name, NULLSTRINGS values too.
(Options::set_stringpool_name): New method.
(long_options): Add options --string-pool-name, --null-strings.
(Options::parse_options): Implement options -P, -Q and --null-strings.
* src/input.cc (Input::read_input): Recognize declarations %pic,
%define string-pool-name, %null-strings.
* src/output.h (Output::output_string_pool,
Output::output_lookup_pools): New method declarations.
(Output::_wordlist_eltype): New field.
* src/output.cc (Output::output_keylength_table): Trivial
simplification.
(Output::output_string_pool): New method.
(output_keyword_entry): Add stringpool_index argument. For SHAREDLIB,
use struct offsets.
(output_keyword_blank_entries): For SHAREDLIB, use -1 instead of "".
(Output::output_keyword_table): Use _wordlist_eltype instead of
_struct_tag. Compute stringpool_index for output_keyword_entry.
(Output::output_lookup_pools): New method.
(Output::output_lookup_function_body): Use _wordlist_eltype instead of
_struct_tag. For SHAREDLIB, use "+ stringpool" to convert offsets to
strings. Use "o >= 0" to test for nonempty table entry.
(Output::output_lookup_function): Call output_lookup_pools.
(Output::output): Initialize _wordlist_eltype. Call
output_lookup_pools.
* tests/jstest4.gperf: New file.
* tests/test-6.exp: Update.
* tests/Makefile.in (check-lang-syntax): Drop test of -p. Add tests of
-P and -Q.
* doc/gperf.texi (User-supplied Struct): Mention that first field has
to be of type 'int' if -P is given.
(Gperf Declarations): Document %pic, %define string-pool-name,
%null-strings.
(Output Details): Update description of option -P. Document options -Q
and --null-strings.
* tests/Makefile.in (check-link-c, check-ada, check-pascal,
check-test): Omit option -p.
* tests/c-parse.exp: Regenerated.
* tests/chill.exp: Regenerated.
* tests/cplusplus.exp: Regenerated.
* tests/gpc.exp: Regenerated.
* tests/java.exp: Regenerated.
* tests/objc.exp: Regenerated.
* tests/test-4.exp: Regenerated.
* src/output.cc (Output::output_lookup_function_body): Omit the
multicompare code section and its variables when it is not used.
* tests/chill.exp: Regenerated.
* src/output.c (Output_Compare::output_firstchar_comparison): New
method.
(Output_Compare_Strcmp::output_comparison,
Output_Compare_Strncmp::output_comparison,
Output_Compare_Memcmp::output_comparison): Use it.
* tests/permutc2.exp: Update.
* tests/smtp.gperf: New file, based on a contribution by Bruce Lilly.
* tests/Makefile.in (check-smtp): New rule.
(check): Depend on it.
(clean): Update.
2002-12-12 Bruno Haible <bruno@clisp.org>
* src/search.h (Search::init_selchars_tuple,
Search::count_duplicates_tuple): Add alpha_unify argument.
(Search::count_duplicates_tuple): New method declaration.
* src/search.cc (Search::init_selchars_tuple,
Search::count_duplicates_tuple): Add alpha_unify argument.
(Search::find_positions): Update.
(Search::count_duplicates_tuple): New method.
(Search::count_duplicates_multiset): Free temp alpha_unify vector.
(Search::find_alpha_inc): Call count_duplicates_tuple.
* src/configure.in: Add test for stack-allocated variable-size arrays.
* src/config.h.in: Regenerated.
* src/search.cc: Include config.h.
(DYNAMIC_ARRAY, FREE_DYNAMIC_ARRAY): New macros.
(Search::find_alpha_inc, Search::count_possible_collisions,
Search::find_asso_values): Use them.
* src/Makefile.in (search.o): Depend on config.h.
* src/search.h (Search::keyword_list_length, Search::max_key_length,
Search::get_max_keysig_size, Search::prepare): Remove declarations.
(Search::prepare): Renamed from Search::preprepare.
(Search::_max_selchars_length): New field.
* src/search.cc (Search::prepare): Renamed from Search::preprepare.
(Search::prepare_asso_values): Merged with old Search::prepare.
Initialize _max_selchars_length.
(Search::keyword_list_length): Remove function. Use _list_len instead.
(Search::max_key_length): Remove function. Use _max_key_len instead.
(Search::get_max_keysig_size): Remove function. Use
_max_selchars_length instead.
(Search::count_possible_collisions, Search::find_asso_values): Update.
(Search::find_good_asso_values): Call just prepare_asso_values.
(Search::~Search): Update.
* src/output.h (Output::output_asso_values_ref): New declaration.
* src/output.cc (char_to_index): Remove variable.
(Output::output_asso_values_ref): New function.
(Output::output_hash_function): Use it.
(Output::output): Update.
* src/positions.h (Positions::is_useall, Positions::set_useall,
Positions::iterator, Positions::reviterator): New method declarations.
(Positions::_useall): New field.
(PositionIterator): Make constructor private. Add a constructor and a
copy constructor.
(PositionIterator::remaining): New declaration.
(PositionReverseIterator): Make constructor private. Add a constructor
and a copy constructor.
(PositionReverseIterator::remaining): New declaration.
(PositionReverseIterator::_minindex): New field.
* src/positions.icc (Positions::Positions): Initialize _useall.
(Positions::operator=): Likewise.
(Positions::is_useall, Positions::set_useall): New methods.
(Positions::sort): Do nothing if _useall is set.
(Positions::iterator, Positions::reviterator): New methods.
(PositionIterator::PositionIterator): New constructor.
(PositionIterator::remaining): New method.
(PositionReverseIterator::PositionReverseIterator): New constructor.
(PositionReverseIterator::next): Use _minindex as bound.
(PositionReverseIterator::remaining): New method.
* src/positions.cc (Positions::add, Positions::remove): Reset the
useall flag.
(Positions::print): Handle the useall case.
* src/options.h (ALLCHARS): Remove.
* src/options.cc (Options::~Options): Update.
(Options::parse_options): Use Positions::set_useall().
* src/keyword.h (KeywordExt::init_selchars_tuple,
KeywordExt::init_selchars_multiset, KeywordExt::init_selchars_low):
Remove use_all_chars argument.
* src/keyword.cc (KeywordExt::init_selchars_low): Remove use_all_chars
argument. Tell the position iterator to stop at _allchars_length.
Remove special case code for -k'*'.
(KeywordExt::init_selchars_tuple, KeywordExt::init_selchars_multiset):
Remove use_all_chars argument.
* src/search.h (Search::init_selchars_tuple): Remove use_all_chars
argument.
(Search::init_selchars_multiset): Likewise.
* src/search.cc (Search::init_selchars_tuple): Remove use_all_chars
argument.
(Search::count_duplicates_tuple, Search::find_positions): Update.
(Search::compute_alpha_unify): Remove special case code for -k'*'.
(Search::init_selchars_multiset): Remove use_all_chars argument.
(Search::count_duplicates_multiset): Update.
(Search::find_alpha_inc): Remove special case code for -k'*'.
(Search::prepare): Update.
(Search::get_max_keysig_size): Update.
* src/output.cc (Output::output_hash_function): Remove special case
code for -k'*'.
* tests/chill.exp: Regenerated.
2002-12-11 Bruno Haible <bruno@clisp.org>
Change the positions to be 0-based, instead of 1-based.
* src/positions.h (Positions::LASTCHAR): Set to -1.
(Positions::MAX_SIZE): New constant.
(Positions::pointer): Change return type.
(Positions::_positions): Change element type.
(PositionIterator::EOS, PositionReverseIterator::EOS): Set to -2.
* src/positions.icc (Positions::pointer): Change return type.
(Positions::sort): Update.
* src/positions.cc (Positions::contains, Positions::add,
Positions::remove): Update.
(Positions::print): Update. Fix off-by-one bug.
* src/options.cc (Options::~Options): Update.
(Options::parse_options): Set BAD_VALUE to -3. Update.
* src/keyword.cc (KeywordExt::init_selchars_low): Update.
* src/search.cc (Search::find_positions, Search::compute_alpha_unify,
Search::find_alpha_inc): Update.
* src/output.cc (Output::output_hash_function): Update. Don't emit
a 'case' statement right after 'default:'.
* tests/c-parse.exp: Regenerated.
* tests/charsets.exp: Regenerated.
* tests/cplusplus.exp: Regenerated.
* tests/java.exp: Regenerated.
* tests/languages.exp: Regenerated.
* tests/modula2.exp: Regenerated.
* tests/objc.exp: Regenerated.
2002-12-10 Bruno Haible <bruno@clisp.org>
* src/options.h: Reorder enum values.
(Options::short_usage, Options::long_usage): Make static.
* src/options.cc (Options::short_usage); No longer print a monster
usage line.
(Options::print_options): Improve output of options like
--key-positions=1,2,$.
(Options::~Options): Update.
* src/options.h (UPPERLOWER): New enum value.
* src/options.cc (Options::long_usage): Document option --ignore-case.
(Options::~Options): Update.
(long_options): Add option --ignore-case.
(Options::parse_options): Handle option --ignore-case.
* src/input.cc (Input::read_input): Recognize option %ignore-case.
* src/keyword.h (KeywordExt::init_selchars_tuple,
KeywordExt::init_selchars_multiset, KeywordExt::init_selchars_low):
Add alpha_unify argument.
* src/keyword.cc (KeywordExt::init_selchars_low): Add alpha_unify
argument.
(KeywordExt::init_selchars_tuple): Add alpha_unify argument.
(KeywordExt::init_selchars_multiset): Add alpha_unify argument.
* src/search.h (Search::compute_alpha_size,
Search::compute_alpha_unify): New declarations.
(Search::init_selchars_multiset): Add alpha_unify argument.
(Search::_alpha_unify): New field.
* src/search.cc (Search::compute_alpha_size,
Search::compute_alpha_unify): New functions.
(Search::init_selchars_tuple): Update.
(Search::find_positions): Temporarily set _alpha_unify. Perform a
case insensitive comparison if needed.
(Search::init_selchars_multiset): Add alpha_unify argument.
(Search::count_duplicates_multiset): Call compute_alpha_unify.
(Search::find_alpha_inc): Temporarily set _alpha_unify. At the end,
set _alpha_size and _alpha_unify.
(Search::prepare): Update. Don't compute _alpga_size here.
(Search::optimize): Propagate unified asso_values.
(Search::~Search) Delete _alpha_unify.
* src/output.cc (output_upperlower_strcmp, output_upperlower_strncmp,
output_upperlower_memcmp): New functions.
(Output_Compare_Strcmp::output_comparison,
Output_Compare_Strncmp::output_comparison,
Output_Compare_Memcmp::output_comparison): Use the case-insensitive
comparison function if --ignore-case was given.
(Output::output): Emit the auxiliary case-insensitive comparison
function if needed.
* tests/permutc2.gperf, tests/permutc2.exp: New files.
* tests/Makefile.in (check-test): Also check permutc2.gperf.
* tests/test-6.exp: Update.
* doc/gperf.texi (Gperf Declarations): Document %ignore-case.
(Input Details): Document option --ignore-case.
* NEWS: Update.
* src/search.cc (Search::optimize): Fill unused asso_values[] entries
with a large value.
* src/output.h (Output::Output): Remove occurrences argument.
(Output::_occurrences): Remove field.
* src/output.cc (Output::Output): Remove occurrences argument.
(Output::output_hash_function): Ignore _occurrences.
* src/main.cc (main): Don't pass the _occurrences to Output.
* src/search.cc (Search::preprepare): Exit if keywords contain
out-of-range characters.
* src/search.cc (for): Define so as to avoid errors with old compilers.
* src/options.h (SHAREDLIB): New enum value.
* src/options.cc (Options::short_usage): Mention option -P.
(Options::long_usage): Document option -P.
(long_options): Add option --pic.
(Options::parse_options): Handle option -P/--pic.
* src/output.cc (output_keyword_blank_entries): When SHAREDLIB is
specified, emit NULL pointers instead of "".
(Output::output_lookup_function_body): When SHAREDLIB is specified
and SWITCH and DUP and not specified, test the table entry against
NULL before the string comparison.
* tests/test-6.exp: Update.
* doc/gperf.texi (Output Details): Document option -P.
* NEWS: Update.
Suggested by Ulrich Drepper.
2002-12-08 Bruno Haible <bruno@clisp.org>
* tests/permut2.gperf, tests/permut2.exp: New files.
* tests/permut3.gperf, tests/permut3.exp: New files.
* tests/charsets.gperf: New file, from Bruce Lilly.
* tests/charsets.exp: New file.
* tests/languages.gperf: New file, from Bruce Lilly.
* tests/languages.exp: New file.
* Makefile.in (check-test): Test them all.
Completely new asso_values search algorithm.
* src/search.h (Search::compute_occurrence, Search::clear_determined,
Search::set_determined, Search::already_determined, Search::reorder):
Remove functions.
(Search::init_asso_values, Search::sort_by_occurrence,
Search::compute_occurrence, Search::sort_by_occurrence,
Search::has_collisions, Search::collision_prior_to): Remove functions.
(Search::compute_partition, Search::count_possible_collisions,
Search::unchanged_partition): New method declarations.
(Search::_determined): Remove field.
* src/search.cc (Search::prepare): Don't initialize _determined.
(Search::compute_occurrence, greater_by_occurrence,
Search::clear_determined, Search::set_determined,
Search::already_determined, Search::reorder): Remove functions.
(Search::init_asso_values, compute_disjoint_union,
Search::sort_by_occurrence, Search::compute_occurrence,
Search::sort_by_occurrence, Search::has_collisions,
Search::collision_prior_to): Remove functions.
(StackEntry): Remove class.
(EquivalenceClass, Step): New classes.
(equals, Search::compute_partition, delete_partition,
Search::count_possible_collisions, Search::unchanged_partition): New
functions.
(Search::find_asso_values): Completely rewritten.
(Search::find_good_asso_values): Don't call reorder().
(Search::~Search): Don't free _determined.
* src/keyword.h (KeywordExt::_occurrence): Remove field.
* src/options.h (ORDER, FAST, OPT_CHOICE): Remove enum values.
(Options::_iterations): Remove field.
* src/options.icc (Options::get_iterations): Remove method.
* src/options.cc (Options::long_usage): Remove mention of -f and -o.
(Options::Options): Don't initialize _iterations.
(Options::~Options): Update.
(Options::parse_options): Do nothing for options -f, -o, -O.
* doc/gperf.texi: (Contributors): Update.
(Algorithmic Details): Remove options -f and -o. Update description
of option -s.
* tests/c-parse.exp, tests/chill.exp, tests/cplusplus.exp,
tests/gpc.exp, tests/java.exp, tests/modula2.exp, tests/objc.exp,
tests/test-4.exp): Regenerated, smaller than before.
* tests/test-6.exp: Update.
* NEWS: Update.
2002-12-08 Bruno Haible <bruno@clisp.org>
* src/search.h (Search::_alpha_size): Change type to 'unsigned int'.
(Search::_asso_value_max): Likewise.
* src/search.cc (Search::prepare_asso_values): Update.
(Search::init_asso_values): Update.
(Search::~Search): Update.
* src/output.h (Output::Output): Change alpha_size type to
'unsigned int'.
(Output::_alpha_size): Change type to 'unsigned int'.
* src/output.cc (Output::Output): Change alpha_size type to
'unsigned int'.
(Output::output_hash_function): Update.
2002-12-07 Bruno Haible <bruno@clisp.org>
* src/options.h (OPT_CHOICE): New enum value.
* src/options.cc (Options::~Options): Update.
(long_options): New option --optimized-collision-resolution.
(Options::parse_options): Accept option -O.
* src/search.h (Search::sort_by_occurrence): Change argument to
'unsigned int'.
(Search::compute_occurrence, Search::sort_by_occurrence): New method
declarations.
* src/search.cc (Search::sort_by_occurrence): Change argument to
'unsigned int'.
(Search::compute_occurrence, Search::sort_by_occurrence): New methods.
(Search::find_asso_values): Implement OPT_CHOICE. More debugging
output.
* src/search.cc (Search::prepare_asso_values) [DEBUG]: Also print
the keyword list in order.
(Search::find_asso_values) [DEBUG]: Upon failure, print the union_set.
* src/options.h (Options::get_size_multiple): Change return type to
float.
(Options::_size_multiple): Change type to float.
* src/options.icc (Options::get_size_multiple): Change return type to
float.
* src/options.cc (Options::long_usage): Update description of option
-s.
(Options::~Options): Update.
(Options::parse_options): For option -s, accept a fraction.
* src/search.cc (Search::prepare_asso_values): Use get_size_multiple
as it is.
* tests/test-6.exp: Update.
* doc/gperf.texi (Algorithmic Details): Update description of option
-s.
2002-12-04 Bruno Haible <bruno@clisp.org>
Improve debugging output.
* src/hash-table.h (Hash_Table::dump): New method.
* src/hash-table.cc (Hash_Table::dump): New method, extracted from
destructor.
(Hash_Table::~Hash_Table): No longer print the contents.
* src/positions.h (PositionReverseIterator): New class.
* src/positions.icc (PositionReverseIterator::PositionReverseIterator,
PositionReverseIterator::next): New methods.
* src/search.cc (Search::find_positions): If debugging, print the
result.
(Search::find_alpha_inc): If debugging, print the result.
(Search::prepare): Explicitly dump the hash table's contents here.
Portability fixes.
* src/positions.h (Positions::LASTCHAR, Positions::MAX_KEY_POS,
PositionIterator::EOS): Define as compile-time constants using enum.
* src/bool-array.cc (Bool_Array::~Bool_Array): Remove const qualifier
of pointer to be deleted.
* src/input.cc (Input::~Input): Likewise.
* src/keyword.cc (KeywordExt::delete_selchars): Likewise.
* src/main.cc (main): Likewise.
* src/hash-table.cc (Hash_Table::~Hash_Table): Limit scope of 'for'
variables.
* src/search.cc (Search::prepare_asso_values): Use a static_cast to
convert from time_t to long. This is possible because ISO C 99 says
time_t is a numeric type.
2002-11-20 Bruno Haible <bruno@clisp.org>
* src/search.cc (Search::find_asso_values): Avoid gcc warnings about
uninitialized variables.
Implement backtracking.
* src/search.h (Search::has_collisions): Renamed from
Search::less_collisions. Return a boolean.
* src/search.cc (Search::has_collisions): Renamed from
Search::less_collisions. Return a boolean.
(StackEntry): Remove field _collisions_so_far.
(Search::find_asso_values): Backtrack when encountering an unresolved
collision. Assume collisions_so_far is always zero.
(Search::optimize): Exit if there are accidental duplicates at the end.
* src/output.cc (Output::num_hash_values): Simply return the list
length.
(Output::output_keylength_table): Remove handling of accidental
duplicates.
(Output::output_keyword_table, Output::output_lookup_array): Likewise.
(output_switch_case, output_switches): Likewise.
* doc/gperf.texi (Algorithmic Details): Adjust description of options
-D, -f, -o, -r.
(Bugs): Remove note about missing backtracking.
(Projects): Likewise.
2002-11-19 Bruno Haible <bruno@clisp.org>
Prepare for backtracking.
* src/search.h (Search::try_asso_value, Search::change_some_asso_value):
Remove declarations.
(Search::less_collisions, Search::collision_prior_to): New declarations.
(Search::_fewest_collisions, Search::_union_set, Search::_num_done):
Remove fields.
* src/search.cc (Search::prepare_asso_values): Don't initialize
_union_set.
(Search::try_asso_value, Search::change_some_asso_value): Remove
methods.
(Search::less_collisions, Search::collision_prior_to): New methods.
(StackEntry): New class.
(Search::find_asso_values): Reorganized to use pseudo-recursion.
(Search::~Search): Don't free _union_set.
* src/search.h (Search::find_good_asso_values): New declaration.
* src/search.cc: Add comments about the basic structure of the
algorithm.
(Search::find_positions): Move the option[POSITIONS] test to here.
(Search::find_good_asso_values): New method, extracted from
Search::optimize.
(Search::optimize): Remove option[POSITIONS] test. Call
find_good_asso_values.
2002-11-17 Bruno Haible <bruno@clisp.org>
* src/options.cc (Options::parse_options): Include copyright notice
and authors in --version output.
Avoid artificial duplicates.
* src/keyword.h (KeywordExt::init_selchars_tuple): New declaration.
(KeywordExt::init_selchars_multiset): Renamed from
KeywordExt::init_selchars.
(KeywordExt::init_selchars_low): New declaration.
* src/keyword.cc (KeywordExt::init_selchars_low): Renamed from
KeywordExt::init_selchars. Add alpha_inc argument. Remove sorting.
(KeywordExt::init_selchars_tuple): New method.
(KeywordExt::init_selchars_multiset): New method, replaces
KeywordExt::init_selchars.
* src/search.h (Search::init_selchars_tuple): Renamed from
Search::init_selchars.
(Search::count_duplicates_tuple): Renamed from Search::count_duplicates.
(Search::init_selchars_multiset, Search::count_duplicates_multiset,
Search::find_alpha_inc): New declarations.
(Search::_alpha_inc): New field.
(Search::_alpha_size, Search::_occurrences, Search::_asso_values,
Search::_determined): Make non-const.
* src/search.cc (Search::Search): Don't initialize _key_positions,
_alpha_size, _occurrences, _asso_values, _determined here.
(Search::init_selchars_tuple): Renamed from Search::init_selchars.
(Search::count_duplicates_tuple): Renamed from Search::count_duplicates.
(Search::find_positions): Update.
(Search::init_selchars_multiset, Search::count_duplicates_multiset,
Search::find_alpha_inc): New methods.
(Search::prepare): Move preprepare, find_positions calls away.
Initialize _alpha_size, _occurrences, _asso_values, _determined here.
(Search::optimize): Call preprepare, find_positions here. Initialize
_key_positions here.
(Search::~Search): Deallocate _alpha_inc.
* src/output.cc (Output::Output): Add alpha_inc argument.
(Output::output_hash_function): Use _alpha_inc.
* src/output.h (Output::Output): Add alpha_inc argument.
(Output::_alpha_inc): New field.
* src/main.cc (main): Pass _alpha_inc from Search to Output.
* tests/chill.exp: Update.
* doc/gperf.texi (Algorithmic Details): Remove description of
artificial duplicates.
* src/keyword.h (KeywordExt::_selchars): Change type to
'const unsigned int *'.
* src/keyword.cc (sort_char_set): Change argument type to
'unsigned int *'.
(KeywordExt::init_selchars): Update.
* src/search.h (Search::sort_by_occurrence): Change argument type to
'unsigned int *'.
(Search::try_asso_value): Change argument type to 'unsigned int'.
(Search::_union_set): Change type to 'unsigned int *'.
* src/search.cc (Search::prepare, Search::compute_occurrence,
Search::set_determined, Search::already_determined,
Search::prepare_asso_values, Search::compute_hash): Update.
(compute_disjoint_union): Change argument types to 'unsigned int *'.
(Search::sort_by_occurrence): Likewise.
(Search::try_asso_value): Change argument type to 'unsigned int'.
(Search::change_some_asso_value, Search::~Search): Update.
* src/hash-table.cc (Hash_Table::~Hash_Table, Hash_Table::equal,
Hash_Table::insert): Update.
* src/positions.h: New file, extracted from options.h.
* src/positions.icc: New file, extracted from options.icc.
* src/positions.cc: New file, extracted from options.cc.
* src/options.h: Include positions.h. Move classes Positions and
PositionsIterator away.
* src/options.icc: Move classes Positions and PositionsIterator away.
* src/options.cc: Move class Positions away.
* src/keyword.cc: Include positions.h instead of options.h.
* src/output.h: Include positions.h instead of options.h.
* src/search.h: Include positions.h instead of options.h.
* src/Makefile.in (OBJECTS): Add positions.o.
(POSITIONS_H): New variable.
(OPTIONS_H, SEARCH_H, OUTPUT_H, keyword.o): Use it.
(positions.o): New rule.
* src/options.h (POSITIONS): New enum value.
(Positions::Positions): New copy constructor.
(Positions::operator=, Positions::contains, Position::add,
Positions::remove, Positions::print): New method declaration.
(Options::get_max_keysig_size): Remove method.
* src/options.icc (Positions::Positions): New copy constructor.
(Positions::operator=): New method.
(Options::get_max_keysig_size): Remove method.
* src/options.cc (Options::Options): Initialize _key_positions
trivially.
(Options::parse_options): Option -k sets POSITIONS.
(Positions::contains, Positions::add, Positions::remove,
Positions::print): New methods.
* src/hash-table.cc (Hash_Table::~Hash_Table): Compute the field
width explicitly, instead of using Options::get_max_keysig_size.
* src/keyword.h (KeywordExt::init_selchars): Add arguments
use_all_chars, positions.
(KeywordExt::delete_selchars): New declaration.
* src/keyword.cc (KeywordExt::init_selchars): Add arguments
use_all_chars, positions. Remove error message if there are no key
positions.
(KeywordExt::delete_selchars): New method.
* src/search.h: Include options.h.
(Search::preprepare, Search::init_selchars, Search::delete_selchars,
Search::count_duplicates, Search::find_positions): New declarations.
(Search::_key_positions): New field.
* src/search.cc (Search::Search): Initialize _key_positions.
(Search::preprepare, Search::init_selchars, Search::delete_selchars,
Search::count_duplicates, Search::find_positions): New functions.
(Search::prepare): Call preprepare and find_positions. Tweak error
message.
(Search::get_max_keysig_size): Use _key_positions instead of
option.get_key_positions().
(Search::optimize): Tweak error message.
* src/output.h: Include options.h.
(Output::Output): Add Positions argument.
(Output::_key_positions): New field.
* src/output.cc (Output::Output): Add Positions argument.
(Output::output_hash_function): Omit the table if there are no
positions at all. Use _key_positions instead of
option.get_key_positions().
(Output::output): Output the computed positions as a comment.
* src/main.cc (main): Pass the Positions from Searcher to Output.
* src/Makefile.in (SEARCH_H, OUTPUT_H): Include OPTIONS_H.
* tests/Makefile.in (check-test): Pass key positions explicitly.
* tests/gpc.exp: Update.
* tests/test-4.exp: Update.
* doc/gperf.texi (Algorithmic Details): Mention that -k is not needed
usually.
2002-11-16 Bruno Haible <bruno@clisp.org>
* src/options.h (Options::get_slot_name): Renamed from
Options::get_key_name.
(Options::set, Options::set_language, Options::set_total_switches,
Options::set_function_name, Options::set_slot_name,
Options::set_class_name, Options::set_hash_name,
Options::set_wordlist_name, Options::set_delimiters): New method
declarations.
(Options::_language): New field.
(Options::_slot_name): Renamed from Options::_key_name.
* src/options.icc (Options::set): New method.
(Options::get_slot_name): Renamed from Options::get_key_name.
* src/options.cc (DEFAULT_FUNCTION_NAME): Renamed from DEFAULT_NAME.
(DEFAULT_SLOT_NAME): Renamed from DEFAULT_NAME.
(Options::Options): Initialize _language. Update.
(Options::~Options): Update.
(Options::set_language, Options::set_total_switches,
Options::set_function_name, Options::set_slot_name,
Options::set_class_name, Options::set_hash_name,
Options::set_wordlist_name, Options::set_delimiters): New methods.
(Options::parse_options): Call set_language. Update.
* src/input.cc (is_declaration, is_declaration_with_arg,
is_define_declaration): New functions.
(Input::read_input): Accept %DECL declarations.
* src/output.cc (Output::output_lookup_function_body): Update.
* doc/gperf.texi (Declarations): Add new subnodes.
(User-supplied Struct, Gperf Declarations, C Code Inclusion): New
nodes.
(Keywords, Output Format, Binary Strings, Options): Mention %
declarations as being equivalent to the command line options.
* src/options.cc (Options::long_usage): Rename options -H, -N, -l, -G.
(long_options): Add --hash-function-name, --lookup-function-name,
--compare-lengths.
* doc/gperf.texi (Output Details): Rename options -H, -N, -l, -G.
* tests/test-6.exp: Update.
* src/options.cc (DEFAULT_DELIMITERS): Remove newline.
* src/options.cc (Options::long_usage): Change default --delimiters.
* doc/gperf.texi (Input Details): Likewise.
* tests/test-6.exp: Update.
* doc/gperf.texi: Move description of option -l from section
Algorithmic Details to section Output Details.
* src/options.cc (Options::long_usage): Likewise.
* tests/test-6.exp: Update.
2002-11-12 Bruno Haible <bruno@clisp.org>
* src/options.h (Output::get_output_file_name): New method.
(Output::_output_file_name): New field.
* src/options.icc (Options::get_output_file_name): New method.
* src/options.cc (Options::long_usage): Document option --output-file.
(Options::Options): Initialize _output_file_name.
(long_options): Add --output-file.
(Options::parse_options): Handle it.
* src/main.cc (main): Open the output file if given by name.
* doc/gperf.texi (Output File): New section.
* tests/test-6.exp: Update.
2002-11-10 Bruno Haible <bruno@clisp.org>
* src/input.cc (pretty_input_file_name): New function.
(read_input): Use it in all error and warning messages.
* src/keyword.h (Keyword::_lineno): New field.
* src/input.h (Input::_struct_decl_lineno): New field.
* src/input.cc (Input::read_input): Set _struct_decl_lineno. Fill
each keyword's _lineno field.
* src/main.cc (main): Pass _struct_decl_lineno from Input to Output.
* src/output.h (Output::Output) Add struct_decl_lineno argument.
(Output::_struct_decl_lineno): New field.
* src/output.cc (Output::Output) Add struct_decl_lineno argument.
(output_keyword_entry): Emit #line directive before table entry.
(Output::output): Emit #line directive before _struct_decl.
Fix memory leaks.
* src/keyword.h (empty_string): New declaration.
* src/keyword.cc (empty_string): New variable.
* src/input.h (Input::_input): Make public.
(Input::_input_end): New field.
* src/input.cc (read_input): When removing leading whitespace from
struct_decl, reallocate it. For rest, use empty_string instead of "".
Set _input_end.
(Input::~Input): Delete _struct_decl, _struct_tag, _return_type.
* src/search.cc (Search::prepare): When removing an element from
the keyword list, delete the list node.
(Search::~Search): Delete _occurrences, _asso_values.
* src/main.cc (main): Between Search::~Search and Input::~Input,
destroy the keyword list.
Rewrite the input routines.
* src/input.h: Don't include read-line.h.
(Input): Don't inherit from class Read_Line.
(Input::read_keys, Input::strcspn, Input::set_output_types,
Input::get_array_type, Input::save_include_src,
Input::get_special_input): Remove declarations.
(Input::read_input): New declaration.
(Input::_struct_decl): Renamed from Input::_array_type.
(Input::_verbatim_declarations): Renamed from Input::_include_src.
(Input::_verbatim_code): Replaces Input::_additional_code.
* src/input.cc: Completely rewritten.
* src/output.h (Output::Output): Update the verbatim_* arguments.
(Output::_struct_decl): Renamed from Output::_array_type.
(Output::_verbatim_declarations): Renamed from Output::_include_src.
(Output::_verbatim_code): Replaces Output::_additional_code.
* src/output.cc (Output::Output): Update the verbatim_* arguments.
(Output::output): Output the verbatim_* code pieces with #line.
* src/main.cc (main): Call Input::read_input instead of
Input::read_keys. Update Output::Output arguments.
* src/read-line.h: Remove file.
* src/read-line.cc, src/read-line.icc: Remove files.
* src/Makefile.in (OBJECTS): Remove read-line.o.
(READ_LINE_H): Remove variable.
(INPUT_H): Update.
(read-line.o): Remove rule.
* doc/gperf.texi (Declarations): Correct the example.
(Keywords): Mention that lines starting with % are forbidden here.
* tests/c-parse.exp: Update.
* tests/cplusplus.exp: Update.
* tests/gpc.exp: Update.
* tests/java.exp: Update.
* tests/objc.exp: Update.
* tests/test-4.exp: Update.
* src/options.h (Options::get_input_file_name): New declaration.
(Options::_input_file_name): New field.
* src/options.icc (Options::get_input_file_name): New method.
* src/options.cc (Options::Options): Initialize _input_file_name.
(Options::parse_options): Don't open input file, only store it in
_input_file_name.
* src/main.cc (main): Open input file here.
Print an error message upon write error on the output file.
Upgrade to autoconf-2.52.
* configure.in: Use AC_CONFIG_SUBDIRS instead of AC_OUTPUT_SUBDIRS.
* Makefile.devel (configure, lib/configure, src/configure,
tests/configure, doc/configure): Use autoconf-2.52.
2002-11-09 Bruno Haible <bruno@clisp.org>
* doc/gperf.texi: Talk about "bytes" instead of "characters". Talk
about "keywords", not "keys". Talk about "input file", not "keyfile".
(@menu): Fix a menu entry.
(Contributors): Don't mention cperf.
(Motivation): Fix an off-by-one error in the definition of "minimal".
Mention GNU Java. Recommend http URL instead of anonymous ftp.
(Search Structures): Mention GNU Java.
(Output Format): Drop reference to node 'Implementation'.
(Output Details): Talk about "slot-name" instead of "key name".
(Algorithmic Details): Talk about "selected byte positons", not
"key positions". Upper limit is now 255. Explain a third reason
why duplicates can occur. Describe negative effects of
--occurrence-sort.
(Implementation): Remove chapter.
2002-11-07 Bruno Haible <bruno@clisp.org>
* src/bool-array.cc (Bool_Array::~Bool_Array): Free _storage_array.
* src/search.cc (Search::~Search): Free _union_set, _determined.
* tests/Makefile.in (check-test): Don't redirect stderr.
2002-11-05 Bruno Haible <bruno@clisp.org>
* src/keyword-list.h (mergesort_list): New declarations.
* src/keyword-list.cc (Keyword_Comparison): New type.
(merge, mergesort_list): New functions, moved here from search.cc.
* src/search.h (Search::merge, Search::merge_sort): Remove methods.
(Search::_occurrence_sort, Search::_hash_sort): Remove fields.
* src/search.cc (Search::merge, Search::merge_sort): Remove methods.
(greater_by_occurrence, less_by_hash_value): New functions.
(Search::reorder, Search::sort): Use mergesort_list.
2002-11-04 Bruno Haible <bruno@clisp.org>
* src/options.h (Options::_asso_iterations): New field.
(Options::get_asso_iterations): New method declaration.
* src/options.icc (Options::get_asso_iterations): New method.
* src/options.cc (Options::short_usage): Mention j<jump> and m<num>.
(Options::long_usage): Document option -m.
(Options::Options): Initialize _asso_iterations.
(Options::~Options): Print _asso_iterations too.
(long_options): Add --multiple-iterations.
(Options::parse_options): Handle option -m.
* src/keyword-list.h (copy_list, delete_list): New declarations.
* src/keyword-list.cc (copy_list, delete_list): New functions.
* src/search.h (Search::_initial_asso_value, Search::_jump): New fields.
* src/search.cc (Search::prepare_asso_values): Initialize
_initial_asso_value and _jump here.
(Search::init_asso_values): Use _initial_asso_value.
(Search::try_asso_value): Use _jump.
(Search::optimize): If option -m was given, iterate over different
values for _initial_asso_value and _jump.
* doc/gperf.texi (Algorithmic Details): Document option -m.
* tests/test-6.exp: Update.
2002-11-03 Bruno Haible <bruno@clisp.org>
Bug fix: When option -j 0 was used without option -r, the output was
not random.
* src/search.h (Search::prepare_asso_values): New method declaration.
* src/search.cc (Search::prepare_asso_values): New method, extracted
from Search::init_asso_values. Call srand also when "-j 0" was given.
(Search::optimize): Call prepare_asso_values().
* src/hash-table.h (Hash_Table::_ignore_length, Hash_Table::equal):
Declare as const.
* src/hash-table.cc (Hash_Table::equal): Declare as const.
* src/input.h (Input::_factory): Declare as const.
* src/keyword-list.h (Keyword_List::first, KeywordExt_List::first):
Declare as const.
* src/keyword-list.icc (Keyword_List::first, KeywordExt_List::first):
Declare as const.
* src/output.h (Output::num_hash_values, Output::output_constants,
Output::output_hash_function, Output::output_keylength_table,
Output::output_keyword_table, Output::output_lookup_array,
Output::output_lookup_tables, Output::output_lookup_function_body,
Output::output_lookup_function, Output::_array_type,
Output::_additional_code, Output::_include_src, Output::_total_keys,
Output::_total_duplicates, Output::_max_key_len, Output::_min_key_len):
Declare as const.
* src/output.cc (Output::num_hash_values, Output::output_constants,
Output::output_hash_function, Output::output_keylength_table,
Output::output_keyword_table, Output::output_lookup_array,
Output::output_lookup_tables, Output::output_lookup_function_body,
Output::output_lookup_function): Declare as const.
* src/search.h (Search::merge, Search::merge_sort,
Search::compute_occurrence, Search::already_determined,
Search::keyword_list_length, Search::max_key_length,
Search::get_max_keysig_size, Search::compute_hash,
Search::sort_by_occurrence): Declare as const.
* src/search.cc (Search::merge, Search::merge_sort,
Search::compute_occurrence, Search::already_determined,
Search::keyword_list_length, Search::max_key_length,
Search::get_max_keysig_size, Search::compute_hash,
Search::sort_by_occurrence): Declare as const.
* src/output.cc (Output::output): Set char_to_index to a cast in all
cases. Avoids gcc warnings on the generated code.
* src/output.cc (Output_Enum): Prepend an underscore to field names.
(Output_Expr1): Likewise.
(Output::output_hash_function): Simplify the special case for "-k 1,$".
* src/search.h (Search::init_asso_values, Search::find_asso_values):
New declarations.
(Search::try_asso_value): Renamed from Search::affects_prev.
(Search::change_some_asso_value): Renamed from Search::change.
(Search::set_asso_max, Search::get_asso_max): Remove methods.
(Search::_union_set): New field.
* src/search.cc (Search::init_asso_values): New method, extracted
from Search::optimize.
(Search::try_asso_value): Renamed from Search::affects_prev. Take the
iteration count as argument.
(Search::change_some_asso_value): Renamed from Search::change. Don't
make union_set static. Don't increment _fewest_collisions here.
(Search::find_asso_values): New method, extracted from
Search::optimize.
(Search::optimize); Update.
* src/search.h (Search::compute_hash): Renamed from Search::hash.
(Search::compute_disjoint_union): Remove declaration.
(Search::sort_by_occurrence): Renamed from Search::sort_set.
* src/search.cc (Search::compute_hash): Renamed from Search::hash.
(compute_disjoint_union): Renamed from Search::compute_disjoint_union.
(Search::sort_by_occurrence): Renamed from Search::sort_set.
(Search::change): Simplify loop.
* src/search.h (Search::clear_determined): New declaration.
* src/search.cc (Search::clear_determined): New method.
(Search::already_determined): Optimize.
(Search::reorder): Even when the next keyword after the current one
is completely determined, move all determined keywords after the
current one.
Compute the occurrences after removal of duplicates, not before.
* src/keyword.h (KeywordExt::init_selchars): Remove occurrences
argument.
* src/keyword.cc (KeywordExt::init_selchars): Likewise.
* src/search.cc (Search::prepare): Reorder the code. Compute the
occurrences after removal of duplicates.
(Search::merge_sort): Optimize the loop.
(Search::compute_occurrence): Renamed from Search::get_occurrence.
* src/search.h (Search::compute_occurrence): Renamed from
Search::get_occurrence.
* tests/chill.exp: Regenerated.
Bug fix: The hash table could fail to detect duplicates, between
keywords of different length, when option -n (option[NOLENGTH]) was
given.
* src/hash-table.h (Hash_Table::Hash_Table): Pass table size, not
vector and vector size as arguments.
(Hash_Table::_log_size): New field.
(Hash_Table::equal): New declaration.
* src/hash-table.cc (size_factor): New variable.
(Hash_Table::Hash_Table): Pass table size, not vector and vector size
as arguments. Allocate the vector here.
(Hash_Table::~Hash_Table): Deallocate the vector here.
(Hash_Table::equal): New function.
(Hash_Table::insert): Use it. Don't use item->_allchars_length for the
increment if _ignore_length is true.
* src/search.cc (TABLE_MULTIPLE): Remove variable.
(Search::prepare): Update.
2002-11-02 Bruno Haible <bruno@clisp.org>
Provide documentation also in PDF format.
* doc/Makefile.in (pdfdir, TEXI2PDF): New variables.
(all): Depend on pdf.
(pdf, gperf.pdf): New rules.
(maintainer-clean): Remove the PDF file.
* src/keyword-list.icc: New file, extracted from keyword-list.h.
* src/keyword-list.h: Include keyword-list.icc. Move inline methods
to there.
* src/keyword-list.cc: Include keyword-list.icc.
* src/Makefile.in (KEYWORD_LIST_H): Add keyword-list.icc.
* lib/hashpjw.h (hashpjw): Change argument type to 'unsigned char *'.
* lib/hash.cc (hashpjw): Likewise.
* src/keyword.icc: New file.
* src/keyword.h: Include keyword.icc.
(KeywordExt::_selchars): Change type to 'unsigned char *'.
* src/keyword.cc: Include keyword.icc.
(Keyword::Keyword, KeywordExt::KeywordExt): Move to keyword.icc.
(sort_char_set): Change argument type to 'unsigned char *'.
(KeywordExt::init_selchars): Update.
* src/search.h (Search::compute_disjoint_union): Change argument types
to 'unsigned char *'.
(Search::sort_set): Likewise.
(Search::affects_prev): Change argument type to 'unsigned char'.
* src/search.cc (Search::prepare): Initialize _duplicate_link here.
(Search::get_occurrence, Search::set_determined,
Search::already_determined, Search::hash): Update.
(Search::compute_disjoint_union): Change argument types to
'unsigned char *'.
(Search::sort_set): Likewise.
(Search::affects_prev): Change argument type to 'unsigned char'.
(Search::change): Update.
* src/Makefile.in (KEYWORD_H): Add keyword.icc.
* src/options.cc (Options::parse_options): Fix error message.
* src/read-line.h (Read_Line::Read_Line): Make FILE* argument
mandatory. Move body to read-line.icc.
* src/read-line.icc (Read_Line::Read_Line): New constructor.
* src/input.h (Input::Input): Add FILE* argument.
* src/input.cc (Input::Input): Likewise.
* src/main.cc (main): Pass stdin to Input constructor.
* src/options.h (DEFAULTCHARS): Remove.
(Positions::MAX_KEY_POS): Set to 255.
(Positions::_positions): Increase array size.
(PositionIterator::EOS): Set to -1.
(PositionIterator::_index): Change type to 'unsigned int'.
* src/options.icc (Positions::Positions): Don't store
PositionIterator::EOS.
(PositionIterator::next): Produce PositionIterator::EOS here.
* src/options.cc (Options::long_usage): Use MAX_KEY_POS, not
MAX_KEY_POS-1.
(PositionStringParser): Rename field _size to _in_range. Rename
field _curr_value to _range_curr_value. Rename field _upper_bound
to _range_upper_bound.
(PositionStringParser::nextPosition): Comments.
(Options::Options): Update.
(Options::~Options): Update.
(long_options): Use NULL, not 0.
(Options::parse_options): Set BAD_VALUE to -2; -1 is now EOS. Bug fix:
Check against array overflow when more than MAX_KEY_POS positions are
given. Don't store PositionIterator::EOS.
Check against extra arguments before opening the input file.
* src/output.cc (Output::output_hash_function): Change test which
was for option[DEFAULTCHARS].
* tests/test-6.exp: Update.
* src/options.h (Options::get_delimiters): Renamed from
Options::get_delimiter.
* src/options.icc (Options::get_delimiters): Renamed from
Options::get_delimiter.
* src/input.cc (Input::read_keys): Update.
Bug fix.
* src/options.cc (Options::print_options): Escape backquote inside
double-quoted strings.
Bug fix.
* src/keyword.cc (KeywordExt::init_selchars): Avoid comparison with
uninitialized member variable. Found with 'valgrind'.
* src/version.cc: Include version.h.
* src/Makefile.in (OBJECTS): Reorder.
(KEYWORD_H, KEYWORD_LIST_H, INPUT_H, SEARCH_H, OUTPUT_H): New
variables.
(HASH_TABLE_H): Update.
(options.o, read-line.o, keyword.o, keyword-list.o, input.o, search.o,
output.o, main.o): Update dependencies.
* src/vectors.h: Remove file.
* src/vectors.cc: Remove file.
* src/search.h: Don't include vectors.h.
(Search): Don't inherit from Vectors. New fields _alpha_size,
_occurrences, _asso_values.
(Search::_determined, Search::get_occurrence, Search::set_determined,
Search::already_determined, Search::hash, Search::sort_set): Make
nonstatic.
* src/search.cc (Search::Search): Initialize _alpha_size, _occurrences,
_asso_values, _determined.
(Search::optimize, Search::~Search): Update.
* src/output.h: Don't include vectors.h.
(Output): Remove field _v. New fields _alpha_size, _occurrences,
_asso_values.
(Output::Output): Replace Vectors* argument with alpha_size,
occurrences, asso_values.
* src/output.cc (Output::Output): Replace Vectors* argument with
alpha_size, occurrences, asso_values.
(Output::output_hash_function): Update.
* src/main.cc (main): Don't set Vectors::ALPHA_SIZE.
Pass _alpha_size, _occurrences, _asso_values from Search to Output.
* src/keyword.h: Don't include vectors.h.
* src/Makefile.in (OBJECTS): Remove vectors.o.
(VECTORS_H): Remove variable.
(vectors.o): Remove rule.
* src/search.h: New file, combines src/key-list.h, src/gen-perf.h.
* src/search,cc: New file, combines src/key-list.cc, src/gen-perf.cc.
* src/key-list.h: Remove file.
* src/key-list.cc: Remove file.
* src/gen-perf.h: Remove file.
* src/gen-perf.cc: Remove file.
* src/main.cc (KeywordExt_Factory): Moved here from gen-perf.cc.
(main): Inline some code from gen-perf.cc.
* src/keyword.h (KeywordExt::init_selchars): Take the occurrences
vector as argument.
* src/keyword.cc (KeywordExt::init_selchars): Take the occurrences
vector as argument.
* src/input.cc (Input::set_output_types): Initialize _array_type,
_return_type, _struct_tag.
(Input::read_keys): Initialize _additional_code.
* src/Makefile.in (OBJECTS): Add search.o.
Remove key-list.o, gen-perf.o.
(KEY_LIST_H, GEN_PERF_H): Remove variables.
(gen-perf.o, key-list.o): Remove rules.
(search.o): New rule.
* *, */*: Update copyright notice to GPL version 2.
* src/keyword-list.h (Keyword_List): New class.
(KeywordExt_List): Inherit from it.
* src/keyword-list.cc (Keyword_List::Keyword_List): New constructor.
(KeywordExt_List::KeywordExt_List): Update.
* src/input.h (Input::Input): Add Keyword_Factory argument.
(Input::_factory): New field.
(Input::_head): Change type to Keyword_List*.
(Input::parse_line): New declaration.
* src/input.cc (Input::Input): New constructor.
(Input::parse_line): Renamed from parse_line. Use the _factory.
(Input::read_keys): Update.
* src/key-list.cc (KeywordExt_Factory): New class.
(Key_List::read_keys): Pass a KeywordExt_Factory as Input constructor
argument.
Avoid g++ -Wold-style-cast warnings.
* src/bool-array.icc: Use new-style casts.
* src/gen-perf.cc: Likewise.
* src/input.cc: Likewise.
* src/key-list.cc: Likewise.
* src/keyword.cc: Likewise.
* src/options.cc: Likewise.
* src/output.cc: Likewise.
* src/hash-table.cc: Likewise. Remove (char *) cast in memset argument.
* src/keyword-list.h (KeywordExt_List): Don't inherit from KeywordExt.
(KeywordExt_List::KeywordExt_List): Take a KeywordExt* as argument.
(KeywordExt_List::_car): New field.
(KeywordExt_List::first): Use it.
* src/keyword-list.cc (KeywordExt_List::KeywordExt_List): Take a
KeywordExt* as argument.
* src/input.cc (parse_line): Create the KeywordExt separately.
Start using bool.
* src/bool-array.h (Bool_Array::set_bit): Change return type to bool.
* src/bool-array.icc (Bool_Array::set_bit): Likewise.
* src/gen-perf.h (Gen_Perf::affects_prev): Likewise.
* src/gen-perf.cc (Gen_Perf::affects_prev): Likewise.
* src/hash-table.h (Hash_Table::_ignore_length): Change type to bool.
(Hash_Table::Hash_Table): Change 3rd argument type to bool.
* src/hash-table.cc (Hash_Table::Hash_Table): Likewise.
* src/input.h (Input::_additional_code): Change type to bool.
* src/input.cc (Input::read_keys): Update.
* src/key-list.h (Key_List::_occurrence_sort, Key_List::_hash_sort,
Key_List::_additional_code): Change type to bool.
(Key_List::_determined): Change element type to bool.
(Key_List::already_determined): Change return type to bool.
* src/key-list.cc (Key_List::_determined): Change element type to bool.
(Key_List::set_determined): Update.
(Key_List::already_determined): Change return type to bool.
(Key_List::reorder, Key_List::sort, Key_List::Key_List): Update.
* src/options.h (Positions::sort): Change return type to bool.
(Options::operator[]): Likewise.
* src/options.icc (Positions::sort): Change return type to bool.
(Options::operator[]): Likewise.
* src/output.h (Output::Output): Change 5th argument type to bool.
(Output::_additional_code): Change type to bool.
* src/output.cc (Output::Output): Change 5th argument type to bool.
2002-10-16 Bruno Haible <bruno@clisp.org>
* src/*.h: Align all member names at column 24.
2002-10-15 Bruno Haible <bruno@clisp.org>
* src/input.h: New file.
* src/input.cc: New file, extracted from key-list.cc.
* src/key-list.h (Key_List): Don't inherit from Read_Line.
(Key_List::get_special_input,
Key_List::save_include_src, Key_List::get_array_type,
Key_List::strcspn, Key_List::set_output_types): Remove methods.
* src/key-list.cc (Key_List::get_special_input,
Key_List::save_include_src, Key_List::get_array_type,
Key_List::strcspn, Key_List::set_output_types, parse_line): Move to
src/input.cc.
(Key_List::read_keys): Use Input::read_keys.
(Key_List::Key_List): Update.
* src/gen-perf.cc: Update.
* src/Makefile.in (OBJECTS): Add input.o.
(input.o): New rule.
2002-10-14 Bruno Haible <bruno@clisp.org>
* src/options.cc: Don't include "vector.h".
(Options::parse_options): Don't initialize Vectors::ALPHA_SIZE here.
* src/vectors.cc (Vectors::ALPHA_SIZE): Don't initialize here.
* src/gen-perf.cc (Gen_Perf::Gen_Perf): Initialize Vectors::ALPHA_SIZE.
* src/options.h (Positions): New class.
(PositionIterator): New class.
(Options::parse_options): Renamed from Options::operator().
(Options::get_asso_max, Options::set_asso_max): Move to class Key_List.
(Options::reset, Options::get): Remove, replaced by class
PositionIterator.
(Options::get_initial_asso_value): Renamed from Options::initial_value.
(Options::key_sort): Remove, replaced by Positions::sort.
(Options): Make all fields and methods non-static.
* src/options.icc (Positions::Positions, Positions::operator[],
Positions::get_size, Positions::pointer, Positions::set_size,
Positions::sort, PositionIterator::PositionIterator,
PositionIterator::next): New methods.
(Options::get_initial_asso_value): Renamed from Options::initial_value.
(Options::get_size_multiple): New method.
(Options::get_key_positions): New method.
(Options::get_max_keysig_size): Implement using _key_positions.
* src/options.cc (Options::long_usage): Split big string into small
pieces.
(PositionStringParser): Prefix field names with _.
(Options::Options): Update.
(Options::~Options): Fix explanation of of _size_multiple. Don't print
_key_positions if it is effectively ignored.
(Options::parse_options): Renamed from Options::operator(). Update.
* src/key-list.h (Key_List): New field _size. New methods get_asso_max,
set_asso_max, get_max_keysig_size.
* src/key-list.cc (Key_List::read_keys): Don't make side effects on
options.
(Key_List::dump): Use Key_List::get_max_keysig_size() instead of
Options::get_max_keysig_size().
(Key_List::get_max_keysig_size): New function.
* src/hash-table.cc (Hash_Table::~Hash_Table): Compute the field
width on the fly if option[ALLCHARS].
* src/gen-perf.cc (Gen_Perf::Gen_Perf): Update,
Use Options::get_size_multiple() instead of Options::get_asso_max().
Use Key_List::get_asso_max() instead of Options::get_asso_max(). Use
Key_List::get_max_keysig_size() instead of
Options::get_max_keysig_size().
(Gen_Perf::affects_prev): Likewise.
(Gen_Perf::change): Likewise.
* src/keyword.cc: Update.
* src/main.cc: Update.
* src/output.cc: Update.
* tests/test-6.exp: Update.
2002-10-13 Bruno Haible <bruno@clisp.org>
* src/bool-array.*: Some polishing.
* src/options.h (Options::operator=, Options::operator!=): Remove
unused methods.
* src/options.icc (Options::operator=, Options::operator!=): Remove.
* src/*.h: Prefix all field names with _.
* src/*.cc, src/*.icc: Update.
* src/*: Simplify declarations of functions without arguments.
2002-10-04 Bruno Haible <bruno@clisp.org>
* src/output.h: New file, extracted from key-list.h.
* src/output.cc: New file, extracted from key-list.cc.
* src/key-list.h (Key_List): Make some fields protected. Move output
routines to src/output.h.
* src/key-list.cc: Move output routines to src/output.cc.
* src/gen-perf.cc (Gen_Perf::doit_all): Use class Output.
* src/Makefile.in (OBJECTS): Add output.o.
(output.o): New rule.
2002-10-03 Bruno Haible <bruno@clisp.org>
* src/iterator.h: Remove file.
* src/iterator.cc: Remove file.
* src/options.cc: (PositionStringParser): New class, taken from old
iterator.cc.
* src/Makefile.in (OBJECTS): Remove iterator.o.
(ITERATOR_H): Remove variable.
(iterator.o): Remove rule.
* src/keyword-list.h: New file.
* src/keyword-list.cc: New file.
* src/list-node.h: Remove file.
* src/list-node.cc: Remove file.
* src/keyword.h (KeywordExt::init_selchars): New declaration.
* src/keyword.cc (sort_char_set, KeywordExt::init_selchars): New, from
old list-node.cc.
* src/gen-perf.cc: Replace List_Node by KeywordExt or KeywordExt_List,
as appropriate.
* src/hash-table.h: Likewise.
* src/key-list.h: Likewise.
* src/key-list.cc: Likewise.
* src/Makefile.in (OBJECTS): Remove list-node.o, add keyword-list.o.
(LIST_NODE_H): Remove macro.
(list-node.o): Remove rule.
(keyword-list.o): New rule.
* src/keyword.h (KeywordExt): New class.
* src/keyword.cc (KeywordExt): New constructor.
* src/list-node.h (List_Node): Inherit from KeywordExt.
* src/list-node.cc: Update.
* src/gen-perf.cc: Update.
* src/hash-table.cc: Update.
* src/key-list.cc: Update.
(output_keyword_entry): Change argument type to KeywordExt*.
* src/keyword.h: New file.
* src/keyword.cc: New file.
* src/list-node.h (List_Node): Extend Keyword.
* src/list-node.cc: Update.
* src/gen-perf.cc: Update.
* src/hash-table.cc: Update.
* src/key-list.cc: Update.
* src/Makefile.in (OBJECTS): Add keyword.o.
(keyword.o): New rule.
* src/key-list.cc (Key_List::read_keys): Allocate the memory for the
hash table using 'new'.
(Key_List::output_lookup_array): Allocate the memory for the duplicates
array using 'new'.
* src/options.h (LARGE_STACK_ARRAYS): Remove definition.
* src/main.cc (main): Remove setrlimit call.
* src/configure.in: Don't test for unistd.h, sys/time.h,
sys/resource.h, getrlimit, setrlimit.
* src/bool-array.h (Bool_Array): Make all members non-static.
Add an argument to the constructor. Remove init(), rename reset() to
clear(), rename find() to set_bit().
* src/bool-array.icc: Move init() code into the constructor.
Rename reset() to clear(), rename find() to set_bit().
* src/gen-perf.h (Gen_Perf): Add collision_detector member.
* src/gen-perf.cc: Update.
* src/gen-perf.h (Gen_Perf::doit_all): Renamed from
Gen_Perf::operator ().
* src/gen-perf.cc (Gen_Perf::doit_all): Renamed from
Gen_Perf::operator ().
* src/main.cc: Update.
* src/read-line.h (Read_Line::read_next_line): Renamed from
Read_Line::get_line.
* src/read-line.icc: Likewise.
* src/read-line.cc: Update.
* src/key-list.cc: Update.
* lib/getline.h: New file.
* lib/getline.cc: New file.
* lib/Makefile.in (OBJECTS): Add getline.o.
(getline.o): New rule.
* src/read-line.h (Read_Line::readln_aux): Remove declaration.
* src/read-line.cc (Read_Line::readln_aux): Remove function.
* src/read-line.icc (Read_Line::get_line): Use ::get_line.
* src/options.h (LARGE_STACK): Remove macro.
* src/bool-array.h (STORAGE_TYPE): Remove type.
Use 'unsigned int' instead of STORAGE_TYPE.
* src/bool-array.cc: Likewise.
* src/bool-array.icc: Likewise.
* src/gen-perf.cc: Likewise.
* src/new.cc: Remove file.
* src/Makefile.in (OBJECTS): Remove new.o.
(new.o): Remove rule.
* src/configure.in: Remove test for HAVE_THROW_DECL.
* acconfig.h: Remove file.
* src/trace.h: Remove file.
* src/trace.cc: Remove file.
* src/Makefile.in (OBJECTS): Remove trace.o.
(TRACE_H): Remove variable.
(trace.o): Remove rule.
Update all dependencies.
* src/bool-array.h, src/bool-array.cc, src/bool-array.icc: Don't use T.
* src/gen-perf.cc: Likewise.
* src/hash-table.cc: Likewise.
* src/iterator.cc: Likewise.
* src/key-list.cc: Likewise.
* src/list-node.cc: Likewise.
* src/main.cc: Likewise.
* src/new.cc: Likewise.
* src/options.h, src/options.cc, src/options.icc: Likewise.
* src/read-line.h, src/read-line.cc, src/read-line.icc: Likewise.
* tests/Makefile.in: Use gperf option -I, to avoid gcc-3.x warnings.
* tests/test.c: Don't use gets(), to avoid warnings.
2001-08-02 Bruno Haible <bruno@linuix.math.u-bordeaux.fr>
* doc/gperf.texi: Change bug report address to <bug-gnu-gperf@gnu.org>.
* README: Updated.
2000-12-18 Bruno Haible <bruno@linuix.math.u-bordeaux.fr>
* src/configure.in: Add check for rand() in libm. Needed for BeOS.
* src/Makefile.in (LIBS): Use @GPERF_LIBM@ instead of hardwiring -lm.
2000-11-20 Bruno Haible <bruno@linuix.math.u-bordeaux.fr>
* doc/help2man: Update to version 1.23.
2000-09-26 Bruno Haible <bruno@linuix.math.u-bordeaux.fr>
* gperf-2.7.2 released.
* doc/gperf.texi: Add a second bug report address
<gperf-bugs@lists.sourceforge.net>.
* README: Updated.
2000-08-28 Bruno Haible <bruno@linuix.math.u-bordeaux.fr>
* lib/getopt.h (struct option): Use "const" also when compiling in
C++ mode. Avoids warnings from Sun CC and HP-UX aCC.
* doc/Makefile.in (docdir): Change from $(datadir)/doc/@PACKAGE@ to
$(prefix)/doc/@PACKAGE@, following the newest GNU standards.
2000-08-20 Bruno Haible <bruno@linuix.math.u-bordeaux.fr>
* src/version.cc: Bump version number to 2.7.2.
* doc/gperf.texi: Likewise.
* doc/texinfo.tex: Update to post-texinfo-4.0 version. @code in the
title page now chooses a larger font. The overall layout of the
text is denser.
* AUTHORS: New file.
* tests/Makefile.in (all): Add check-lang-utf8 and check-lang-ucs2.
(check-lang-utf8, check-lang-ucs2): New targets.
(clean): Remove lu8out and lu2out.
* tests/lang-utf8.gperf, tests/lang-utf8.exp: New files.
* tests/lang-ucs2.gperf, tests/test2.c, tests/lang-ucs2.in,
tests/lang-ucs2.exp: New files.
Allow the use of embedded NULs in keys.
* lib/hash.h (hashpjw): Add a length argument.
* lib/hash.cc (hashpjw): Likewise. Don't stop when encountering a NUL
character.
* src/hash-table.h (Hash_Table constructor): Add ignore_len argument.
(Hash_Table::ignore_length): New field.
(Hash_Table::insert): Renamed from Hash_Table::operator(). Remove
ignore_length argument.
* src/hash-table.cc (NIL): Remove macro.
(Hash_Table constructor): Add ignore_len argument. Use it to
initialize ignore_length.
(Hash_Table destructor): Specify explicit length of char_set and
key.
(Hash_Table::insert): Renamed from Hash_Table::operator(). Remove
ignore_length argument. Pass explicit length to hashpjw. Compare
char_set using memcmp, not strcmp.
* src/list-node.h (List_Node): Rename field length to key_length.
New field char_set_length.
(List_Node constructor): Accept key and rest, not the entire line.
* src/list-node.cc (List_Node constructor): Accept key and rest, not
the entire line. Don't NUL terminate key and char_set. Specify
explicit length of key. Initialize char_set_length field.
* src/key-list.cc: Include <ctype.h>.
(parse_line): New function.
(Key_List::read_keys): Call parse_line instead of new List_Node.
Pass option[NOLENGTH] to Hash_Table constructor, not
Hash_Table::insert. Specify explicit length of key and char_set.
(Key_List::get_occurrence): Use explicit length of char_set.
(Key_List::set_determined): Likewise.
(Key_List::already_determined): Likewise.
(output_string): Add length argument. Output unprintable characters
using octal escape sequence.
(output_keyword_entry): Use explicit length of key.
(Key_List::output_lookup_array): Specify explicit length of key.
(output_switch_case): Likewise.
(Key_List::dump): Likewise.
* src/gen-perf.h (Gen_Perf::compute_disjoint_union): Add two length
arguments.
* src/gen-perf.cc (Gen_Perf::compute_disjoint_union): Likewise. Don't
stop when encountering NUL characters. Don't NUL terminate the
result.
(Gen_Perf::hash): Use explicit length of char_set.
(Gen_Perf::change): Specify explicit length of key.
* doc/gperf.texi: Document it.
* doc/help2man: New file, help2man version 1.022.
* Makefile.devel (all): Add doc/gperf.1.
(doc/gperf.1): New target.
* doc/gperf.1: Automatically generated.
* mkinstalldirs: New file, from automake-1.4, grep-2.4.1, guile-1.4,
libtool-1.3.3, make-3.79.1, tar-1.13.
* src/Makefile.in (MKINSTALLDIRS): New variable.
(install, installdirs): Use it instead of mkdir.
* doc/Makefile.in (MKINSTALLDIRS): New variable.
(install, installdirs): Use it instead of mkdir.
* INSTALL: Update.
2000-08-19 Bruno Haible <bruno@linuix.math.u-bordeaux.fr>
* src/key-list.cc (Output_Compare_Memcmp): New class.
(Key_List::output_lookup_function): When option -l is given, use
memcmp instead of strcmp or strncmp.
* doc/gperf.texi: The bug report address is <bug-gnu-utils@gnu.org>.
The download address is ftp.gnu.org. Remove mention of -a and -g
options (now nops). Explain effect of -c option.
* doc/configure.in (PACKAGE): New variable.
* doc/Makefile.in (datadir, docdir): New variables.
(dvidir, htmldir): Change values.
(install, installdirs): Update.
* src/configure.in: Rename cache variable gp_cxx_throw_decl to
gp_cv_cxx_throw_decl.
* src/key-list.cc (Key_List::output_hash_function): When outputting
__inline, take advantage of C++ compilers which have inline.
* src/key-list.cc (Output_Compare_Strncmp::output_comparison):
After the call to strncmp, verify that expr2 is not longer than
`len'.
Reported by Carlo Wood <carlo@runaway.xs4all.nl>.
* src/key-list.cc (Key_List::output_lookup_function_body): Avoid
emitting the loop for dealing with duplicates if
total_duplicates == 0.
* src/key-list.cc (Key_List::read_keys): Don't accept an empty key.
* src/Makefile.in (install, installdirs, uninstall): Respect
$(DESTDIR).
* doc/Makefile.in (install, installdirs, uninstall): Likewise.
* src/options.cc (Options::print_options): Escape the arguments which
contain special characters.
* tests/c-parse.gperf: Updated from gcc-2.95.2/gcc/c-parse.gperf.
* tests/objc.gperf: New file, from gcc-2.95.2/gcc/objc/objc.gperf.
* tests/chill.gperf: New file, from gcc-2.95.2/gcc/ch/gperf.
* tests/cplusplus.gperf: New file, from gcc-2.95.2/gcc/cp/gxx.gperf.
* tests/gplus.gperf: Remove file.
* tests/java.gperf: New file, from gcc-2.95.2/gcc/java/keyword.gperf.
* tests/Makefile: Check them all.
* tests/c-parse.exp: Renamed from tests/test-1.exp.
* tests/modula2.exp: Renamed from tests/test-2.exp.
* tests/cplusplus.exp: Renamed from tests/test-3.exp.
* tests/gpc.exp: Renamed from tests/test-5.exp.
* src/key-list.cc (output_switch_case): Add trailing semicolon to
lengthptr assignment line. Fixes bug with -D and -S.
From Reini Urban <rurban@sbox.tu-graz.ac.at>. Also reported by
David Hunter.
* tests/Makefile.in (check-lang-syntax): Perform each test with -D
once without and once with duplicates.
* src/key-list.cc (output_keyword_entry): Avoid outputting a struct
initializer of the form {"key",}.
* src/iterator.cc: Don't include <stream.h>.
From Michael Deutschmann <ldeutsch@mail.netshop.net>.
* tests/Makefile.in (VALIDATE, check-lang-syntax): Use $(srcdir) where
appropriate.
Reported by Andreas Schwab <schwab@issan.informatik.uni-dortmund.de>.
* tests/validate: Don't run -traditional tests by default.
* src/main.cc (main): Check for write error on stdout before returning.
* src/Makefile.in (LDFLAGS): New definition, to catch the value given
at configure time.
Make the structure initializers customizable. Based on a patch by
Kaveh R. Ghazi <ghazi@caip.rutgers.edu>.
* src/options.h (Options::get_initializer_suffix,
Options::initializer_suffix): New declarations.
* src/options.icc (Options::get_initializer_suffix): New function.
* src/options.cc (DEFAULT_INITIALIZER_SUFFIX): New constant.
(Options::initializer_suffix): New variable.
(Options::short_usage): Document option "-F".
(Options::long_usage): Document option "-F".
(Options constructor): Initialize initializer_suffix.
(Options destructor): Dump initializer_suffix.
(long_options): Add option "-F".
(Options::operator()): Accept option "-F". Sets initializer_suffix.
* src/key-list.cc (output_keyword_blank_entries): Output
initializer_suffix.
* doc/gperf.texi: Document option "-F".
* COPYING: Replace with GPL version 2 (with new FSF address and Y2K
safe year format).
* doc/gpl.texinfo: New file.
* doc/gperf.texi: Document it.
* doc/Makefile.in (gperf.info, gperf.dvi, gperf.html, gperf_toc.html):
Update dependencies.
* doc/Makefile.in (MAKEINFO): Unset LANG while running makeinfo.
1998-05-20 Bruno Haible <bruno@linuix.mathematik.uni-karlsruhe.de>
* doc/Makefile.in (gperf.dvi, clean): Remove gperf.cps.
(install, installdirs, uninstall): Don't install gperf.dvi. The
info and HTML documentations are sufficient for on-line use, and
users who wish to print the documentation (in PS or DVI format)
can do this directly off the source distribution.
(DVIPS): Use "-D600" instead of "-Pljfour", for portability.
1998-05-20 Akim Demaille <demaille@inf.enst.fr>
* doc/gperf.texi: Many modifications:
(Output Format): Declare `hash' and `in_word_set' as functions.
(Concept Index): New section.
(Title page): Use standard presentation.
(Top): Use @top instead of @unnumbered so that automatic master
update works.
(Motivation): Avoid spaces in @var.
(Options): Use the standard name ``Invoking ...''.
(Options): Declare also the long form of the options.
(Options): Remove redundant @itemize when @table is used.
1998-05-08 Andreas Schwab <schwab@issan.informatik.uni-dortmund.de>
* aclocal.m4 (CL_PROG_INSTALL): Set cl_cv_path_install, not
ac_cv_path_install.
Sat May 2 13:20:54 1998 Bruno Haible <bruno@linuix.mathematik.uni-karlsruhe.de>
* gperf-2.7 released.
Sat May 2 12:31:51 1998 Bruno Haible <bruno@linuix.mathematik.uni-karlsruhe.de>
* src/version.cc (version_string): Remove the "(C++ version)" suffix.
It's redundant: the early C versions of gperf are called cperf.
Reported by Karl Berry.
* src/option.cc (Options::operator()): Trim the output of "gperf -v".
Thu Apr 16 13:22:16 1998 Bruno Haible <bruno@linuix.mathematik.uni-karlsruhe.de>
* lib/Makefile.in, src/Makefile.in: Don't use $(TARGET_ARCH).
Solaris "make" sets it to a value not understood by "cc".
Wed Apr 15 23:52:14 1998 Bruno Haible <bruno@linuix.mathematik.uni-karlsruhe.de>
* lib/Makefile.in, src/Makefile.in: Don't use implicit rules. Don't
use $<. AIX "make" and OSF/1 "make" have problems with both.
* src/gen-perf.cc, src/key-list.cc: Cast free() argument to char*,
otherwise it doesn't compile on SunOS 4.
* src/key-list.h: Declare structs outside of other declarations,
needed for OSF/1 cxx 5.5.
* lib/getopt.h: Use prototypes if __STDC__ || __cplusplus.
Don't give a prototype for getopt(), to avoid error on SunOS 4.
* lib/getopt.c: Declare strncmp, to avoid warnings.
Tue Apr 14 23:24:07 1998 Bruno Haible <bruno@linuix.mathematik.uni-karlsruhe.de>
* lib/GetOpt.{h,cc}: Remove files.
* lib/getopt.{h,c}, lib/getopt1.c: New files, from GNU libc.
* lib/configure.in (AC_INIT): Search for hash.cc, not GetOpt.cc.
* lib/Makefile.in (OBJECTS): Remove GetOpt.o, add getopt.o, getopt1.o.
(getopt.o, getopt1.o, hash.o): Use explicit building rules. Some
"make"s don't support to have both implicit rules for "%.o : %.c"
and "%.o : %.cc" in the same Makefile.
* lib/hash.{h,cc}: Remove #pragma; there are no templates here.
* src/option.h (Options::usage): Remove.
(Options::short_usage, Options::long_usage): Declare.
* src/option.cc (Options::usage): Remove.
(Options::short_usage, Options::long_usage): New functions.
(long_options): New array.
(Options::operator()): Use getopt_long instead of GetOpt::operator(),
change all references to GetOpt members.
* src/std-err.{h,cc}: Remove files.
* src/gen-perf.cc, src/key-list.cc, list-node.cc, new.cc, options.cc:
Call fprintf(stderr) instead of Std_Err::report_error().
* src/key-list.h, src/list-node.h, src/options.h: Don't use class
Std_Err any more.
* src/option.cc (program_name): New variable.
* src/Makefile.in: Remove STD_ERR_H.
(OBJECTS): Remove std-err.o.
Mon Mar 23 01:03:35 1998 Bruno Haible <bruno@linuix.mathematik.uni-karlsruhe.de>
* aclocal.m4, {lib,src,tests}/configure.in: Remove CL_CC_WORKS and
CL_CXX_WORKS, already contained in autoconf 2.12.
* src/gen-perf.cc, src/key-list.cc: Move some code from
Gen_Perf::Gen_Perf() to Key_List::output().
* src/Makefile.in: Update dependencies.
* src/options.{h,cc}: Remove option "-p".
* src/key-list.cc (Key_List::set_output_types): Rewrite.
(default_array_type, default_return_type): Remove.
* src/key-list.cc: Adjust "const" handling.
+ With option "-t" [TYPE], don't emit wrong code if there is no
space before the struct's opening brace.
Sun Mar 22 16:59:15 1998 Bruno Haible <bruno@linuix.mathematik.uni-karlsruhe.de>
* src/key-list.{h,cc}: Completely reorganized the output routines.
Rewrote from scratch the output_switch() function. Use classes
(Output_Constants, Output_Expr, Output_Compare) for abstraction.
In particular:
+ Don't emit trailing whitespace and spurious blank lines.
+ Adjust indentation of the arrays.
+ Don't emit commas at the end of array initializers and
struct initializers.
+ With option "-l" [LENTABLE], compare the length before
fetching the word from memory.
+ With option "-S" [SWITCH], emit the comparison code just once,
not once in every switch statement.
+ With option "-S" [SWITCH], choose the right switch statement
through a binary search, not a linear search.
+ With option "-S" [SWITCH], emit straightforward comparisons
instead of switch statements with just one "case" label.
+ With options "-S -p -t" [SWITCH, POINTER, TYPE], don't emit
spurious empty elements at the beginning of the wordlist array.
+ With option "-D" [DUP] and not option "-S" [SWITCH], if there
is no more room for duplicate entries in the lookup array,
don't call `assert (i != 0)'. Instead, make the array larger :-)
+ With option "-D" [DUP], if there are no duplicates, don't
automatically fall back to the non-"-D" algorithm. If the user
wants the non-"-D" algorithm, he can just not specify "-D".
+ With option "-D" [DUP] and either options "-p -t" [POINTER, TYPE]
or not option "-S" [SWITCH], don't emit spurious empty elements
at the beginning of the wordlist array.
+ With option "-D" [DUP], simplify the detection and processing
of duplicate entries in the lookup array.
+ With options "-D -l" [DUP, LENTABLE] and not option "-S" [SWITCH],
don't forget to emit the lengthtable array.
+ With options "-D -l -S" [DUP, LENTABLE, SWITCH], don't forget to
compare the lengths before comparing the strings.
* src/gen-perf.cc: No need to include <assert.h>.
* src/options.cc: Likewise.
* src/options.cc: Don't use `errno' after freopen failed.
* src/std-err.cc: `report_error' doesn't call strerror(errno) any
more. No need to include <string.h> and <errno.h>.
* tests/Makefile.in (check-*): Any difference between .exp and .out
is a failure. Don't ignore whitespace differences.
* tests/Makefile.in (check-lang-syntax): Add some more checks.
Fri Mar 20 00:54:54 1998 Bruno Haible <bruno@linuix.mathematik.uni-karlsruhe.de>
* tests/jscript.gperf: Renamed from tests/javascript.gperf, because
of Minix and SVR2 14-character filename limit.
* src/key-list.cc (output_string): New function.
(Key_List::output_switch, Key_List::output_keyword_table): Call it.
* src/options.{h,icc,cc} (get_wordlist_name): New function. Add
option -W.
* src/key-list.cc (Key_List::output_switch,
Key_List::output_keyword_table, Key_List::output_lookup_function):
Use it.
Patch from William Bader <wbader@CSEE.Lehigh.Edu>.
* src/version.cc: Bump version number directly from 2.5 to 2.7,
because Schmidt's last release from 1991 carries version number 2.6.
Tue Jul 30 00:02:39 1991 Douglas C. Schmidt (schmidt at net4.ics.uci.edu)
* Fixed a small bug in the Key_List::output_keyword_table routine
that caused an extra newline to be printed if there where no
leading blank entries... (who cares, right?!)
Mon Jul 29 22:05:40 1991 Douglas C. Schmidt (schmidt at net4.ics.uci.edu)
* Modified the handling of the -E (emit enums rather than
#defines) option in conjunction with the -G option. Now, if -G
and -E are given the enums are generated outside the lookup
function, rather than within it!
Mon Apr 8 18:17:04 1991 Doug Schmidt (schmidt at net4.ics.uci.edu)
* Yucko, there was a bug in the handling of -c (and of course the
new -I command in key-list.cc). Apparently when I added the
super-duper hack that provided support for duplicate keys I
forgot to update the strcmp output...
Mon Mar 9 02:19:04 1998 Bruno Haible <bruno@linuix.mathematik.uni-karlsruhe.de>
* Moved the documentation to doc/, put the stuff borrowed from
libg++ into lib/.
* Rewrote all Makefile.in's for better compliance with GNU standards.
* Autoconf based configuration. Rewrote all configure.in's. Added
aclocal.m4, with macros from CLISP and CLN. Added Makefile.devel.
* src/depend: Removed. Dependencies are now in src/Makefile.in.
* src/bool-array.icc: New file, contains inline functions, from both
src/bool-array.h and src/bool-array.cc.
* src/options.icc: New file, contains inline functions, from both
src/options.h and src/options.cc.
* src/read-line.icc: New file, contains inline functions, from both
src/read-line.h and src/read-line.cc.
* src/bool-array.h: Don't include <std.h>.
* src/bool-array.cc: Include <string.h>.
* src/gen-perf.cc: No need to include <ctype.h>. Don't include
<_G_config.h>.
* src/hash-table.cc: Don't include <std.h> and <builtin.h>. Include
<string.h> and lib/hash.h instead.
* src/iterator.cc: Don't include <std.h>.
* src/key-list.cc: Don't include <builtin.h>. Include <string.h> and
<stdlib.h> instead.
* src/list-node.cc: Don't include <std.h>. Include <stdlib.h> instead.
Remove `index' hack.
* src/main.cc: Don't include <_G_config.h>.
* src/new.cc: Don't include <std.h>. Include <stdlib.h> instead.
* src/options.cc: Don't include <builtin.h>. Include <string.h> and
<stdlib.h> instead.
* src/read-line.cc: Don't include <builtin.h>. Include <stdlib.h>
instead.
* src/std-err.cc: Don't include <std.h>. Include <string.h> (for Irix).
* src/vectors.h: No need to include <stdio.h>.
* src/version.cc: No need to include <stdio.h>.
* src/bool-array.h: Change `STORAGE_TYPE' from int to unsigned int.
* src/bool-array.{h,cc}: Change type of `Bool_Array::size' from int
to unsigned int.
* src/bool-array.{h,cc}: Change type of `Bool_Array::init' argument
from STORAGE_TYPE to unsigned int.
* src/gen-perf.{h,cc}: Change two `Gen_Perf::compute_disjoint_union'
argument types from `char *' to `const char *'.
* src/iterator.h: Change type of `Iterator::str' and argument of
`Iterator::Iterator' from `char *' to `const char *'.
* src/iterator.cc: Cast to `unsigned char' before calling `isdigit'.
* src/key-list.{h,cc}: Change type of `Key_List::array_type',
`Key_List::return_type', `Key_List::struct_tag',
`Key_List::include_src', `default_array_type', `default_return_type'
and return type of `Key_List::get_array_type',
`Key_List::get_special_input', `Key_List::save_include_src' from
`char *' to `const char *'.
* src/key-list.cc: Change "pretty gross" assignment.
* src/key-list.cc: Don't use `alloca', HP-UX CC lacks it.
* lib/GetOpt.cc: Likewise.
* src/key-list.cc (merge): Use iteration instead of recursion.
* src/list-node.{h,cc}: Change type of `List_Node::key',
`List_Node::rest', `List_Node::char_set' from `char *' to
`const char *'.
* src/new.cc: Don't use BUFSIZ. Conditionalize the throw() declaration.
* src/read-line.h: Don't use BUFSIZ.
* src/read-line.cc: Make CHUNK_SIZE a constant, and use allocate the
buffers on the stack by default. Use memcpy for copying buffers.
Include <string.h>.
* src/read-line.icc (get_line): Use iteration instead of tail recursion.
Don't call ungetc(EOF,stdin).
* src/std-err.{h,cc}: Change type of `Std_Err::program_name' and of
argument `Std_Err::report_error' from `char *' to `const char *'.
* src/std-err.cc: `report_error' doesn't call `exit' any more. All
callers changed to do that themselves.
* src/trace.h: Make constructor/destructor calls non-inline.
* src/key-list.cc (output_hash_function): If option[CPLUSPLUS],
always make the hash function inline.
(output): Declare the hash function inline, with the right name.
* src/options.{h,cc}, src/gen-perf.cc, src/key-list.cc: Remove
options -g, making it on by default. Remove option -a. Instead,
introduce "-L KR-C", "-L C", "-L ANSI-C", "-L C++".
* src/options.{h,cc}, src/key-list.cc: Add option -I.
* src/key-list.cc: Don't emit "const" if compiling in mode "-L KR-C".
* src/key-list.cc: Don't emit a comma at the end of an enum list.
* src/main.cc: Remove COUNT_TIME code.
* src/vectors.h, src/key-list.cc, src/options.cc, src/list-node.cc:
ALPHA_SIZE defaults to 256 now. Add option -7.
* tests/javascript.gperf: New file.
* tests/jstest*.gperf, tests/validate: New tests.
Sat Jan 31 01:38:11 1998 Alexandre Oliva <oliva@dcc.unicamp.br>
* src/Makefile.in ($(TARGETPROG)): Add $(CFLAGS).
Wed Jan 28 01:56:00 1998 Manfred Hollstein <manfred@s-direktnet.de>
* configure.in (package_makefile_rules_frag): New and
redirect stderr from ${srcdir}/config.shared to
${package_makefile_rules_frag}.
* src/configure.in: Ditto.
* tests/configure.in: Ditto.
Fri Jan 23 08:00:41 1998 H.J. Lu (hjl@gnu.org)
* gperf.texi (@ichapter): Changed to @chapter.
Wed Jan 14 09:16:48 1998 H.J. Lu (hjl@gnu.org)
* src/key-list.cc, src/key-list.h (Key_List::strcspn): Don't
define if strcspn is defined.
Fri Jan 24 13:23:47 1997 Mike Stump <mrs@cygnus.com>
* src/new.cc (operator delete): Add the exception specification.
Mon Feb 5 19:29:16 1996 Per Bothner <bothner@kalessin.cygnus.com>
* src/read-line.cc (Read_Line::readln_aux): Handle EOF if last line
has a length which is an exact multiple of CHUNK_SIZE. (Used to throw
away the line's contents.) From Bruno Haible <haible@ilog.ilog.fr>.
* src/Makefile.in ($(TARGETPROG)): Add -lm to link line.
Tue Jun 11 13:43:50 1996 Brendan Kehoe <brendan@lisa.cygnus.com>
* src/list-node.cc (List_Node): Reorder init of nodes to
match declaration order.
* src/hash-table.cc (Hash_Table): Likewise.
Tue Oct 10 16:37:28 1995 Mike Stump <mrs@cygnus.com>
* src/new.cc: Since malloc/delete are not paired, we cannot call
free.
Wed Jan 4 12:40:14 1995 Per Bothner <bothner@kalessin.cygnus.com>
* src/Makefile.in ($(TARGETPROG)): Link with $(LDFLAGS).
Patch from John Interrante <interran@uluru.stanford.edu>.
Sat Nov 5 19:12:48 1994 Jason Merrill (jason@phydeaux.cygnus.com)
* src/Makefile.in (LIBS): Remove.
Tue Oct 18 17:51:14 1994 Per Bothner <bothner@kalessin.cygnus.com>
* src/std-err.cc: Use stderror, instead of the non-standard
sys_nerr and sys_errlist.
Sat Sep 17 22:02:13 1994 Per Bothner (bothner@kalessin.cygnus.com)
* src/key-list.cc (output_hash_function):
Patch from William Bader <wbader@CSEE.Lehigh.Edu>.
Fri Jul 15 09:38:11 1994 Per Bothner (bothner@cygnus.com)
* src/std-err.cc: #include <errno.h>, and only declare
extern int errno if errno is not a macro.
Mon May 30 17:29:34 1994 Per Bothner (bothner@kalessin.cygnus.com)
* Makefile.in (src_all, install): Make sure to add '/' after
`pwd` in $rootme, as expected by FLAGS_TO_PASS.
Wed May 11 00:47:22 1994 Jason Merrill (jason@deneb.cygnus.com)
Make libg++ build with gcc -ansi -pedantic-errors
* src/options.h: Lose commas at end of enumerator lists.
Sun Dec 5 19:16:40 1993 Brendan Kehoe (brendan@lisa.cygnus.com)
* src/hash-table.cc (Hash_Table::~Hash_Table): Don't pass an
argument to fprintf, since it's not expecting one.
Fri Nov 26 19:03:18 1993 Per Bothner (bothner@kalessin.cygnus.com)
* src/list-node.cc: #undef index, for the sake of broken NeXT,
Thu Nov 4 11:16:03 1993 Per Bothner (bothner@kalessin.cygnus.com)
* Makefile.in (install): Use INSTALL_DATA for gperf.1.
Mon Oct 25 18:40:51 1993 Per Bothner (bothner@kalessin.cygnus.com)
* src/key-list.cc (Key_List::read_keys): Use POW macro
to increase hash table size to power of 2.
* options.h (LARGE_STACK_ARRAYS): New flag. Defaults to zero.
* gen-perf.cc, key-list.cc, read-line.cc:
Only stack-allocate large arrays if LARGE_STACK_ARRAYS is set.
* main.cc (main): Only call setrlimit (RLIMIT_STACK, ...)
if LARGE_STACK_ARRAYS.
Mon Oct 4 17:45:08 1993 Per Bothner (bothner@kalessin.cygnus.com)
* src/gen-perf.cc: Always use ANSI rand/srand instead of BSDisms.
Wed Aug 18 12:19:53 1993 Per Bothner (bothner@kalessin.cygnus.com)
* Makefile.in (src_all): Make less verbose output.
Fri May 28 14:01:18 1993 Per Bothner (bothner@rtl.cygnus.com)
* src/gen-perf.cc (Gen_Perf::change): Don't use gcc-specific
2-operand conditional expression.
* src/key-list.cc (Key_List::output_lookup_array):
Don't use variable-size stack arrays, unless compiled by g++.
Tue May 4 14:08:44 1993 Per Bothner (bothner@cygnus.com)
Changes (mostly from Peter Schauer) to permit compilation
using cfront 3.0 and otherwise be ARM-conforming.
* src/key-list.h: class Key_List must use public derivation
of base class Std_Err (because Gen_Perf::operator() in gen-perf.cc
calls Std_Err::report_error).
* src/gen-perf.cc (Gen_Perf::affects_prev), src/hash-table.cc
(Hash_Table::operator()): Don't use gcc-specific 2-operand
conditional expression.
* src/iterator.cc (Iterator::operator()): Don't use gcc-specific
range construct in case label.
* key-list.cc (Key_List::output_lookup_array, Key_List::read_keys),
src/gen-perf.cc (Gen_Perf::operator(), src/read-line.cc
(Read_Line::readln_aux): If not gcc, don't allocate
variable-sized arrays on stack.
* src/new.cc (operator new): Argument type should be size_t.
* key-list.cc (Key_List::output_lookup_array, Key_List::read_keys),
new/cc (::operator new): Don't use non-standard >?= operator.
Tue Apr 27 20:11:30 1993 Per Bothner (bothner@cygnus.com)
* src/Makefile.in: Define TARGETPROG, and use it.
Mon Apr 19 00:29:18 1993 Per Bothner (bothner@cygnus.com)
* Makefile.in, configure.in: Re-vamped configure scheme.
* gperf.texinfo: Renamed to gperf.texi.
* src/bool-array.{h,cc}: ANSIfy bzero->memset.
Sat Jan 30 20:21:28 1993 Brendan Kehoe (brendan@lisa.cygnus.com)
* tests/Makefile.in (mostlyclean): Also delete aout, cout, m3out,
pout, and preout.
Tue Dec 29 08:58:17 1992 Ian Lance Taylor (ian@cygnus.com)
* Makefile.in: pass $(FLAGS_TO_PASS) to all calls to make.
(FLAGS_TO_PASS): added INSTALL, INSTALL_DATA, INSTALL_PROGRAM.
Mon Dec 21 18:46:46 1992 Per Bothner (bothner@rtl.cygnus.com)
* tests/expected.* renamed to *.exp to fit in 14 chars.
* tests/Makefile.in: Update accordingly.
Also rename output.* to *.out.
* src/Makefile.in (clean): Remove gperf program.
Wed Dec 9 14:33:34 1992 Per Bothner (bothner@cygnus.com)
* src/hash-table.cc, src/bool-array.h: ANSIfy bzero->memset.
Thu Dec 3 19:34:12 1992 Per Bothner (bothner@cygnus.com)
* Makefile.in (distclean, realclean): Don't delete
Makefile before recursing.
Fri Nov 6 13:41:49 1992 Per Bothner (bothner@rtl.cygnus.com)
* key-list.{h,cc}: Remove MAX_INT (and similar) constant
fields from Key_List class, and use INT_MAX (etc) from limits.h.
* key-list.{h,cc}, options.{h,cc}, vectors.h: Removed all
uses of initialized const fields, as they are non-standard
- and their use was easy to do away with. Mostly, just
made the constants static non-fields in the .cc file.
Mon Nov 2 13:10:11 1992 Per Bothner (bothner@cygnus.com)
* tests/Makefile.in: When generating cinset.c, don't pass -C,
since -C assumes an ANSI compiler. Add the -C flag (with -a)
when generating test.out.3 instead.
* tests/expected.out.3: Update accordingly.
Wed Aug 12 11:47:54 1992 Per Bothner (bothner@cygnus.com)
* Makefile.in: Factor out common flags into $(FLAGS_TO_PASS).
* Makefile.in: 'install-info' depends on gperf.info.
Mon Aug 10 11:39:52 1992 Ian Lance Taylor (ian@dumbest.cygnus.com)
* Makefile.in, src/Makefile.in: always create installation
directories.
Mon Jul 20 15:33:21 1992 Mike Stump (mrs@cygnus.com)
* src/new.cc (operator new): Add cast from void * to char *,
since it is not a standard conversion.
Wed Jun 17 16:25:30 1992 Per Bothner (bothner@rtl.cygnus.com)
* src/gen-perf.cc: #include <_G_config.h> for _G_SYSV.
* src/key-list.cc: alloca() hair.
* src/main.cc (main): Only call getrlimit if _G_HAVE_SYS_RESOURCE.
* Makefile,in, {src,test}/Makefile.in: Fix *clean rules.
Fri May 29 13:21:13 1992 Per Bothner (bothner@rtl.cygnus.com)
* src/gen-perf.cc: Replace USG -> _G_SYSV.
Thu May 14 13:58:36 1992 Per Bothner (bothner@rtl.cygnus.com)
* src/Makefile.in: Don't pass obsolete flag -DUNLIMIT_STACK.
* tests/Makefile.in (clean): Fix.
Sat Mar 7 00:03:56 1992 K. Richard Pixley (rich@rtl.cygnus.com)
* gperf.texinfo: added menu item hook.
Wed Feb 26 18:04:40 1992 K. Richard Pixley (rich@cygnus.com)
* Makefile.in, configure.in: removed traces of namesubdir,
-subdirs, $(subdir), $(unsubdir), some rcs triggers. Forced
copyrights to '92, changed some from Cygnus to FSF.
Sun Jan 26 19:21:58 1992 Per Bothner (bothner at cygnus.com)
* tests/Makefile.in: Use re-directed stdin instead of file
name in argv. This allows us to remove the filename
from the output, the expected output, and hence the diffs.
(Note that the input file is in $(srcdir), which we cannot
place in the expected out files.)
* tests/expected.out.[1235]: Edit out input filename,
to match new output.
Thu Jun 28 16:17:27 1990 Doug Schmidt (schmidt at brilliant)
* Wow, first fix on the new job! There was a dumb error
in Key_List::output_lookup_function, where I printed the
string "&wordlist[key]" instead of the correct "&wordlist[index]".
* Added a couple of #ifdefs for USG support.
Sun Jun 3 17:16:36 1990 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Updated the version number to 2.5 and sent to Doug Lea for release
with the latest GNU libg++.
* Changed the error handling when a keyword file cannot be opened
(now calls perror).
Wed May 30 14:49:40 1990 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Instrumented the source code with trace statements automagically
inserted using my new automated trace instrumentation tool!
Wed May 9 11:47:41 1990 Doug Schmidt (schmidt at siam.ics.uci.edu)
* Really fixed the previous bug. Turns out that a small amount
of logic had to be duplicated to handle static links that occur
as part of dynamic link chains. What a pain!!!
Tue May 8 23:11:44 1990 Doug Schmidt (schmidt at siam.ics.uci.edu)
* Fixed a stupid bug in Key_List::output_lookup_array that was
causing incorrect counts to be generated when there were both
static and dynamic links occurring for the same hash value.
Also simplified the code that performs the logic in this routine.
Mon Apr 30 17:37:24 1990 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Fixed stupid bug in Key_List::output_lookup_array that was
making the generated lookup[] array contain `chars' even
when the values stored in the chars are greater than 127!
* Changed the behavior of the -G (global table) option so that it
will output the `length[]' array in the global scope along with
the `word_list[]' array.
* Fixed a stupid bug in Key_List::output_lookup_function that
would always output the complicated `duplicate-handling' lookup
logic, even when there were no duplicates in the input!
* Yikes, had to modify a bunch of stuff in key-list.cc to correctly
handle duplicate entries. Changed the generated code so that
the MIN_HASH_VALUE is no longer subtracted off when calculating
the hash value for a keyword. This required changing some other
code by substituting MAX_HASH_VALUE for TOTAL_KEYS in several places.
Finally, this means that the generated tables may contain leading
null entries, but I suppose it is better to trade-off space to get
faster performance...
Mon Mar 26 13:08:43 1990 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Updated version number to 2.4 to reflect the latest changes.
* Changed the main program so that it always prints out gperf's
execution timings to the generated output file.
Sun Mar 25 12:39:30 1990 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Added the -Z option so that users can specify the name of the
generated class explicitly. Updated documentation to reflect
this change.
* Modified the generated C++ class interface so that the functions
are declared static (to remove the overhead of passing the `this'
pointer). This means that operator()() can no longer be used,
since it only works on non-static member functions.
Also changed things so that there is no constructor (why waste
the extra call, when it doesn't do anything, eh?)
* Modified the behavior of Key_List::output when the -L C++ option
is enabled. Previously the code generated use const data members
to record MIN_WORD_LENGTH, MIN_HASH_VALUE, etc. However, as
pointed out by James Clark this may result in suboptimal behavior
on the part of C++ compilers that can't inline these values.
Therefore, the new behavior is identical to what happens with
-L C, i.e., either #defines or function-specific enums are used.
Why sacrifice speed for some abstract notion of `code purity?' ;-)
Tue Mar 6 18:17:42 1990 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Added the -E option that defines constant values using an enum
local to the lookup function rather than with #defines. This
also means that different lookup functions can reside in the
same file. Thanks to James Clark (jjc@ai.mit.edu).
Sat Mar 3 20:19:00 1990 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Added a special case to key_list::output_switch that doesn't
generate extra comparisons when the `-S' is given an argument
of 1 (the normal case). This should speed up the generated
code output a tad...
Fri Feb 23 14:21:28 1990 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Renamed all instances of member function get_keysig_size
to get_max_keysig_size, since this is more precise...
* Changed all occurrences of charset to keysig (stands for ``key
signature'') to reflect the new naming convention used in the
USENIX paper.
Thu Feb 22 11:28:36 1990 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Changed the name of the generated associated values table from
asso_value to asso_values to reflect conventions in the USENIX
C++ paper.
Thu Feb 15 23:29:03 1990 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Updated the gperf.texinfo file to fix some formatting problems
that had crept in since last time.
Wed Feb 14 23:27:24 1990 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Fixed stupid bug in key-list.cc (get_special_input), wher
gperf replaced each '%' with the succeeding character.
* Added support for multiple target language generation. Currently
handled languages are C and C++, with C as the default. Updated
documentation and option handler to reflect the changes.
* Added a global destructor to new.cc and removed the #ifdef, since
the bloody thing now works with libg++.
Mon Feb 14 13:00:00 1990 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Found out that my gperf paper was accepted at the upcoming
USENIX C++ Conference in San Francisco. Yow!
Tue Jan 30 09:00:29 1990 Doug Schmidt (schmidt at zola.ics.uci.edu)
* #ifdef'd out the new.cc memory allocator, since there are
problems with this and the libg++ stuff.
* Changed key-list.h so that class Vectors is a public (rather
than private) base class for class Key_List. The previous
form was illegal C++, but wasn't being caught by the old
g++ compiler. Should work now... ;-)
Sun Dec 10 14:08:23 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Added several changes from rfg@ics.uci.edu. These changes
help to automate the build process.
Wed Nov 15 15:49:33 1989 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Removed conditional compilation for GATHER_STATISTICS. There's
really no good reason to avoid collecting this info at run-time,
since that section of code is *hardly* the bottleneck... ;-)
* Simplified the C output routines in Key_List::set_output_types
and Key_List::output_keyword_table a bit in order to
speed-up and clean up the code generation.
* Modified function Key_List::get_special_input so that it does
not try to `delete' a buffer that turned out to be too short.
This is important since the new memory management scheme
does not handle deletions. However, adding a small amount of
garbage won't hurt anything, since we generally don't do this
operation more than a couple times *at most*!
* Created a new file (new.cc) which includes my own overloaded
operator new. This function should dramatically reduce the
number of calls to malloc since it grabs large chunks and
doles them out in small pieces. As a result of this change
the class-specific `operator new' was removed from class List_Node.
Tue Nov 14 21:45:30 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Continued to refine the great hack. The latest trick is to
try and replace most uses of dynamic memory (i.e., calls to
new) with uses of gcc dynamic arrays (i.e., an alloca solution).
This makes life much easier for the overall process-size, since
it reduces the amount of overhead for memory management. As a
side-effect from this change there is no reason to have the
Bool_Array::dispose member function, so it's outta here!
* Fixed a stupid bug that was an disaster waiting to happen...
Instead of making the boolean array large enough to index
max_hash_value it was only large enough to index max_hash_value
- 1. Once again, an off-by-one mistake in C/C++!!!!
Mon Nov 13 19:38:27 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Added the final great hack! This allows us to generate hash tables
for near-perfect hash functions that contain duplicates, *without*
having to use switch statements! Since many compilers die on large
switch statements this feature is essential. Furthermore, it appears
that the generated code is often *smaller* than that put out by
compilers, even though a large, sparse array must be created.
Here's the general idea:
a. Generate the wordlist as a contiguous block of keywords,
just as before when using a switch statement. This
wordlist *must* be sorted by hash value.
b. Generate the lookup array, which is an array of signed
{chars,shorts,ints}, (which ever allows full coverage of
the wordlist dimensions). If the value v, where v =
lookup[hash(str,len)], is >= 0 and < TOTAL_KEYWORDS, then we
simply use this result as a direct access into the wordlist
array to snag the keyword for comparison.
c. Otherwise, if v is < -TOTAL_KEYWORDS or > TOTAL_KEYWORDS
this is an indication that we'll need to search through
some number of duplicates hash values. Using a hash
linking scheme we'd then index into a different part of
the hash table that provides the starting index and total
length of the duplicate entries to find via linear search!
Sun Nov 12 13:48:10 1989 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Simplified Key_List::output_min_max considerably by recognizing
that since the keyword list was already sorted by hash value finding
the min and max values is trivial!
* Improved the debugging diagnostics considerably in classes Key_List,
Hash_Table, and Gen_Perf.
* Modified the `-s' option so that a negative argument is now
interpreted to mean `allow the maximum associated value to be
about x times *smaller* than the number of input keys.' This
should help prevent massive explosion of generated hash table
size for large keysets.
Sat Nov 11 11:31:13 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Added a field in class Key_List that counts the total number
of duplicate keywords, both static and dynamic.
* Added a new member function Bool_Array that deletes the dynamic
memory allocated to Bool_Array::storage_array. This space may
be needed for subsequent options, so it made sense to free it as
soon as possible...
* Renamed file/class Alpha_Vectors to Vectors, to avoid problems
with 14 character length filenames on SYSV. Also changed file
adapredefined.gperf to adadefs.gperf in the ./tests directory.
* Modified class Options by changing the member function
Options::total_positions to Options::get_charset_size and
Options::set_charset_size. These two routines now either return
the total charset size *or* the length of the largest keyword
if the user specifies the -k'*' (ALLCHARS) option. This change
cleans up client code.
* Merged all the cperf changes into gperf.
* Made sure to explicitly initialize perfect.fewest_collisions to
0.
* Cleaned up some loose ends noticed by Nels Olson.
1. Removed `if (collisions <= perfect.fewest_collisions)'
from Gen_Perf::affects_prev since it was superfluous.
2. Removed the fields best_char_value and best_asso_value
from Gen_Perf. There were also unnecessary.
3. Fixed a braino in the Bool_Array::bool_array_reset
function. Since iteration numbers can never be zero
the `if (bool_array.iteration_number++ == 0)' must be
`if (++bool_array.iteration_number == 0).'
4. Modified Std_Err::report_error so that it correctly handles
"%%".
* It is important to note that -D no longer enables -S.
There is a good reason for this change, which will become
manifested in the next release... (suspense!).
* Made some subtle changes to Key_List::print_switch so that if finally
seems to work correctly. Needs more stress testing, however...
* Made a major change to the Key_List::print_switch function.
The user can now specify the number of switch statements to generate
via an argument to the -S option, i.e., -S1 means `generate 1
switch statement with all keywords in it,' -S2 means generate
2 switch statements with 1/2 the elements in each one, etc.
Hopefully this will fix the problem with C compilers not being
able to generate code for giant switch statements (but don't
hold your breath!)
* Changed Key_List::length function to Key_List::keyword_list_length.
* Added a feature to main.c that prints out the starting wall-clock
time before the program begins and prints out the ending wall-clock
time when the program is finished.
* Added the GATHER_STATISTICS code in hash-table.c so we can
keep track of how well double hashing is doing. Eventually,
GATHER_STATISTICS will be added so that all instrumentation
code can be conditionally compiled in.
* Fixed a stupid bug in Key_List::print_switch routine. This
was necessary to make sure the generated switch statement worked
correctly when *both* `natural,' i.e., static links and dynamic
links, i.e., unresolved duplicates, hash to the same value.
* Modified Bool_Array::~Bool_Array destructor so that
it now frees the bool_array.storage_array when it is no longer
needed. Since this array is generally very large it makes sense
to return the memory to the freelist when it is no longer in use.
* Changed the interface to constructor Hash_Table::Hash_Table. This
constructor now passed a pointer to a power-of-two sized buffer that
serve as storage for the hash table. Although this weakens information
hiding a little bit it greatly reduces dynamic memory fragmentation,
since we can now obtain the memory via a call to alloca, rather
than malloc. This change modified Key_List::read_keys calling
interface.
* Since alloca is now being used more aggressively a conditional
compilation section was added in main.c. Taken from GNU GCC,
this code gets rid of any avoidable limit on stack size so that
alloca does not fail. It is only used if the -DRLIMIT_STACK
symbol is defined when gperf is compiled.
* Added warnings in option.c so that user's would be informed
that -r superceeds -i on the command-line.
* Rewrote Gen_Perf::affects_prev. First, the code structure
was cleaned up considerably (removing the need for a dreaded
goto!). Secondly, a major change occurred so that Gen_Perf::affects_prev
returns FALSE (success) when fewest_hits gets down to whatever
it was after inserting the previous key (instead of waiting for
it to reach 0). In other words, it stops trying if it can
resolve the new collisions added by a key, even if there are
still other old, unresolved collisions. This modification was
suggested by Nels Olson and seems to *greatly* increase the
speed of gperf for large keyfiles. Thanks Nels!
* In a similar vein, inside the Gen_Perf::change routine
the variable `perfect.fewest_collisions is no longer initialized
with the length of the keyword list. Instead it starts out at
0 and is incremented by 1 every time change () is called.
The rationale for this behavior is that there are times when a
collision causes the number of duplicates (collisions) to
increase by a large amount when it would presumably just have
gone up by 1 if none of the asso_values were changed. That is,
at the beginning of change(), you could initialize fewest_hits
to 1+(previous value of fewest_hits) instead of to the number of
keys. Thanks again, Nels.
* Replaced alloca with new in the Gen_Perf::change function.
This should eliminate some overhead at the expense of a little
extra memory that is never reclaimed.
* Renamed Gen_Perf::merge_sets to Gen_Perf::compute_disjoint_union
to reflect the change in behavior.
* Added the -e option so users can supply a string containing
the characters used to separate keywords from their attributes.
The default behavior is ",\n".
* Removed the char *uniq_set field from LIST_NODE and modified
uses of uniq_set in perfect.c and keylist.c. Due to changes
to Gen_Perf::compute_disjoint_sets this field was no longer
necessary, and its removal makes the program smaller and
potentially faster.
* Added lots of changes/fixes suggested by Nels Olson
(umls.UUCP!olson@mis.ucsf.edu). In particular:
1. Changed Bool_Array so that it would dynamically create
an array of unsigned shorts rather than ints if the
LO_CAL symbol was defined during program compilation.
This cuts the amount of dynamic memory usage in half,
which is important for large keyfile input.
2. Added some additional debugging statements that print extra
info to stderr when the -d option is enabled.
3. Fixed a really stupid bug in Key_List::print_switch
A right paren was placed at the wrong location, which broke
strlen ().
4. Fixed a subtle problem with printing case values when keylinks
appear. The logic failed to account for the fact that there
can be keylinks *and* regular node info also!
5. Changed the behavior of Key_List::read_keys so that it would
honor -D unequivocally, i.e., it doesn't try to turn off dup
handling if the user requests it, even if there are no
immediate links in the keyfile input.
6. Modified the -j option so that -j 0 means `try random values
when searching for a way to resolve collisions.'
7. Added a field `num_done' to the Gen_Perf struct. This is used
to report information collected when trying to resolve
hash collisions.
8. Modified the merge_sets algorithm to perform a disjoint
union of two multisets. This ensures that subsequent
processing in Gen_Perf::affect_prev doesn't
waste time trying to change an associated value that is
shared between two conflicting keywords.
9. Modified Gen_Perf::affects_prev so that it doesn't try
random jump values unless the -j 0 option is enabled.
10. Fixed a silly bug in Gen_Perf::change. This problem caused
gperf to seg fault when the -k* option was given and the
keyfile file had long keywords.
Sun Oct 29 00:18:55 1989 Doug Schmidt (schmidt at siam.ics.uci.edu)
* Modified class-specific new operations for Read_Line and
List_Node so they don't fail if SIZE is larger than twice
the previous buffer size. Note we double buffer size
everytime the previous buffer runs out, as a heuristic
to reduce future calls to malloc.
Sun Oct 22 13:49:43 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Updated gperf version number to 2.0. Send to Doug Lea for
incorporation into the long-awaited `official' libg++ 1.36
release!
* Thanks to Nels Olson a silly bug in Gen_Perf::change ()
was fixed. This problem caused gperf to seg fault when
the -k* option was given and the keyfile file had long
keywords.
* Modified Key_List::print_hash_function so that it output
max_hash_value + 1 (rather than just max_hash_value) for
any associated value entries that don't correspond to
keyword charset characters. This should speed up rejection
of non-keyword strings a little in some cases.
Sat Oct 21 19:28:36 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Fixed Key_List::print_hash_function so that it no longer output
things like `return 0 + ...' Although this probably gets
optimized away by even the worst C compilers there isn't any
point tempting fate... ;-)
* Fixed class List_Node's constructor so that it wouldn't a priori
refuse to consider trying to hash keys whose length is less
than the smallest user-specified key position. It turns out
this is not a problem unless the user also specifies the -n
(NOLENGTH) option, in which case such keys most likely
don't have a prayer of being hashed correctly!
* Changed the name of the generated lookup table from `Hash_Table'
to `asso_value' to be consistent with the gperf paper.
Tue Oct 17 14:19:48 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Added a flag GATHER_STATISTICS in the Makefile. If defined
during compilation this turns on certain collection facilities
that track the performance of gperf during its execution. In
particular, I want to see how many collisions occur for the
double hashing Hash_Table.
* Added a safety check so that we don't screw up if the total
number of `resets' of the Bool_Array exceeds MAX_INT. Since
this number is around 2^31 it is unlikely that this would ever
occur for most input, but why take the risk?
* Changed the behavior for the -a (ANSI) option so that the
generated prototypes use int rather than size_t for the LEN
parameter. It was too ugly having to #include <stddef.h> all
over the place...
Mon Oct 16 11:00:35 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Continued to work on the gperf paper for the USENIX C++
conference. At some point this will be merged back into
the gperf documentation...
Sat Oct 14 20:29:43 1989 Doug Schmidt (schmidt at siam.ics.uci.edu)
* Added a majorly neat hack to Bool_Array, suggested by rfg.
The basic idea was to throw away the Ullman array technique.
The Ullman array was used to remove the need to reinitialize all
the Bool_Array elements to zero everytime we needed to determine
whether there were duplicate hash values in the keyword list.
The current trick uses an `iteration number' scheme, which takes
about 1/3 the space and reduces the overall program running a
time by about 20 percent for large input! The hack works as
follows:
1. Dynamically allocation 1 boolean array of size k.
2. Initialize the boolean array to zeros, and consider the first
iteration to be iteration 1.
2. Then on all subsequent iterations we `reset' the bool array by
kicking the iteration count by 1.
3. When it comes time to check whether a hash value is currently
in the boolean array we simply check its index location. If
the value stored there is *not* equal to the current iteration
number then the item is clearly *not* in the set. In that
case we assign the iteration number to that array's index
location for future reference. Otherwise, if the item at
the index location *is* equal to the iteration number we've
found a duplicate. No muss, no fuss!
Mon Oct 2 12:30:54 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Changed some consts in options.h to enumerals, since g++
doesn't seem to like them at the moment!
Sat Sep 30 12:55:24 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Fixed a stupid bug in Key_List::print_hash_function that manifested
itself if the `-k$' option was given (i.e., only use the key[length]
character in the hash function).
* Added support for the -C option. This makes the contents of
all generated tables `readonly'.
* Changed the handling of generated switches so that there is
only one call to str[n]?cmp. This *greatly* reduces the size of
the generated assembly code on all compilers I've seen.
* Fixed a subtle bug that occurred when the -l and -S option
was given. Code produced looked something like:
if (len != key_len || !strcmp (s1, resword->name)) return resword;
which doesn't make any sense. Clearly, this should be:
if (len == key_len && !strcmp (s1, resword->name)) return resword;
Tue Sep 26 10:36:50 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Changed class Read_Line's definition so that it no longer
needs to know about the buffering scheme used to speed up
dynamic memory allocation of input keywords and their
associated attributes. This means that operator new is no longer
a friend of Read_Line.
Mon Sep 25 23:17:10 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Decided that Obstacks had too much overhead, so they were
removed in favor of super-efficient, low-overhead buffered
storage allocation hacks in Read_Line and List_Node.
* No longer try to inline functions that g++ complains about
(Key_List::Merge and Key_List::Merge_Sort).
Sun Sep 24 13:11:24 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Changed classes Read_Line and List_Node to use Obstacks in order
to cache memory allocation for keyword strings and List_Nodes.
* Continued to experiment with inheritance schemes.
* Added a new file `alpha.h', that declares static data shared
(i.e., inherited) between classes List_Node and Key_List.
Tue Sep 12 16:14:41 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Made numerous changes to incorporate multiple inheritance in
gperf.
Wed Aug 16 23:04:08 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Added the -DCOMPILER_FIXED flag to the ./src/Makefile. This
implies that people trying to compile gperf need to have a
working version of the new g++ compiler (1.36.0).
* Removed some extra spaces that were being added in the generated
C code.
Mon Jul 24 17:09:46 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Fixed PRINT_HASH_FUNCTION and PRINT_LOOKUP_FUNCTION in keylist.c
so that the generated functions take an unsigned int length argument.
If -a is enabled the prototype is (const char *str, size_t len).
Fri Jul 21 13:06:15 1989 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Fixed a typo in PRINT_KEYWORD_TABLE in keylist.cc that prevented
the indentation from working correctly.
* Fixed a horrible typo in PRINT_KEYWORD_TABLE in keylist.cc
that prevented links from being printed correctly.
Tue Jul 18 16:04:31 1989 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Fixed up readline.cc and readline.h so that they work OK
with g++ compilers that aren't completely up-to-date.
If symbol COMPILER_FIXED is defined then the behavior
that works on my more recent version of g++ is enabled.
Sun Jul 9 17:53:28 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Changed the ./tests subdirectory Makefile so that it
uses $(CC) instead of gcc.
Sun Jul 2 21:52:15 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Fixed a number of subtle bugs that occurred when -S was
combined with various and sundry options.
* Added the -G option, that makes the generated keyword table
a global static variable, rather than hiding it inside
the lookup function. This allows other functions to directly
access the contents in this table.
* Added the "#" feature, that allows comments inside the keyword
list from the input file. Comment handling takes place in readline.c.
This simplifies the code and reduces the number of malloc calls.
* Also added the -H option (user can give the name of the hash
function) and the -T option (prevents the transfer of the type decl
to the output file, which is useful if the type is already defined
elsewhere).
Thu Jun 22 20:39:39 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Modified many classes so that they would inherit Std_Err as
a base class. This makes things more abstract...
Fri Jun 16 14:23:00 1989 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Modified the -f (FAST) option. This now takes an argument.
The argument corresponds to the number of iterations used
to resolve collisions. -f 0 uses the length of the
keyword list (which is what -f did before). This makes
life much easier when dealing with large keyword files.
Tue Jun 6 17:53:27 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Added the -c (comparison) option. Enabling this
will use the strncmp function for string comparisons.
The default is to use strcmp.
* Fixed a typo in key_list.cc (PRINT_SWITCH). This caused
faulty C code to be generated when the -D, -p, and -t
options were all enabled.
Thu May 25 14:07:21 1989 Doug Schmidt (schmidt at siam.ics.uci.edu)
* Once again, changed class Read_Line to overload global operator
new. Hopefully, this will work...!
Sun May 21 01:51:45 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Modified Key_List::print_hash_function () so that it properly
formats the associated values in the hash table according to
the maximum number of digits required to represent the largest
value.
* Removed the named return value from class Hash_Table's
operator (), since this causes a seg fault when -O is enabled.
No sense tripping subtle g++ bugs if we don't have to.... ;-)
* Removed the operator new hack from Read_Line, since this seemed
to create horrible bus error problems.
* Changed many class member functions and data members to be `static',
if they don't manipulate this!
Fri May 12 23:06:56 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Changed class Std_Err to use static member functions, a la
Ada or Modula 2. This eliminates the need for an explicit
error-handler class object.
* Added the ``named return value'' feature to Hash_Table::operator ()
and Bool_Array::operator [], just for the heck of it.... ;-)
* Changed the previous hack in Read_Line so that we now use
the overloaded global `new' instead of NEW_STRING!
Wed May 3 17:36:55 1989 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Updated to version 1.7. This reflects the recent major changes
and the new C port.
* Modified the GNU getopt.cc routine to have a class-based interface.
* Fixed a typo in Perfect.cc ~Perfect that prevented the actual maximum
hash table size from being printed (maybe the stream classes
weren't so bad after all.... ;-).
* Added support for the -f option. This generates the perfect
hash function ``fast.'' It reduces the execution time of
gperf, at the cost of minimizing the range of hash values.
Tue May 2 16:23:29 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Added an efficiency hack to Read_Line. Instead of making
a call to operator NEW (a.k.a. malloc) for each input string
a new member function NEW_STRING stores a large buffer from
which new strings are carved out, growing the buffer if
necessary. It might be useful to add this throughout the
program....
* Removed all unnecessary calls to DELETE. If the program is about
to exit it is silly to waste time freeing memory.
* Added the GNU getopt program to the distribution. This makes
GPERF portable to systems that don't include getopt in libc.
* Added a strcspn member to class Key_List. This also increases
portability.
* Added the get_include_src function from keylist.c as a member
function in class Key_List. Hopefully every function is
now associated with a class. This aids abstraction and
modularity.
* Ported gperf to C. From now on both K&R C and GNU G++ versions
will be supported. There will be two ChangeLog files, one
for each version of the program.
Mon May 1 16:41:45 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Fixed a bug with -k'*'. This now prints out *all* the cases
up to the length of the longest word in the keyword set.
Sun Apr 30 12:15:25 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Removed all use of the stream classes. Too ugly, slow, and
not handled by the c++-mode formatter....
* Modified the handling of links (i.e., keywords that have
identical hash values as other keywords). This should
speed up hash function generation for keyword sets with
many duplicate entries. The trick is to treat duplicate
values as equivalence classes, so that each set of duplicate
values is represented only once in the main list processing.
* Fixed some capitialization typos and indentations mistakes in
Key_List::print_hash_function.
Sat Apr 29 12:04:03 1989 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Fixed a typo/logico in Key_List::print_switch that prevented
the last keyword in the keyword list to be print out. This
requires further examination.....
* Fixed a stupid bug in List_Node::List_node. If the -k'*' option
was enabled the KEY_SET string wasn't getting terminated with
'\0'!
Fri Apr 28 12:38:35 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Renamed strexp.h and strexp.cc to iterator.h and iterator.cc.
Also changed the strexp class to iterator. Continued to work
on style...
* Updated the version number to 1.6. This reflects all the
recent changes.
Thu Apr 27 00:14:51 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Added the -D option that properly handles keyword sets that
contain duplicate hash values.
* Continued the stylistic changes. Added the #pragma once
directive to all the *.h files. Removed all #defines and
replaced them with static consts. Also moved the key_sort
routine from options.cc into the options class as a
member function.
Mon Apr 3 13:26:55 1989 Doug Schmidt (schmidt at zola.ics.uci.edu)
* Made massive stylistic changes to bring source code into
conformance with GNU style guidelines.
Thu Mar 30 23:28:45 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Fixed up the output routines so that they generate code
corresponding to the GNU style guidelines.
Sat Mar 11 13:12:37 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Fixed Stderr constructors so that they wouldn't try to
use the base class initializer syntax for the static
class variable Program_Name. G++ 1.34 is stricter in
enforcing the rules!
Fri Mar 10 11:24:14 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Removed -v and ``| more'' from the Makefile to keep rfg happy...
Thu Mar 2 12:37:30 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Sent latest GNU gperf version 1.5 to Doug Lea for inclusion
into libg++ 1.34. Note that there is a small bug with
the new %{ ... %} source inclusion facility, since it doesn't
understand comments and will barf if %{ or %} appear nested
inside the outermost delimiters. This is too trivial of
a defect to fix at the moment...
Tue Feb 28 11:19:58 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Added the -K option, which allows the user to provide a
alternative name for the keyword structure component.
The default is still ``name.''
* Added the LEX and YACC-like ability to include arbitrary
text at the beginning of the generated C source code output.
This required two new functions Get_Special_Input,
Key_List::Save_Include_Src;
* Fixed memory allocation bug in Key_List::Set_Types.
Variable Return_Type needs 1 additional location
to store the "*" if the -p option is used.
* Added code to NULL terminate both Struct_Tag and Return_Type,
*after* the strncpy (stupid mistake).
Mon Feb 27 14:39:51 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Added a new option -N. This allows the user to specify the
name to be used for the generated lookup function. The
default name is still ``in_word_set.'' This makes it
possible to completely automate the perfect hash function
generation process!
Mon Feb 20 23:33:14 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Corrected the Hash_Table::operator () function so that
*it* is responsible for deciding when a new key has the
same signature as a previously seen key. The key length
information is now used internally to this function to
decide whether to add to the hash table those keys with
the same key sets, but different lengths. Before, this
was handled by the Key_List::Read_Keys function. However,
this failed to work for certain duplicate keys, since
they weren't being entered into the hash table properly.
Sun Feb 19 16:02:51 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Modified class Options by moving the enum Option_Type out
of the class. This is to satisfy the new enumeration
scope rules in C++.
Sun Jan 15 15:12:09 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Incremented the version number upto 1.4 to reflect the new
options that affect the generated code. Send the new
distribution off to Michael for use with g++ 1.33.
* Added a fix to Key_List::Read_Keys so that it checks for links
properly when the -n option is used. Previously, it didn't
catch obvious links, which caused it to spend large amount
of time searching for a solution that could never occur!
* Modified the Key_List data structure to record *both* the
minimum and the maximum key lengths. This information
is now computed in Key_List::Read_Keys, and thus
Key_List::Print_Min_Max doesn't need to bother.
* Modifed the key position iterator scheme in options.cc to
eliminate the need for member function Options::Advance.
Now, the Options::Get function performs the advancement
automatically, obviating the need for an extra function call.
* Added the new function Options::Print_Options, to print out
the user-specified command line options to generated C
output file.
* Added a new function, Key_List::Print_Keylength_Table,
which creates a table of lengths for use in speeding
up the keyword search. This also meant that a new
option, -l (LENTABLE) is recognized. It controls
whether the length table is printed and the comparison
made in the generated function ``in_word_set.''
* Added a comment at the top of the generated C code
output file that tells what version of gperf was used.
Next, I'll also dump out the command line options
as a comment too. Thanks to Michael Tiemann for the
feedback on this.
* Fixed the -n option to make it work correctly with
other parts of the program (most notably the Perfect::Hash
function and the computation of minimum and maximum lengths.
Fri Jan 13 21:25:27 1989 Doug Schmidt (schmidt at siam.ics.uci.edu)
* Realized the the need to add a test that will enable
optimziation of the generated C code in the ``hash'' function
by checking whether all the requested key positions are
guaranteed to exist due to the comparison in `in_word_set.''
I'll put this in soon....
Thu Jan 12 20:09:21 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Added pascal, modula3, and modula2 tests inputs to the
Makefile
* Recognised that there is a bug with the -n option. However
I'm too busy to fix it properly, right now. The problem
is that the generated #define end up being 0, since that's
my hack to make -n work. This needs complete rethinking!
Tue Jan 10 00:08:16 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Added a new option, -n, that instructs gperf to not use the
length of an identifier when computing the hash functions.
I'm not sure how useful this is!
* Retransmitted the distribution to rocky.oswego.edu. Hopefully,
this will work!
* Began fixing the indentation and capitalization to conform
to the GNU coding guidelines.
Mon Jan 9 22:23:18 1989 Doug Schmidt (schmidt at pompe.ics.uci.edu)
* Fixed horrible bug in Read_Line::Readln_Aux. This was
a subtle and pernicous off-by-1 error, that overwrote
past the last character of the input string buffer. I
think this fault was killing the vax!
* Yow, fixed an oversight in List_Node::List_Node, where the
pointer field Next was uninitialized. Luckily, the new routine
seems to return 0 filled objects the first time through!
Sun Jan 8 13:43:14 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Modified the ``key linked'' diagnostic in Key_List::Read_Keys
to be more helpful and easy to read.
* Fixed the List_Node::List_Node so that it would ignore trailing
fields if the -t option was not enabled.
* Moved the List_Node declarations out of keylist.h and
into a file of its own, called listnode.cc and listnode.h
Made Set_Sort a member function of class List_Node.
* Massively updated the documentation in the gperf.texinfo file.
* Polished off the major revision to the print functions,
added a few new tests in the Makefile to check for the
validity of the program and ftp'ed the entire distribution
off to Doug Lea for libg++. ( changed it to
1.3 to reflect the major changes with the generated
C code ).
* Fixed Key_List::Print_Switch to deal with the -p and -t options.
This meant that the ``still-born'' function Key_List::
Print_Type_Switch was superflous, so I removed it.
Also, removed the restriction in Option that the -p and
-t options couldn't be used simultaneously.
* Modified List_Node::List_Node, to perform only 1 call to
``new'' when dynamically allocating memory for the Key_Set
and the Uniq_Set.
Sat Jan 7 14:10:51 1989 Doug Schmidt (schmidt at glacier.ics.uci.edu)
* Fixed a big bug with the new policy of nesting the
wordlist inside of generated function ``in_word_set.''
I'd forgotten to declare the wordlist array as static!
( arrgh ).
* Added a new function Key_List::Set_Types, that figures out
the return type for generated function ``in_word_set,''
the user-defined ``struct tag,'' if one is used, and also
formates the array type for the static local array.
* Changed the print routines to take advantage of the
new -p option.
* Began adding the hooks to allow the return of a pointer
to a user defined struct location from the generated
``in_word_set'' function instead of the current 0 or 1
return value. Created function Key_List::Print_Type_Switch
and added option -p to class Option, allowing the user to
request generation of the aforementioned pointers returned
instead of booleans.
* Put in checks in class Option to make sure that -S and -t
options are not used simultaneously. This restriction
will be removed in subsequent releases, once I decide on
a clean way to implement it.
* Sent version 1.2 to Doug Lea for possible inclusion into
the libg++ distribution.
* Moved the static word_list array inside the generated function
in_word_set. This supports better data hiding.
* Added a texinfo file, gperf.texinfo
* Revised the Makefile to cleanup the droppings from texinfo
and changed the name of gperf.cc and gperf.h to perfect.cc
and perfect.h.
Fri Jan 6 13:04:45 1989 Doug Schmidt (schmidt at crimee.ics.uci.edu)
* Implemented the switch statement output format. Much better
for large datasets in terms of space used.
* Added new functions to break up the Key_List::Output function.
Functions added were Key_List::Print_Switch, Key_List::Print_Min_Max,
Key_List::Print_Keyword_Table, Key_List::Print_Hash_Function,
and Key_List::Print_Lookup_Function. This simplifies the
big mess in Key_List::Output considerably!
* Added switch statement option to Options, which potentially
trades time for space in the generated lookup code.
Thu Jan 5 22:46:34 1989 Doug Schmidt (schmidt at siam.ics.uci.edu)
* Released version 1.1
* Fixed a bug with Gperf::Merge_Set, it was skipping letters shared
between the Set_1 and Set_2.
* Added the optimal min/max algorithm in Key_List::Output. This
runs in O ( 3n/2 ), rather than O ( 2n ) time.
* Changed Gperf::Sort_Set to use insertion sort, rather than
bubble sort.
* Added a check in Key_List::Output for the special case where
the keys used are 1,$. It is possible to generate more
efficient C code in this case.