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

Unified Diff: src/core/SkVarAlloc.h

Issue 718203006: Revert of Deparameterize SkVarAlloc. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 6 years, 1 month 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 | « src/core/SkRecord.h ('k') | src/core/SkVarAlloc.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/core/SkVarAlloc.h
diff --git a/src/core/SkVarAlloc.h b/src/core/SkVarAlloc.h
index 19b2d75e76ca2467eb1a38eb55df269b0a548049..0a7864b37a878e265c7e12eccdee4ee03123d331 100644
--- a/src/core/SkVarAlloc.h
+++ b/src/core/SkVarAlloc.h
@@ -5,21 +5,22 @@
class SkVarAlloc : SkNoncopyable {
public:
- SkVarAlloc();
+ // SkVarAlloc will never allocate less than smallest bytes at a time.
+ // When it allocates a new block, it will be at least growth times bigger than the last.
+ SkVarAlloc(size_t smallest, float growth);
~SkVarAlloc();
// Returns contiguous bytes aligned at least for pointers. You may pass SK_MALLOC_THROW, etc.
char* alloc(size_t bytes, unsigned sk_malloc_flags) {
bytes = SkAlignPtr(bytes);
- if (bytes > fRemaining) {
+ if (fByte + bytes > fLimit) {
this->makeSpace(bytes, sk_malloc_flags);
}
- SkASSERT(bytes <= fRemaining);
+ SkASSERT(fByte + bytes <= fLimit);
char* ptr = fByte;
fByte += bytes;
- fRemaining -= bytes;
return ptr;
}
@@ -27,8 +28,10 @@
void makeSpace(size_t bytes, unsigned flags);
char* fByte;
- unsigned fRemaining;
- unsigned fLgMinSize;
+ const char* fLimit;
+
+ unsigned fSmallest;
+ const float fGrowth;
struct Block;
Block* fBlock;
« no previous file with comments | « src/core/SkRecord.h ('k') | src/core/SkVarAlloc.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698