diff options
| author | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-23 01:10:46 +0000 |
|---|---|---|
| committer | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-23 01:10:46 +0000 |
| commit | b8c1eb454dc4f246c64a6b1b5b472af102e88103 (patch) | |
| tree | 6413d1400ee4ade44838ecd530736840d1f4f64b /net/tools | |
| parent | be2da4de5227f4f1cbb8455a58045c5e51076474 (diff) | |
| download | chromium_src-b8c1eb454dc4f246c64a6b1b5b472af102e88103.zip chromium_src-b8c1eb454dc4f246c64a6b1b5b472af102e88103.tar.gz chromium_src-b8c1eb454dc4f246c64a6b1b5b472af102e88103.tar.bz2 | |
base: Add IsHexDigit function to string_util.h
Removed duplicated IsHex functions and converted the callers along the way.
(Note: this was a TODO for jungshik).
BUG=None
TEST=trybots
Signed-off-by: Thiago Farina <tfarina@chromium.org>
Review URL: http://codereview.chromium.org/2870058
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53428 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/tools')
| -rw-r--r-- | net/tools/dump_cache/url_to_filename_encoder.cc | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/net/tools/dump_cache/url_to_filename_encoder.cc b/net/tools/dump_cache/url_to_filename_encoder.cc index 89a1ca4..1462faa 100644 --- a/net/tools/dump_cache/url_to_filename_encoder.cc +++ b/net/tools/dump_cache/url_to_filename_encoder.cc @@ -11,18 +11,14 @@ using std::string; namespace { -inline bool IsHexDigit(unsigned char c) { - return (('0' <= c && c <= '9') || ('A' <= c && c <= 'F') || - ('a' <= c && c <= 'f')); -} - // Returns 1 if buf is prefixed by "num_digits" of hex digits // Teturns 0 otherwise. // The function checks for '\0' for string termination. int HexDigitsPrefix(const char* buf, int num_digits) { - for (int i = 0; i < num_digits; i++) + for (int i = 0; i < num_digits; i++) { if (!IsHexDigit(buf[i])) return 0; // This also detects end of string as '\0' is not xdigit. + } return 1; } |