diff options
author | ericroman@google.com <ericroman@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-12-02 06:59:09 +0000 |
---|---|---|
committer | ericroman@google.com <ericroman@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-12-02 06:59:09 +0000 |
commit | e34c85d80d946862ae594ee720a2baf6e62746d5 (patch) | |
tree | 2de970a953485b1cbe89f787a9476618271008c4 /net/http/http_auth_handler.cc | |
parent | 96fbab401bc5dcfe7fab1f5b99c1629f05fde5ab (diff) | |
download | chromium_src-e34c85d80d946862ae594ee720a2baf6e62746d5.zip chromium_src-e34c85d80d946862ae594ee720a2baf6e62746d5.tar.gz chromium_src-e34c85d80d946862ae594ee720a2baf6e62746d5.tar.bz2 |
Minor refactorings:
- Change scheme from a (const char*) to a (std::string). This is in response to an earlier comment.
Background:
It used to be a (const char*) to avoid allocating a string object since all the callers initialize it with a string literal.
- Change the initial value of nonce_count_ from 1 to 0. This doesn't functionally change anything (we just change to a pre-increment rather than post-increment), but I prefer the the start-at-zero style.
Review URL: http://codereview.chromium.org/12846
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@6208 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/http/http_auth_handler.cc')
-rw-r--r-- | net/http/http_auth_handler.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/net/http/http_auth_handler.cc b/net/http/http_auth_handler.cc index b0a7b97..dd9482f 100644 --- a/net/http/http_auth_handler.cc +++ b/net/http/http_auth_handler.cc @@ -12,7 +12,6 @@ bool HttpAuthHandler::InitFromChallenge(std::string::const_iterator begin, HttpAuth::Target target) { target_ = target; score_ = -1; - scheme_ = NULL; bool ok = Init(begin, end); |