This is a bit cleaner and avoids borrow checking problems.
Depends on D11416
Diff Detail
Diff Detail
- Repository
- rMOZILLACENTRAL mozilla-central
- Branch
- HEAD
- Lint
No Linters Available - Unit
No Unit Test Coverage
Event Timeline
| gfx/webrender/src/texture_cache.rs | ||
|---|---|---|
| 278 | nit: Having this as an enum instead of a bool would make it easier to read calling code. | |
Comment Actions
Thanks for the reviews!
| gfx/webrender/src/texture_cache.rs | ||
|---|---|---|
| 278 | Heh, I considered doing this before, but we already had an EntryKind, which also contained details in the cache case. I've gone ahead and added two more patches on top of this stack, one to rename EntryKind to EntryDetails, and another to introduce a detail-less EntryKind, which I've used for this logic. Code is much nicer now. :-) | |