|
1833a686
|
2023-09-20T16:21:21
|
|
Fix FixedVector bugs and unit tests
Problems:
- test was not testing assignment operators;
- fixed compilation error in r-value assignment operator;
- r-value constructor/assignment were not resetting size.
Additionally updated FixedVector.Constructors to better test copy and
assignment operations.
Bug: angleproject:2435
Bug: angleproject:8127
Change-Id: Ic501b8d85af0280801c7abec8fb20a0ddf67580b
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/4874039
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Commit-Queue: Igor Nazarov <i.nazarov@samsung.com>
Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
|
|
0ce016c1
|
2018-04-09T10:59:56
|
|
FixedVector: Add "full" method.
Will be useful for an optimization to the Buffer Subject/Observer
pattern.
Also cleans up an ASSERT.
Bug: angleproject:2389
Bug: chromium:829906
Change-Id: I2f8313ab531bca61947a51cc2396c04fb5d4bb1d
Reviewed-on: https://chromium-review.googlesource.com/1002883
Reviewed-by: Luc Ferron <lucferron@chromium.org>
Reviewed-by: Lingfeng Yang <lfy@google.com>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Jamie Madill <jmadill@chromium.org>
|
|
8957e832
|
2018-04-06T10:49:28
|
|
Add pop_back() / back() method for FixedVector
Good for implementing stacks of fixed size (in the pop operation).
+ fix an issue where push_back() only allows adding up to max_size() - 1
elements.
BUG=angleproject:2306
Change-Id: I99b3c8416055f069fa563b684c102fd6aec116bc
Reviewed-on: https://chromium-review.googlesource.com/1000031
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Lingfeng Yang <lfy@google.com>
|
|
2231b4e0
|
2018-03-26T16:44:59
|
|
Add a FixedVector class to have "variable" size vectors on the stack.
Wraps a std::array and a size parameter to give the std::vector interface
without making allocations.
BUG=angleproject:2435
Change-Id: I7df0be1310446a2f163766149bf631a8692be9ad
Reviewed-on: https://chromium-review.googlesource.com/981267
Commit-Queue: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
|