unicode: use new faster htable
authorDavid Herrmann <dh.herrmann@gmail.com>
Wed, 23 Oct 2013 16:05:19 +0000 (18:05 +0200)
committerDavid Herrmann <dh.herrmann@gmail.com>
Wed, 23 Oct 2013 16:06:22 +0000 (18:06 +0200)
Replace the old shl_hashtable with shl_htable, which avoids the additional
allocations and is considerably faster. The underlying hash-table is still
the same, but our boilerplate code is reduced heavily.

Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
LICENSE_htable [new file with mode: 0644]
Makefile.am
src/shl_htable.c [new file with mode: 0644]
src/shl_htable.h [new file with mode: 0644]
src/tsm_unicode.c

diff --git a/LICENSE_htable b/LICENSE_htable
new file mode 100644 (file)
index 0000000..2d2d780
--- /dev/null
@@ -0,0 +1,510 @@
+
+                  GNU LESSER GENERAL PUBLIC LICENSE
+                       Version 2.1, February 1999
+
+ Copyright (C) 1991, 1999 Free Software Foundation, Inc.
+       51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+ Everyone is permitted to copy and distribute verbatim copies
+ of this license document, but changing it is not allowed.
+
+[This is the first released version of the Lesser GPL.  It also counts
+ as the successor of the GNU Library Public License, version 2, hence
+ the version number 2.1.]
+
+                            Preamble
+
+  The licenses for most software are designed to take away your
+freedom to share and change it.  By contrast, the GNU General Public
+Licenses are intended to guarantee your freedom to share and change
+free software--to make sure the software is free for all its users.
+
+  This license, the Lesser General Public License, applies to some
+specially designated software packages--typically libraries--of the
+Free Software Foundation and other authors who decide to use it.  You
+can use it too, but we suggest you first think carefully about whether
+this license or the ordinary General Public License is the better
+strategy to use in any particular case, based on the explanations
+below.
+
+  When we speak of free software, we are referring to freedom of use,
+not price.  Our General Public Licenses are designed to make sure that
+you have the freedom to distribute copies of free software (and charge
+for this service if you wish); that you receive source code or can get
+it if you want it; that you can change the software and use pieces of
+it in new free programs; and that you are informed that you can do
+these things.
+
+  To protect your rights, we need to make restrictions that forbid
+distributors to deny you these rights or to ask you to surrender these
+rights.  These restrictions translate to certain responsibilities for
+you if you distribute copies of the library or if you modify it.
+
+  For example, if you distribute copies of the library, whether gratis
+or for a fee, you must give the recipients all the rights that we gave
+you.  You must make sure that they, too, receive or can get the source
+code.  If you link other code with the library, you must provide
+complete object files to the recipients, so that they can relink them
+with the library after making changes to the library and recompiling
+it.  And you must show them these terms so they know their rights.
+
+  We protect your rights with a two-step method: (1) we copyright the
+library, and (2) we offer you this license, which gives you legal
+permission to copy, distribute and/or modify the library.
+
+  To protect each distributor, we want to make it very clear that
+there is no warranty for the free library.  Also, if the library is
+modified by someone else and passed on, the recipients should know
+that what they have is not the original version, so that the original
+author's reputation will not be affected by problems that might be
+introduced by others.
+\f
+  Finally, software patents pose a constant threat to the existence of
+any free program.  We wish to make sure that a company cannot
+effectively restrict the users of a free program by obtaining a
+restrictive license from a patent holder.  Therefore, we insist that
+any patent license obtained for a version of the library must be
+consistent with the full freedom of use specified in this license.
+
+  Most GNU software, including some libraries, is covered by the
+ordinary GNU General Public License.  This license, the GNU Lesser
+General Public License, applies to certain designated libraries, and
+is quite different from the ordinary General Public License.  We use
+this license for certain libraries in order to permit linking those
+libraries into non-free programs.
+
+  When a program is linked with a library, whether statically or using
+a shared library, the combination of the two is legally speaking a
+combined work, a derivative of the original library.  The ordinary
+General Public License therefore permits such linking only if the
+entire combination fits its criteria of freedom.  The Lesser General
+Public License permits more lax criteria for linking other code with
+the library.
+
+  We call this license the "Lesser" General Public License because it
+does Less to protect the user's freedom than the ordinary General
+Public License.  It also provides other free software developers Less
+of an advantage over competing non-free programs.  These disadvantages
+are the reason we use the ordinary General Public License for many
+libraries.  However, the Lesser license provides advantages in certain
+special circumstances.
+
+  For example, on rare occasions, there may be a special need to
+encourage the widest possible use of a certain library, so that it
+becomes a de-facto standard.  To achieve this, non-free programs must
+be allowed to use the library.  A more frequent case is that a free
+library does the same job as widely used non-free libraries.  In this
+case, there is little to gain by limiting the free library to free
+software only, so we use the Lesser General Public License.
+
+  In other cases, permission to use a particular library in non-free
+programs enables a greater number of people to use a large body of
+free software.  For example, permission to use the GNU C Library in
+non-free programs enables many more people to use the whole GNU
+operating system, as well as its variant, the GNU/Linux operating
+system.
+
+  Although the Lesser General Public License is Less protective of the
+users' freedom, it does ensure that the user of a program that is
+linked with the Library has the freedom and the wherewithal to run
+that program using a modified version of the Library.
+
+  The precise terms and conditions for copying, distribution and
+modification follow.  Pay close attention to the difference between a
+"work based on the library" and a "work that uses the library".  The
+former contains code derived from the library, whereas the latter must
+be combined with the library in order to run.
+\f
+                  GNU LESSER GENERAL PUBLIC LICENSE
+   TERMS AND CONDITIONS FOR COPYING, DISTRIBUTION AND MODIFICATION
+
+  0. This License Agreement applies to any software library or other
+program which contains a notice placed by the copyright holder or
+other authorized party saying it may be distributed under the terms of
+this Lesser General Public License (also called "this License").
+Each licensee is addressed as "you".
+
+  A "library" means a collection of software functions and/or data
+prepared so as to be conveniently linked with application programs
+(which use some of those functions and data) to form executables.
+
+  The "Library", below, refers to any such software library or work
+which has been distributed under these terms.  A "work based on the
+Library" means either the Library or any derivative work under
+copyright law: that is to say, a work containing the Library or a
+portion of it, either verbatim or with modifications and/or translated
+straightforwardly into another language.  (Hereinafter, translation is
+included without limitation in the term "modification".)
+
+  "Source code" for a work means the preferred form of the work for
+making modifications to it.  For a library, complete source code means
+all the source code for all modules it contains, plus any associated
+interface definition files, plus the scripts used to control
+compilation and installation of the library.
+
+  Activities other than copying, distribution and modification are not
+covered by this License; they are outside its scope.  The act of
+running a program using the Library is not restricted, and output from
+such a program is covered only if its contents constitute a work based
+on the Library (independent of the use of the Library in a tool for
+writing it).  Whether that is true depends on what the Library does
+and what the program that uses the Library does.
+
+  1. You may copy and distribute verbatim copies of the Library's
+complete source code as you receive it, in any medium, provided that
+you conspicuously and appropriately publish on each copy an
+appropriate copyright notice and disclaimer of warranty; keep intact
+all the notices that refer to this License and to the absence of any
+warranty; and distribute a copy of this License along with the
+Library.
+
+  You may charge a fee for the physical act of transferring a copy,
+and you may at your option offer warranty protection in exchange for a
+fee.
+\f
+  2. You may modify your copy or copies of the Library or any portion
+of it, thus forming a work based on the Library, and copy and
+distribute such modifications or work under the terms of Section 1
+above, provided that you also meet all of these conditions:
+
+    a) The modified work must itself be a software library.
+
+    b) You must cause the files modified to carry prominent notices
+    stating that you changed the files and the date of any change.
+
+    c) You must cause the whole of the work to be licensed at no
+    charge to all third parties under the terms of this License.
+
+    d) If a facility in the modified Library refers to a function or a
+    table of data to be supplied by an application program that uses
+    the facility, other than as an argument passed when the facility
+    is invoked, then you must make a good faith effort to ensure that,
+    in the event an application does not supply such function or
+    table, the facility still operates, and performs whatever part of
+    its purpose remains meaningful.
+
+    (For example, a function in a library to compute square roots has
+    a purpose that is entirely well-defined independent of the
+    application.  Therefore, Subsection 2d requires that any
+    application-supplied function or table used by this function must
+    be optional: if the application does not supply it, the square
+    root function must still compute square roots.)
+
+These requirements apply to the modified work as a whole.  If
+identifiable sections of that work are not derived from the Library,
+and can be reasonably considered independent and separate works in
+themselves, then this License, and its terms, do not apply to those
+sections when you distribute them as separate works.  But when you
+distribute the same sections as part of a whole which is a work based
+on the Library, the distribution of the whole must be on the terms of
+this License, whose permissions for other licensees extend to the
+entire whole, and thus to each and every part regardless of who wrote
+it.
+
+Thus, it is not the intent of this section to claim rights or contest
+your rights to work written entirely by you; rather, the intent is to
+exercise the right to control the distribution of derivative or
+collective works based on the Library.
+
+In addition, mere aggregation of another work not based on the Library
+with the Library (or with a work based on the Library) on a volume of
+a storage or distribution medium does not bring the other work under
+the scope of this License.
+
+  3. You may opt to apply the terms of the ordinary GNU General Public
+License instead of this License to a given copy of the Library.  To do
+this, you must alter all the notices that refer to this License, so
+that they refer to the ordinary GNU General Public License, version 2,
+instead of to this License.  (If a newer version than version 2 of the
+ordinary GNU General Public License has appeared, then you can specify
+that version instead if you wish.)  Do not make any other change in
+these notices.
+\f
+  Once this change is made in a given copy, it is irreversible for
+that copy, so the ordinary GNU General Public License applies to all
+subsequent copies and derivative works made from that copy.
+
+  This option is useful when you wish to copy part of the code of
+the Library into a program that is not a library.
+
+  4. You may copy and distribute the Library (or a portion or
+derivative of it, under Section 2) in object code or executable form
+under the terms of Sections 1 and 2 above provided that you accompany
+it with the complete corresponding machine-readable source code, which
+must be distributed under the terms of Sections 1 and 2 above on a
+medium customarily used for software interchange.
+
+  If distribution of object code is made by offering access to copy
+from a designated place, then offering equivalent access to copy the
+source code from the same place satisfies the requirement to
+distribute the source code, even though third parties are not
+compelled to copy the source along with the object code.
+
+  5. A program that contains no derivative of any portion of the
+Library, but is designed to work with the Library by being compiled or
+linked with it, is called a "work that uses the Library".  Such a
+work, in isolation, is not a derivative work of the Library, and
+therefore falls outside the scope of this License.
+
+  However, linking a "work that uses the Library" with the Library
+creates an executable that is a derivative of the Library (because it
+contains portions of the Library), rather than a "work that uses the
+library".  The executable is therefore covered by this License.
+Section 6 states terms for distribution of such executables.
+
+  When a "work that uses the Library" uses material from a header file
+that is part of the Library, the object code for the work may be a
+derivative work of the Library even though the source code is not.
+Whether this is true is especially significant if the work can be
+linked without the Library, or if the work is itself a library.  The
+threshold for this to be true is not precisely defined by law.
+
+  If such an object file uses only numerical parameters, data
+structure layouts and accessors, and small macros and small inline
+functions (ten lines or less in length), then the use of the object
+file is unrestricted, regardless of whether it is legally a derivative
+work.  (Executables containing this object code plus portions of the
+Library will still fall under Section 6.)
+
+  Otherwise, if the work is a derivative of the Library, you may
+distribute the object code for the work under the terms of Section 6.
+Any executables containing that work also fall under Section 6,
+whether or not they are linked directly with the Library itself.
+\f
+  6. As an exception to the Sections above, you may also combine or
+link a "work that uses the Library" with the Library to produce a
+work containing portions of the Library, and distribute that work
+under terms of your choice, provided that the terms permit
+modification of the work for the customer's own use and reverse
+engineering for debugging such modifications.
+
+  You must give prominent notice with each copy of the work that the
+Library is used in it and that the Library and its use are covered by
+this License.  You must supply a copy of this License.  If the work
+during execution displays copyright notices, you must include the
+copyright notice for the Library among them, as well as a reference
+directing the user to the copy of this License.  Also, you must do one
+of these things:
+
+    a) Accompany the work with the complete corresponding
+    machine-readable source code for the Library including whatever
+    changes were used in the work (which must be distributed under
+    Sections 1 and 2 above); and, if the work is an executable linked
+    with the Library, with the complete machine-readable "work that
+    uses the Library", as object code and/or source code, so that the
+    user can modify the Library and then relink to produce a modified
+    executable containing the modified Library.  (It is understood
+    that the user who changes the contents of definitions files in the
+    Library will not necessarily be able to recompile the application
+    to use the modified definitions.)
+
+    b) Use a suitable shared library mechanism for linking with the
+    Library.  A suitable mechanism is one that (1) uses at run time a
+    copy of the library already present on the user's computer system,
+    rather than copying library functions into the executable, and (2)
+    will operate properly with a modified version of the library, if
+    the user installs one, as long as the modified version is
+    interface-compatible with the version that the work was made with.
+
+    c) Accompany the work with a written offer, valid for at least
+    three years, to give the same user the materials specified in
+    Subsection 6a, above, for a charge no more than the cost of
+    performing this distribution.
+
+    d) If distribution of the work is made by offering access to copy
+    from a designated place, offer equivalent access to copy the above
+    specified materials from the same place.
+
+    e) Verify that the user has already received a copy of these
+    materials or that you have already sent this user a copy.
+
+  For an executable, the required form of the "work that uses the
+Library" must include any data and utility programs needed for
+reproducing the executable from it.  However, as a special exception,
+the materials to be distributed need not include anything that is
+normally distributed (in either source or binary form) with the major
+components (compiler, kernel, and so on) of the operating system on
+which the executable runs, unless that component itself accompanies
+the executable.
+
+  It may happen that this requirement contradicts the license
+restrictions of other proprietary libraries that do not normally
+accompany the operating system.  Such a contradiction means you cannot
+use both them and the Library together in an executable that you
+distribute.
+\f
+  7. You may place library facilities that are a work based on the
+Library side-by-side in a single library together with other library
+facilities not covered by this License, and distribute such a combined
+library, provided that the separate distribution of the work based on
+the Library and of the other library facilities is otherwise
+permitted, and provided that you do these two things:
+
+    a) Accompany the combined library with a copy of the same work
+    based on the Library, uncombined with any other library
+    facilities.  This must be distributed under the terms of the
+    Sections above.
+
+    b) Give prominent notice with the combined library of the fact
+    that part of it is a work based on the Library, and explaining
+    where to find the accompanying uncombined form of the same work.
+
+  8. You may not copy, modify, sublicense, link with, or distribute
+the Library except as expressly provided under this License.  Any
+attempt otherwise to copy, modify, sublicense, link with, or
+distribute the Library is void, and will automatically terminate your
+rights under this License.  However, parties who have received copies,
+or rights, from you under this License will not have their licenses
+terminated so long as such parties remain in full compliance.
+
+  9. You are not required to accept this License, since you have not
+signed it.  However, nothing else grants you permission to modify or
+distribute the Library or its derivative works.  These actions are
+prohibited by law if you do not accept this License.  Therefore, by
+modifying or distributing the Library (or any work based on the
+Library), you indicate your acceptance of this License to do so, and
+all its terms and conditions for copying, distributing or modifying
+the Library or works based on it.
+
+  10. Each time you redistribute the Library (or any work based on the
+Library), the recipient automatically receives a license from the
+original licensor to copy, distribute, link with or modify the Library
+subject to these terms and conditions.  You may not impose any further
+restrictions on the recipients' exercise of the rights granted herein.
+You are not responsible for enforcing compliance by third parties with
+this License.
+\f
+  11. If, as a consequence of a court judgment or allegation of patent
+infringement or for any other reason (not limited to patent issues),
+conditions are imposed on you (whether by court order, agreement or
+otherwise) that contradict the conditions of this License, they do not
+excuse you from the conditions of this License.  If you cannot
+distribute so as to satisfy simultaneously your obligations under this
+License and any other pertinent obligations, then as a consequence you
+may not distribute the Library at all.  For example, if a patent
+license would not permit royalty-free redistribution of the Library by
+all those who receive copies directly or indirectly through you, then
+the only way you could satisfy both it and this License would be to
+refrain entirely from distribution of the Library.
+
+If any portion of this section is held invalid or unenforceable under
+any particular circumstance, the balance of the section is intended to
+apply, and the section as a whole is intended to apply in other
+circumstances.
+
+It is not the purpose of this section to induce you to infringe any
+patents or other property right claims or to contest validity of any
+such claims; this section has the sole purpose of protecting the
+integrity of the free software distribution system which is
+implemented by public license practices.  Many people have made
+generous contributions to the wide range of software distributed
+through that system in reliance on consistent application of that
+system; it is up to the author/donor to decide if he or she is willing
+to distribute software through any other system and a licensee cannot
+impose that choice.
+
+This section is intended to make thoroughly clear what is believed to
+be a consequence of the rest of this License.
+
+  12. If the distribution and/or use of the Library is restricted in
+certain countries either by patents or by copyrighted interfaces, the
+original copyright holder who places the Library under this License
+may add an explicit geographical distribution limitation excluding those
+countries, so that distribution is permitted only in or among
+countries not thus excluded.  In such case, this License incorporates
+the limitation as if written in the body of this License.
+
+  13. The Free Software Foundation may publish revised and/or new
+versions of the Lesser General Public License from time to time.
+Such new versions will be similar in spirit to the present version,
+but may differ in detail to address new problems or concerns.
+
+Each version is given a distinguishing version number.  If the Library
+specifies a version number of this License which applies to it and
+"any later version", you have the option of following the terms and
+conditions either of that version or of any later version published by
+the Free Software Foundation.  If the Library does not specify a
+license version number, you may choose any version ever published by
+the Free Software Foundation.
+\f
+  14. If you wish to incorporate parts of the Library into other free
+programs whose distribution conditions are incompatible with these,
+write to the author to ask for permission.  For software which is
+copyrighted by the Free Software Foundation, write to the Free
+Software Foundation; we sometimes make exceptions for this.  Our
+decision will be guided by the two goals of preserving the free status
+of all derivatives of our free software and of promoting the sharing
+and reuse of software generally.
+
+                            NO WARRANTY
+
+  15. BECAUSE THE LIBRARY IS LICENSED FREE OF CHARGE, THERE IS NO
+WARRANTY FOR THE LIBRARY, TO THE EXTENT PERMITTED BY APPLICABLE LAW.
+EXCEPT WHEN OTHERWISE STATED IN WRITING THE COPYRIGHT HOLDERS AND/OR
+OTHER PARTIES PROVIDE THE LIBRARY "AS IS" WITHOUT WARRANTY OF ANY
+KIND, EITHER EXPRESSED OR IMPLIED, INCLUDING, BUT NOT LIMITED TO, THE
+IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+PURPOSE.  THE ENTIRE RISK AS TO THE QUALITY AND PERFORMANCE OF THE
+LIBRARY IS WITH YOU.  SHOULD THE LIBRARY PROVE DEFECTIVE, YOU ASSUME
+THE COST OF ALL NECESSARY SERVICING, REPAIR OR CORRECTION.
+
+  16. IN NO EVENT UNLESS REQUIRED BY APPLICABLE LAW OR AGREED TO IN
+WRITING WILL ANY COPYRIGHT HOLDER, OR ANY OTHER PARTY WHO MAY MODIFY
+AND/OR REDISTRIBUTE THE LIBRARY AS PERMITTED ABOVE, BE LIABLE TO YOU
+FOR DAMAGES, INCLUDING ANY GENERAL, SPECIAL, INCIDENTAL OR
+CONSEQUENTIAL DAMAGES ARISING OUT OF THE USE OR INABILITY TO USE THE
+LIBRARY (INCLUDING BUT NOT LIMITED TO LOSS OF DATA OR DATA BEING
+RENDERED INACCURATE OR LOSSES SUSTAINED BY YOU OR THIRD PARTIES OR A
+FAILURE OF THE LIBRARY TO OPERATE WITH ANY OTHER SOFTWARE), EVEN IF
+SUCH HOLDER OR OTHER PARTY HAS BEEN ADVISED OF THE POSSIBILITY OF SUCH
+DAMAGES.
+
+                     END OF TERMS AND CONDITIONS
+\f
+           How to Apply These Terms to Your New Libraries
+
+  If you develop a new library, and you want it to be of the greatest
+possible use to the public, we recommend making it free software that
+everyone can redistribute and change.  You can do so by permitting
+redistribution under these terms (or, alternatively, under the terms
+of the ordinary General Public License).
+
+  To apply these terms, attach the following notices to the library.
+It is safest to attach them to the start of each source file to most
+effectively convey the exclusion of warranty; and each file should
+have at least the "copyright" line and a pointer to where the full
+notice is found.
+
+
+    <one line to give the library's name and a brief idea of what it does.>
+    Copyright (C) <year>  <name of author>
+
+    This library is free software; you can redistribute it and/or
+    modify it under the terms of the GNU Lesser General Public
+    License as published by the Free Software Foundation; either
+    version 2.1 of the License, or (at your option) any later version.
+
+    This library is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+    Lesser General Public License for more details.
+
+    You should have received a copy of the GNU Lesser General Public
+    License along with this library; if not, write to the Free Software
+    Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+
+Also add information on how to contact you by electronic and paper mail.
+
+You should also get your employer (if you work as a programmer) or
+your school, if any, to sign a "copyright disclaimer" for the library,
+if necessary.  Here is a sample; alter the names:
+
+  Yoyodyne, Inc., hereby disclaims all copyright interest in the
+  library `Frob' (a library for tweaking knobs) written by James
+  Random Hacker.
+
+  <signature of Ty Coon>, 1 April 1990
+  Ty Coon, President of Vice
+
+That's all there is to it!
+
+
index e6dfdb7..d480820 100644 (file)
@@ -95,9 +95,8 @@ noinst_LTLIBRARIES += libshl.la
 
 libshl_la_SOURCES = \
        src/shl_array.h \
-       src/shl_hashtable.h \
-       external/htable.h \
-       external/htable.c \
+       src/shl_htable.h \
+       src/shl_htable.c \
        src/shl_llog.h
 libshl_la_CPPFLAGS = $(AM_CPPFLAGS)
 libshl_la_LDFLAGS = $(AM_LDFLAGS)
diff --git a/src/shl_htable.c b/src/shl_htable.c
new file mode 100644 (file)
index 0000000..e79a7c0
--- /dev/null
@@ -0,0 +1,428 @@
+/*
+ * SHL - Dynamic hash-table
+ *
+ * Written-by: Rusty Russell <rusty@rustcorp.com.au>
+ * Adjusted-by: David Herrmann <dh.herrmann@gmail.com>
+ * Licensed under LGPLv2+ - see LICENSE_htable file for details
+ */
+
+/*
+ * Please see ccan/htable/_info at:
+ *   https://github.com/rustyrussell/ccan/tree/master/ccan/htable
+ * for information on the hashtable algorithm. This file copies the code inline
+ * and is released under the same conditions.
+ *
+ * At the end of the file you can find some helpers to use this htable to store
+ * objects with "unsigned long" or "char*" keys.
+ */
+
+#include <assert.h>
+#include <errno.h>
+#include <limits.h>
+#include <stdbool.h>
+#include <stdint.h>
+#include <stdlib.h>
+#include <string.h>
+#include "shl_htable.h"
+
+#define COLD __attribute__((cold))
+
+struct htable {
+       /* KEEP IN SYNC WITH "struct shl_htable_int" */
+       size_t (*rehash)(const void *elem, void *priv);
+       void *priv;
+       unsigned int bits;
+       size_t elems, deleted, max, max_with_deleted;
+       /* These are the bits which are the same in all pointers. */
+       uintptr_t common_mask, common_bits;
+       uintptr_t perfect_bit;
+       uintptr_t *table;
+};
+
+#define HTABLE_INITIALIZER(name, rehash, priv)                         \
+       { rehash, priv, 0, 0, 0, 0, 0, -1, 0, 0, &name.perfect_bit }
+
+struct htable_iter {
+       size_t off;
+};
+
+/*
+ * INLINE COPY OF ccan/htable.c
+ */
+
+/* We use 0x1 as deleted marker. */
+#define HTABLE_DELETED (0x1)
+
+/* We clear out the bits which are always the same, and put metadata there. */
+static inline uintptr_t get_extra_ptr_bits(const struct htable *ht,
+                                          uintptr_t e)
+{
+       return e & ht->common_mask;
+}
+
+static inline void *get_raw_ptr(const struct htable *ht, uintptr_t e)
+{
+       return (void *)((e & ~ht->common_mask) | ht->common_bits);
+}
+
+static inline uintptr_t make_hval(const struct htable *ht,
+                                 const void *p, uintptr_t bits)
+{
+       return ((uintptr_t)p & ~ht->common_mask) | bits;
+}
+
+static inline bool entry_is_valid(uintptr_t e)
+{
+       return e > HTABLE_DELETED;
+}
+
+static inline uintptr_t get_hash_ptr_bits(const struct htable *ht,
+                                         size_t hash)
+{
+       /* Shuffling the extra bits (as specified in mask) down the
+        * end is quite expensive.  But the lower bits are redundant, so
+        * we fold the value first. */
+       return (hash ^ (hash >> ht->bits))
+               & ht->common_mask & ~ht->perfect_bit;
+}
+
+static void htable_init(struct htable *ht,
+                       size_t (*rehash)(const void *elem, void *priv),
+                       void *priv)
+{
+       struct htable empty = HTABLE_INITIALIZER(empty, NULL, NULL);
+       *ht = empty;
+       ht->rehash = rehash;
+       ht->priv = priv;
+       ht->table = &ht->perfect_bit;
+}
+
+static void htable_clear(struct htable *ht,
+                        void (*free_cb) (void *entry, void *ctx),
+                        void *ctx)
+{
+       size_t i;
+
+       if (ht->table != &ht->perfect_bit) {
+               if (free_cb) {
+                       for (i = 0; i < (size_t)1 << ht->bits; ++i) {
+                               if (entry_is_valid(ht->table[i]))
+                                       free_cb(get_raw_ptr(ht, ht->table[i]),
+                                               ctx);
+                       }
+               }
+
+               free((void *)ht->table);
+       }
+
+       htable_init(ht, ht->rehash, ht->priv);
+}
+
+static void htable_visit(struct htable *ht,
+                        void (*visit_cb) (void *elem, void *ctx),
+                        void *ctx)
+{
+       size_t i;
+
+       if (visit_cb && ht->table != &ht->perfect_bit) {
+               for (i = 0; i < (size_t)1 << ht->bits; ++i) {
+                       if (entry_is_valid(ht->table[i]))
+                               visit_cb(get_raw_ptr(ht, ht->table[i]), ctx);
+               }
+       }
+}
+
+static size_t hash_bucket(const struct htable *ht, size_t h)
+{
+       return h & ((1 << ht->bits)-1);
+}
+
+static void *htable_val(const struct htable *ht,
+                       struct htable_iter *i, size_t hash, uintptr_t perfect)
+{
+       uintptr_t h2 = get_hash_ptr_bits(ht, hash) | perfect;
+
+       while (ht->table[i->off]) {
+               if (ht->table[i->off] != HTABLE_DELETED) {
+                       if (get_extra_ptr_bits(ht, ht->table[i->off]) == h2)
+                               return get_raw_ptr(ht, ht->table[i->off]);
+               }
+               i->off = (i->off + 1) & ((1 << ht->bits)-1);
+               h2 &= ~perfect;
+       }
+       return NULL;
+}
+
+static void *htable_firstval(const struct htable *ht,
+                            struct htable_iter *i, size_t hash)
+{
+       i->off = hash_bucket(ht, hash);
+       return htable_val(ht, i, hash, ht->perfect_bit);
+}
+
+static void *htable_nextval(const struct htable *ht,
+                           struct htable_iter *i, size_t hash)
+{
+       i->off = (i->off + 1) & ((1 << ht->bits)-1);
+       return htable_val(ht, i, hash, 0);
+}
+
+/* This does not expand the hash table, that's up to caller. */
+static void ht_add(struct htable *ht, const void *new, size_t h)
+{
+       size_t i;
+       uintptr_t perfect = ht->perfect_bit;
+
+       i = hash_bucket(ht, h);
+
+       while (entry_is_valid(ht->table[i])) {
+               perfect = 0;
+               i = (i + 1) & ((1 << ht->bits)-1);
+       }
+       ht->table[i] = make_hval(ht, new, get_hash_ptr_bits(ht, h)|perfect);
+}
+
+static COLD bool double_table(struct htable *ht)
+{
+       unsigned int i;
+       size_t oldnum = (size_t)1 << ht->bits;
+       uintptr_t *oldtable, e;
+
+       oldtable = ht->table;
+       ht->table = calloc(1 << (ht->bits+1), sizeof(size_t));
+       if (!ht->table) {
+               ht->table = oldtable;
+               return false;
+       }
+       ht->bits++;
+       ht->max = ((size_t)3 << ht->bits) / 4;
+       ht->max_with_deleted = ((size_t)9 << ht->bits) / 10;
+
+       /* If we lost our "perfect bit", get it back now. */
+       if (!ht->perfect_bit && ht->common_mask) {
+               for (i = 0; i < sizeof(ht->common_mask) * CHAR_BIT; i++) {
+                       if (ht->common_mask & ((size_t)1 << i)) {
+                               ht->perfect_bit = (size_t)1 << i;
+                               break;
+                       }
+               }
+       }
+
+       if (oldtable != &ht->perfect_bit) {
+               for (i = 0; i < oldnum; i++) {
+                       if (entry_is_valid(e = oldtable[i])) {
+                               void *p = get_raw_ptr(ht, e);
+                               ht_add(ht, p, ht->rehash(p, ht->priv));
+                       }
+               }
+               free(oldtable);
+       }
+       ht->deleted = 0;
+       return true;
+}
+
+static COLD void rehash_table(struct htable *ht)
+{
+       size_t start, i;
+       uintptr_t e;
+
+       /* Beware wrap cases: we need to start from first empty bucket. */
+       for (start = 0; ht->table[start]; start++);
+
+       for (i = 0; i < (size_t)1 << ht->bits; i++) {
+               size_t h = (i + start) & ((1 << ht->bits)-1);
+               e = ht->table[h];
+               if (!e)
+                       continue;
+               if (e == HTABLE_DELETED)
+                       ht->table[h] = 0;
+               else if (!(e & ht->perfect_bit)) {
+                       void *p = get_raw_ptr(ht, e);
+                       ht->table[h] = 0;
+                       ht_add(ht, p, ht->rehash(p, ht->priv));
+               }
+       }
+       ht->deleted = 0;
+}
+
+/* We stole some bits, now we need to put them back... */
+static COLD void update_common(struct htable *ht, const void *p)
+{
+       unsigned int i;
+       uintptr_t maskdiff, bitsdiff;
+
+       if (ht->elems == 0) {
+               /* Always reveal one bit of the pointer in the bucket,
+                * so it's not zero or HTABLE_DELETED (1), even if
+                * hash happens to be 0.  Assumes (void *)1 is not a
+                * valid pointer. */
+               for (i = sizeof(uintptr_t)*CHAR_BIT - 1; i > 0; i--) {
+                       if ((uintptr_t)p & ((uintptr_t)1 << i))
+                               break;
+               }
+
+               ht->common_mask = ~((uintptr_t)1 << i);
+               ht->common_bits = ((uintptr_t)p & ht->common_mask);
+               ht->perfect_bit = 1;
+               return;
+       }
+
+       /* Find bits which are unequal to old common set. */
+       maskdiff = ht->common_bits ^ ((uintptr_t)p & ht->common_mask);
+
+       /* These are the bits which go there in existing entries. */
+       bitsdiff = ht->common_bits & maskdiff;
+
+       for (i = 0; i < (size_t)1 << ht->bits; i++) {
+               if (!entry_is_valid(ht->table[i]))
+                       continue;
+               /* Clear the bits no longer in the mask, set them as
+                * expected. */
+               ht->table[i] &= ~maskdiff;
+               ht->table[i] |= bitsdiff;
+       }
+
+       /* Take away those bits from our mask, bits and perfect bit. */
+       ht->common_mask &= ~maskdiff;
+       ht->common_bits &= ~maskdiff;
+       ht->perfect_bit &= ~maskdiff;
+}
+
+static bool htable_add(struct htable *ht, size_t hash, const void *p)
+{
+       if (ht->elems+1 > ht->max && !double_table(ht))
+               return false;
+       if (ht->elems+1 + ht->deleted > ht->max_with_deleted)
+               rehash_table(ht);
+       assert(p);
+       if (((uintptr_t)p & ht->common_mask) != ht->common_bits)
+               update_common(ht, p);
+
+       ht_add(ht, p, hash);
+       ht->elems++;
+       return true;
+}
+
+static void htable_delval(struct htable *ht, struct htable_iter *i)
+{
+       assert(i->off < (size_t)1 << ht->bits);
+       assert(entry_is_valid(ht->table[i->off]));
+
+       ht->elems--;
+       ht->table[i->off] = HTABLE_DELETED;
+       ht->deleted++;
+}
+
+/*
+ * Wrapper code to make it easier to use this hash-table as map.
+ */
+
+void shl_htable_init(struct shl_htable *htable,
+                    bool (*compare) (const void *a, const void *b),
+                    size_t (*rehash)(const void *elem, void *priv),
+                    void *priv)
+{
+       struct htable *ht = (void*)&htable->htable;
+
+       htable->compare = compare;
+       htable_init(ht, rehash, priv);
+}
+
+void shl_htable_clear(struct shl_htable *htable,
+                     void (*free_cb) (void *elem, void *ctx),
+                     void *ctx)
+{
+       struct htable *ht = (void*)&htable->htable;
+
+       htable_clear(ht, free_cb, ctx);
+}
+
+void shl_htable_visit(struct shl_htable *htable,
+                     void (*visit_cb) (void *elem, void *ctx),
+                     void *ctx)
+{
+       struct htable *ht = (void*)&htable->htable;
+
+       htable_visit(ht, visit_cb, ctx);
+}
+
+bool shl_htable_lookup(struct shl_htable *htable, const void *obj, size_t hash,
+                      void **out)
+{
+       struct htable *ht = (void*)&htable->htable;
+       struct htable_iter i;
+       void *c;
+
+       for (c = htable_firstval(ht, &i, hash);
+            c;
+            c = htable_nextval(ht, &i, hash)) {
+               if (htable->compare(obj, c)) {
+                       if (out)
+                               *out = c;
+                       return true;
+               }
+       }
+
+       return false;
+}
+
+int shl_htable_insert(struct shl_htable *htable, const void *obj, size_t hash)
+{
+       struct htable *ht = (void*)&htable->htable;
+       bool b;
+
+       b = htable_add(ht, hash, (void*)obj);
+       return b ? 0 : -ENOMEM;
+}
+
+bool shl_htable_remove(struct shl_htable *htable, const void *obj, size_t hash,
+                      void **out)
+{
+       struct htable *ht = (void*)&htable->htable;
+       struct htable_iter i;
+       void *c;
+
+       for (c = htable_firstval(ht, &i, hash);
+            c;
+            c = htable_nextval(ht, &i, hash)) {
+               if (htable->compare(obj, c)) {
+                       if (out)
+                               *out = c;
+                       htable_delval(ht, &i);
+                       return true;
+               }
+       }
+
+       return false;
+}
+
+/*
+ * Helpers
+ */
+
+bool shl_htable_compare_ulong(const void *a, const void *b)
+{
+       return *(const unsigned long*)a == *(const unsigned long*)b;
+}
+
+size_t shl_htable_rehash_ulong(const void *elem, void *priv)
+{
+       return (size_t)*(const unsigned long*)elem;
+}
+
+bool shl_htable_compare_str(const void *a, const void *b)
+{
+       return !strcmp(*(char**)a, *(char**)b);
+}
+
+/* DJB's hash function */
+size_t shl_htable_rehash_str(const void *elem, void *priv)
+{
+       const char *str = *(char**)elem;
+       size_t hash = 5381;
+
+       for ( ; *str; ++str)
+               hash = (hash << 5) + hash + (size_t)*str;
+
+       return hash;
+}
diff --git a/src/shl_htable.h b/src/shl_htable.h
new file mode 100644 (file)
index 0000000..9591180
--- /dev/null
@@ -0,0 +1,222 @@
+/*
+ * SHL - Dynamic hash-table
+ *
+ * Copyright (c) 2010-2013 David Herrmann <dh.herrmann@gmail.com>
+ * Licensed under LGPLv2+ - see LICENSE_htable file for details
+ */
+
+/*
+ * Dynamic hash-table
+ * Implementation of a self-resizing hashtable to store arbitrary objects.
+ * Entries are not allocated by the table itself but are user-allocated. A
+ * single entry can be stored multiple times in the hashtable. No
+ * maintenance-members need to be embedded in user-allocated objects. However,
+ * the key (and optionally the hash) must be stored in the objects.
+ *
+ * Uses internally the htable from CCAN. See LICENSE_htable.
+ */
+
+#ifndef SHL_HTABLE_H
+#define SHL_HTABLE_H
+
+#include <limits.h>
+#include <stdbool.h>
+#include <stddef.h>
+#include <stdint.h>
+#include <stdlib.h>
+
+/* miscellaneous */
+
+#define shl_htable_offsetof(pointer, type, member) ({ \
+               const typeof(((type*)0)->member) *__ptr = (pointer); \
+               (type*)(((char*)__ptr) - offsetof(type, member)); \
+       })
+
+/* htable */
+
+struct shl_htable_int {
+       size_t (*rehash)(const void *elem, void *priv);
+       void *priv;
+       unsigned int bits;
+       size_t elems, deleted, max, max_with_deleted;
+       /* These are the bits which are the same in all pointers. */
+       uintptr_t common_mask, common_bits;
+       uintptr_t perfect_bit;
+       uintptr_t *table;
+};
+
+struct shl_htable {
+       bool (*compare) (const void *a, const void *b);
+       struct shl_htable_int htable;
+};
+
+#define SHL_HTABLE_INIT(_obj, _compare, _rehash, _priv)                \
+       {                                                       \
+               .compare = (_compare),                          \
+               .htable = {                                     \
+                       .rehash = (_rehash),                    \
+                       .priv = (_priv),                        \
+                       .bits = 0,                              \
+                       .elems = 0,                             \
+                       .deleted = 0,                           \
+                       .max = 0,                               \
+                       .max_with_deleted = 0,                  \
+                       .common_mask = -1,                      \
+                       .common_bits = 0,                       \
+                       .perfect_bit = 0,                       \
+                       .table = &(_obj).htable.perfect_bit     \
+               }                                               \
+       }
+
+void shl_htable_init(struct shl_htable *htable,
+                    bool (*compare) (const void *a, const void *b),
+                    size_t (*rehash)(const void *elem, void *priv),
+                    void *priv);
+void shl_htable_clear(struct shl_htable *htable,
+                     void (*free_cb) (void *elem, void *ctx),
+                     void *ctx);
+void shl_htable_visit(struct shl_htable *htable,
+                     void (*visit_cb) (void *elem, void *ctx),
+                     void *ctx);
+bool shl_htable_lookup(struct shl_htable *htable, const void *obj, size_t hash,
+                      void **out);
+int shl_htable_insert(struct shl_htable *htable, const void *obj, size_t hash);
+bool shl_htable_remove(struct shl_htable *htable, const void *obj, size_t hash,
+                      void **out);
+
+/* ulong htables */
+
+#if SIZE_MAX < ULONG_MAX
+#  error "'size_t' is smaller than 'unsigned long'"
+#endif
+
+bool shl_htable_compare_ulong(const void *a, const void *b);
+size_t shl_htable_rehash_ulong(const void *elem, void *priv);
+
+#define SHL_HTABLE_INIT_ULONG(_obj)                                    \
+       SHL_HTABLE_INIT((_obj), shl_htable_compare_ulong,               \
+                               shl_htable_rehash_ulong,                \
+                               NULL)
+
+static inline void shl_htable_init_ulong(struct shl_htable *htable)
+{
+       shl_htable_init(htable, shl_htable_compare_ulong,
+                       shl_htable_rehash_ulong, NULL);
+}
+
+static inline void shl_htable_clear_ulong(struct shl_htable *htable,
+                                         void (*cb) (unsigned long *elem,
+                                                     void *ctx),
+                                         void *ctx)
+{
+       shl_htable_clear(htable, (void (*) (void*, void*))cb, ctx);
+}
+
+static inline void shl_htable_visit_ulong(struct shl_htable *htable,
+                                         void (*cb) (unsigned long *elem,
+                                                     void *ctx),
+                                         void *ctx)
+{
+       shl_htable_visit(htable, (void (*) (void*, void*))cb, ctx);
+}
+
+static inline bool shl_htable_lookup_ulong(struct shl_htable *htable,
+                                          unsigned long key,
+                                          unsigned long **out)
+{
+       return shl_htable_lookup(htable, (const void*)&key, (size_t)key,
+                                (void**)out);
+}
+
+static inline int shl_htable_insert_ulong(struct shl_htable *htable,
+                                         const unsigned long *key)
+{
+       return shl_htable_insert(htable, (const void*)key, (size_t)*key);
+}
+
+static inline bool shl_htable_remove_ulong(struct shl_htable *htable,
+                                          unsigned long key,
+                                          unsigned long **out)
+{
+       return shl_htable_remove(htable, (const void*)&key, (size_t)key,
+                                (void**)out);
+}
+
+/* string htables */
+
+bool shl_htable_compare_str(const void *a, const void *b);
+size_t shl_htable_rehash_str(const void *elem, void *priv);
+
+#define SHL_HTABLE_INIT_STR(_obj)                                      \
+       SHL_HTABLE_INIT((_obj), shl_htable_compare_str,                 \
+                               shl_htable_rehash_str,                  \
+                               NULL)
+
+static inline void shl_htable_init_str(struct shl_htable *htable)
+{
+       shl_htable_init(htable, shl_htable_compare_str,
+                       shl_htable_rehash_str, NULL);
+}
+
+static inline void shl_htable_clear_str(struct shl_htable *htable,
+                                       void (*cb) (char **elem,
+                                                   void *ctx),
+                                       void *ctx)
+{
+       shl_htable_clear(htable, (void (*) (void*, void*))cb, ctx);
+}
+
+static inline void shl_htable_visit_str(struct shl_htable *htable,
+                                       void (*cb) (char **elem,
+                                                   void *ctx),
+                                       void *ctx)
+{
+       shl_htable_visit(htable, (void (*) (void*, void*))cb, ctx);
+}
+
+static inline size_t shl_htable_hash_str(struct shl_htable *htable,
+                                        const char *str, size_t *hash)
+{
+       size_t h;
+
+       if (hash && *hash) {
+               h = *hash;
+       } else {
+               h = htable->htable.rehash((const void*)&str, NULL);
+               if (hash)
+                       *hash = h;
+       }
+
+       return h;
+}
+
+static inline bool shl_htable_lookup_str(struct shl_htable *htable,
+                                        const char *str, size_t *hash,
+                                        char ***out)
+{
+       size_t h;
+
+       h = shl_htable_hash_str(htable, str, hash);
+       return shl_htable_lookup(htable, (const void*)&str, h, (void**)out);
+}
+
+static inline int shl_htable_insert_str(struct shl_htable *htable,
+                                       char **str, size_t *hash)
+{
+       size_t h;
+
+       h = shl_htable_hash_str(htable, *str, hash);
+       return shl_htable_insert(htable, (const void*)str, h);
+}
+
+static inline bool shl_htable_remove_str(struct shl_htable *htable,
+                                        const char *str, size_t *hash,
+                                        char ***out)
+{
+       size_t h;
+
+       h = shl_htable_hash_str(htable, str, hash);
+       return shl_htable_remove(htable, (const void*)&str, h, (void **)out);
+}
+
+#endif /* SHL_HTABLE_H */
index eb061a2..874915c 100644 (file)
@@ -61,7 +61,7 @@
 #include "libtsm.h"
 #include "libtsm_int.h"
 #include "shl_array.h"
-#include "shl_hashtable.h"
+#include "shl_htable.h"
 
 /*
  * Unicode Symbol Handling
@@ -103,13 +103,12 @@ struct tsm_symbol_table {
        unsigned long ref;
        uint32_t next_id;
        struct shl_array *index;
-       struct shl_hashtable *symbols;
+       struct shl_htable symbols;
 };
 
-static unsigned int hash_ucs4(const void *key)
+static size_t hash_ucs4(const void *key, void *priv)
 {
-       unsigned int val = 5381;
-       size_t i;
+       size_t i, val = 5381;
        const uint32_t *ucs4 = key;
 
        for (i = 0; ucs4[i] <= TSM_UCS4_MAX; ++i)
@@ -134,6 +133,14 @@ static bool cmp_ucs4(const void *a, const void *b)
        }
 }
 
+static void free_ucs4(void *elem, void *priv)
+{
+       uint32_t *v = elem;
+
+       /* key is prefix with actual value so pass correct pointer */
+       free(--v);
+}
+
 int tsm_symbol_table_new(struct tsm_symbol_table **out)
 {
        struct tsm_symbol_table *tbl;
@@ -149,6 +156,7 @@ int tsm_symbol_table_new(struct tsm_symbol_table **out)
        memset(tbl, 0, sizeof(*tbl));
        tbl->ref = 1;
        tbl->next_id = TSM_UCS4_MAX + 2;
+       shl_htable_init(&tbl->symbols, cmp_ucs4, hash_ucs4, NULL);
 
        ret = shl_array_new(&tbl->index, sizeof(uint32_t*), 4);
        if (ret)
@@ -157,16 +165,9 @@ int tsm_symbol_table_new(struct tsm_symbol_table **out)
        /* first entry is not used so add dummy */
        shl_array_push(tbl->index, &val);
 
-       ret = shl_hashtable_new(&tbl->symbols, hash_ucs4, cmp_ucs4,
-                               free, NULL);
-       if (ret)
-               goto err_array;
-
        *out = tbl;
        return 0;
 
-err_array:
-       shl_array_free(tbl->index);
 err_free:
        free(tbl);
        return ret;
@@ -185,7 +186,7 @@ void tsm_symbol_table_unref(struct tsm_symbol_table *tbl)
        if (!tbl || !tbl->ref || --tbl->ref)
                return;
 
-       shl_hashtable_free(tbl->symbols);
+       shl_htable_clear(&tbl->symbols, free_ucs4, NULL);
        shl_array_free(tbl->index);
        free(tbl);
 }
@@ -250,7 +251,6 @@ tsm_symbol_t tsm_symbol_append(struct tsm_symbol_table *tbl,
        uint32_t buf[TSM_UCS4_MAXLEN + 1], nsym, *nval;
        const uint32_t *ptr;
        size_t s;
-       void *tmp;
        bool res;
        int ret;
 
@@ -268,22 +268,32 @@ tsm_symbol_t tsm_symbol_append(struct tsm_symbol_table *tbl,
        buf[s++] = ucs4;
        buf[s++] = TSM_UCS4_MAX + 1;
 
-       res = shl_hashtable_find(tbl->symbols, &tmp, buf);
-       if (res)
-               return (uint32_t)(long)tmp;
+       res = shl_htable_lookup(&tbl->symbols, buf, hash_ucs4(buf, NULL),
+                               (void**)&nval);
+       if (res) {
+               /* key is prefixed with actual value */
+               return *--nval;
+       }
 
-       nval = malloc(sizeof(uint32_t) * s);
+       /* We save the key in nval and prefix it with the new ID. Note that
+        * the prefix is hidden, we actually store "++nval" in the htable. */
+       nval = malloc(sizeof(uint32_t) * (s + 1));
        if (!nval)
                return sym;
 
+       ++nval;
        memcpy(nval, buf, s * sizeof(uint32_t));
+
        nsym = tbl->next_id + 1;
        /* Out of IDs; we actually have 2 Billion IDs so this seems
         * very unlikely but lets be safe here */
        if (nsym <= tbl->next_id++)
                goto err_id;
 
-       ret = shl_hashtable_insert(tbl->symbols, nval, (void*)(long)nsym);
+       /* store ID hidden before the key */
+       *(nval - 1) = nsym;
+
+       ret = shl_htable_insert(&tbl->symbols, nval, hash_ucs4(nval, NULL));
        if (ret)
                goto err_id;
 
@@ -294,7 +304,7 @@ tsm_symbol_t tsm_symbol_append(struct tsm_symbol_table *tbl,
        return nsym;
 
 err_symbol:
-       shl_hashtable_remove(tbl->symbols, nval);
+       shl_htable_remove(&tbl->symbols, nval, hash_ucs4(nval, NULL), NULL);
 err_id:
        --tbl->next_id;
        free(nval);