Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(367)

Unified Diff: include/core/SkString.h

Issue 16356006: change impl to explicitly store length as u32, rather than size_t, so we keep a packed struct on 64… (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/core/SkString.cpp » ('j') | src/core/SkString.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include/core/SkString.h
diff --git a/include/core/SkString.h b/include/core/SkString.h
index 743f07889d079c9d41e2644ee4f1c078978671c0..b98e2c8c597763c99ca31c8e4ce7a3275a9740a6 100644
--- a/include/core/SkString.h
+++ b/include/core/SkString.h
@@ -205,7 +205,7 @@ public:
private:
struct Rec {
public:
- size_t fLength;
+ uint32_t fLength; // logically size_t, but we want it to stay 32bits
int32_t fRefCnt;
char fBeginningOfData;
« no previous file with comments | « no previous file | src/core/SkString.cpp » ('j') | src/core/SkString.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698