| 8fe24272fa6d2fa9eb2458221ed9852d6ec16f56 |
|
07-Jul-2017 |
Brian Salomon <bsalomon@google.com> |
Add mock config to tools and run through gms and benchs without crashing.
Change-Id: I7e2474129ef2b15899ad2baeb8d18f39d05da98c
Reviewed-on: https://skia-review.googlesource.com/21820
Commit-Queue: Brian Salomon <bsalomon@google.com>
Reviewed-by: Greg Daniel <egdaniel@google.com>
/external/skia/include/private/SkTHash.h
|
| 053730de732922ecee21be665af111ce671ce045 |
|
10-Mar-2017 |
Florin Malita <fmalita@chromium.org> |
Fix SkTHashTable dangling values
The element rearrange logic in SkTHashTable::remove() marks empty slots
as such, but does not reset their value.
When breaking out of the rearrange loop, we must also reset the last empty
slot value to avoid retaining unwanted copies.
Change-Id: I8ba2a25088c0aa5210277124e0917224cb295691
Reviewed-on: https://skia-review.googlesource.com/9533
Reviewed-by: Mike Klein <mtklein@chromium.org>
Reviewed-by: Ben Wagner <bungeman@google.com>
Commit-Queue: Florin Malita <fmalita@chromium.org>
/external/skia/include/private/SkTHash.h
|
| 6b00a07d4fa5f9fd2feb0fc50adac0ce6a477e41 |
|
15-Dec-2016 |
Mike Klein <mtklein@chromium.org> |
simplify SkTHash* move support
Don't know why I thought this had to be so complicated before.
BUG=skia:6053
Change-Id: Ie714fed1cb47e9add166d4227d3d31f95eba2411
Reviewed-on: https://skia-review.googlesource.com/6121
Reviewed-by: Hal Canary <halcanary@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
/external/skia/include/private/SkTHash.h
|
| acef51fdd870ca48301a27f0dda30b013f34d540 |
|
14-Dec-2016 |
Herb Derby <herb@google.com> |
Remove tombstones from SkTHash.
* Switch to linear probing - this allows delete to rearrange elements to fill in empty slots
* NULL -> nullptr
Change-Id: I741c2f3bb2734bf638d0c0a78c6cc549f563a5d9
Reviewed-on: https://skia-review.googlesource.com/5980
Commit-Queue: Herb Derby <herb@google.com>
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Mike Klein <mtklein@chromium.org>
/external/skia/include/private/SkTHash.h
|
| db402cab8b23f2f912f880af8be9f51605ad2bc7 |
|
13-Dec-2016 |
Mike Klein <mtklein@chromium.org> |
add move semantics to SkTHash*
The more I look at std::unordered_map and co., the less I like them.
I think we might want to bet on SkTHash*.
As a simple first improvement, add move support.
Next comes shrinking, and then I'll start moving over SkTDynamicHash users.
BUG=skia:6053
Change-Id: Ifdb5d713aab66434ca271c7f18a0cbbb0720099c
Reviewed-on: https://skia-review.googlesource.com/5943
Commit-Queue: Mike Klein <mtklein@chromium.org>
Reviewed-by: Herb Derby <herb@google.com>
Reviewed-by: Hal Canary <halcanary@google.com>
/external/skia/include/private/SkTHash.h
|
| c8d1dd48c01f562cfffe64da0e5cee5ed2713541 |
|
15-Oct-2015 |
mtklein <mtklein@chromium.org> |
SkTHash: hash from fnptr to functor type
Passing &SkGoodHash to SkTHashMap and SkTHashSet doesn't guarantee that it's actually instantiated. Using a functor does.
BUG=skia:
Review URL: https://codereview.chromium.org/1405053002
/external/skia/include/private/SkTHash.h
|
| 385fe4d4b62d7d1dd76116dd570df3290a2f487b |
|
26-Aug-2015 |
halcanary <halcanary@google.com> |
Style Change: SkNEW->new; SkDELETE->delete
DOCS_PREVIEW= https://skia.org/?cl=1316123003
Review URL: https://codereview.chromium.org/1316123003
/external/skia/include/private/SkTHash.h
|
| 469a3fe6edb3fb29acf6c03de662a6f00f6804b8 |
|
07-Aug-2015 |
mtklein <mtklein@chromium.org> |
Add approxBytesUsed to hashes.
BUG=skia:
Review URL: https://codereview.chromium.org/1280653003
/external/skia/include/private/SkTHash.h
|
| fd8ed69447f2a126f7006a1a95356f15feca3797 |
|
28-Jul-2015 |
mtklein <mtklein@chromium.org> |
Move SkTHash.h to include/private.
include/views/SkOSWindow_Win.h includes it.
To move SkTHash.h to include/private, SkChecksum.h needs to go there too. To move SkChecksum.h to include/private, SkTLogic needs to go there too.
This adds a bunch of -Iinclude/private to tools.gyp I missed in the last CL.
No public API changes.
TBR=reed@google.com
BUG=skia:4126
Review URL: https://codereview.chromium.org/1260613006
/external/skia/include/private/SkTHash.h
|
| 478c9e4851eff5feb74dd2f8a14143bec569e1a8 |
|
01-Jul-2015 |
Mike Klein <mtklein@google.com> |
Revert "Move headers used by headers in include/ to include/private."
This reverts commit 928e16565f3e69cd6661e9ad300ac17e3f33c0c8.
BUG=skia:
Review URL: https://codereview.chromium.org/1213093004.
/external/skia/include/private/SkTHash.h
|
| 928e16565f3e69cd6661e9ad300ac17e3f33c0c8 |
|
01-Jul-2015 |
mtklein <mtklein@chromium.org> |
Move headers used by headers in include/ to include/private.
Some of this is transitive, like SkRecords.h used by SkMiniRecorder.h
used by (public) SkPictureRecorder.h.
BUG=skia:
Committed: https://skia.googlesource.com/skia/+/a89f55198bdc58f0b6f6196907ab25a6afc1a661
Review URL: https://codereview.chromium.org/1217293004
/external/skia/include/private/SkTHash.h
|
| 4417c7f8bb85aa1eae536cc50c70c0cc87f31171 |
|
01-Jul-2015 |
jvanverth <jvanverth@google.com> |
Revert "Move headers used by headers in include/ to include/private."
This reverts commit a89f55198bdc58f0b6f6196907ab25a6afc1a661.
Reason: breaking the roll.
TBR=mtklein@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
Review URL: https://codereview.chromium.org/1216033008
/external/skia/include/private/SkTHash.h
|
| a89f55198bdc58f0b6f6196907ab25a6afc1a661 |
|
01-Jul-2015 |
mtklein <mtklein@chromium.org> |
Move headers used by headers in include/ to include/private.
Some of this is transitive, like SkRecords.h used by SkMiniRecorder.h
used by (public) SkPictureRecorder.h.
BUG=skia:
Review URL: https://codereview.chromium.org/1217293004
/external/skia/include/private/SkTHash.h
|