diff options
| author | jadahl@opera.com <jadahl@opera.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-07 22:06:38 +0000 |
|---|---|---|
| committer | jadahl@opera.com <jadahl@opera.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-07 22:06:38 +0000 |
| commit | dede71ec98340e92fa490efc64a0153adb848256 (patch) | |
| tree | 2f6bae016da531d8f46c20167043780e33557c7d /base/bind.h.pump | |
| parent | a8eab3bb4993c6db09e963a436a305d07e327e1e (diff) | |
| download | chromium_src-dede71ec98340e92fa490efc64a0153adb848256.zip chromium_src-dede71ec98340e92fa490efc64a0153adb848256.tar.gz chromium_src-dede71ec98340e92fa490efc64a0153adb848256.tar.bz2 | |
base: Don't generate unused BoundFunctorTraits typedef
Having it there causes a warning or error (unused-local-typedefs).
BUG=
Review URL: https://codereview.chromium.org/133553003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249808 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/bind.h.pump')
| -rw-r--r-- | base/bind.h.pump | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/base/bind.h.pump b/base/bind.h.pump index b321649..2cdd7d5 100644 --- a/base/bind.h.pump +++ b/base/bind.h.pump @@ -93,14 +93,14 @@ $if ARITY > 0 [[, ]] $for ARG , [[const P$(ARG)& p$(ARG)]]) { typedef typename internal::FunctorTraits<Functor>::RunnableType RunnableType; typedef typename internal::FunctorTraits<Functor>::RunType RunType; +$if ARITY > 0 [[ + // Use RunnableType::RunType instead of RunType above because our // checks should below for bound references need to know what the actual // functor is going to interpret the argument as. typedef internal::FunctionTraits<typename RunnableType::RunType> BoundFunctorTraits; -$if ARITY > 0 [[ - // Do not allow binding a non-const reference parameter. Non-const reference // parameters are disallowed by the Google style guide. Also, binding a // non-const reference parameter can make for subtle bugs because the |