diff options
author | 2013-11-18 16:03:59 +0000 | |
---|---|---|
committer | 2013-11-18 16:03:59 +0000 | |
commit | e61a86cfa00ea393ecc4a71fca94e1d476a37ecc (patch) | |
tree | f6ec6c793f279f8226ea40283e707cf3c716e12b /src/core/SkGlyphCache.h | |
parent | 8bf4c0ab7b4acd25566459fc464027c5760d0e5e (diff) |
Guard against most unintentionally ephemeral SkAutoFoo instantiations.
I think I applied the trick everywhere possible. Limitations:
- can't be used with templated classes
- all constructors and destructors must be defined inline
A couple of the SkAutoFoo were unused in Skia, Chromium, and Android, so I
deleted them. This change caught the same bugs Cary found in SkPath, plus one
more in SampleApp.
BUG=
R=reed@google.com, caryclark@google.com
Author: mtklein@google.com
Review URL: https://codereview.chromium.org/72603005
git-svn-id: http://skia.googlecode.com/svn/trunk@12301 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'src/core/SkGlyphCache.h')
-rw-r--r-- | src/core/SkGlyphCache.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/core/SkGlyphCache.h b/src/core/SkGlyphCache.h index 8447337374..52a8132f72 100644 --- a/src/core/SkGlyphCache.h +++ b/src/core/SkGlyphCache.h @@ -273,5 +273,6 @@ private: static bool DetachProc(const SkGlyphCache*, void*); }; +#define SkAutoGlyphCache(...) SK_REQUIRE_LOCAL_VAR(SkAutoGlyphCache) #endif |