| 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. |