This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[review] Add a fast_hash function in common-utils
- From: "Christian Biesinger (Code Review)" <gerrit at gnutoolchain-gerrit dot osci dot io>
- To: Christian Biesinger <cbiesinger at google dot com>, gdb-patches at sourceware dot org
- Cc: Sergio Durigan Junior <sergiodj at redhat dot com>
- Date: Wed, 16 Oct 2019 09:08:54 -0400
- Subject: [review] Add a fast_hash function in common-utils
- Auto-submitted: auto-generated
- References: <gerrit.1571050562000.I77cac0d9aa78fc65316a2af449f52edcae72dc9b@gnutoolchain-gerrit.osci.io>
- Reply-to: cbiesinger at google dot com, sergiodj at redhat dot com, cbiesinger at google dot com, gdb-patches at sourceware dot org
Christian Biesinger has posted comments on this change.
Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/38
......................................................................
Uploaded patch set 2.
(2 comments)
https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/38/1/gdb/utils.h
File gdb/utils.h:
https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/38/1/gdb/utils.h@572
PS1, Line 572: extern unsigned int fast_hash (const char* str, size_t len);
> You need a comment describing what the function does here.
Done
https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/38/1/gdb/utils.c
File gdb/utils.c:
https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/38/1/gdb/utils.c@3439
PS1, Line 3439: unsigned int
> And you need a simple comment here saying […]
Done