diff options
author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-09-01 00:46:33 +0000 |
---|---|---|
committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-09-01 00:46:33 +0000 |
commit | d643172a8573e7e75f325bec2125de8071bfd0fc (patch) | |
tree | 16d43bab4652624b87e1e7acfbecb63310a6e736 /net/proxy/proxy_config.h | |
parent | f5e7e8e162433ab6e8223c8b3da4feb64abddc33 (diff) | |
download | chromium_src-d643172a8573e7e75f325bec2125de8071bfd0fc.zip chromium_src-d643172a8573e7e75f325bec2125de8071bfd0fc.tar.gz chromium_src-d643172a8573e7e75f325bec2125de8071bfd0fc.tar.bz2 |
Reduce number of unnamed-type-template-args violations (mostly when passing values to DCHECK(), ASSERT_EQ(), etc.), generally by naming previously-anonymous enums. We've decided not to eliminate the warning entirely because doing so is only possible with tons of ugly static_cast<>()s in Mac code.
BUG=92247
TEST=Compiles
Review URL: http://codereview.chromium.org/7605019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@99086 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/proxy/proxy_config.h')
-rw-r--r-- | net/proxy/proxy_config.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/proxy/proxy_config.h b/net/proxy/proxy_config.h index 0c353ce..f666e93 100644 --- a/net/proxy/proxy_config.h +++ b/net/proxy/proxy_config.h @@ -110,7 +110,7 @@ class NET_EXPORT ProxyConfig { typedef int ID; // Indicates an invalid proxy config. - enum { INVALID_ID = 0 }; + static const ID kInvalidConfigID = 0; ProxyConfig(); ProxyConfig(const ProxyConfig& config); @@ -120,7 +120,7 @@ class NET_EXPORT ProxyConfig { // Used to numerically identify this configuration. ID id() const { return id_; } void set_id(int id) { id_ = id; } - bool is_valid() const { return id_ != INVALID_ID; } + bool is_valid() const { return id_ != kInvalidConfigID; } // Returns true if the given config is equivalent to this config. bool Equals(const ProxyConfig& other) const; |