Age | Commit message (Collapse) | Author |
|
Change-Id: I018123259cd231d0e74933e492391999e36650e8
|
|
Bug: 277108894
Change-Id: Ibe7074ff8303640505452fd91a9eea2b0e2afaa4
|
|
Change-Id: Iba357b308a79d0c8b560acd4f72b5423c9c83294
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/181031
Reviewed-by: Kayce Basques <kayce@google.com>
Commit-Queue: Chad Norvell <chadnorvell@google.com>
|
|
This reverts commit 50d8c114135f26bd613baa99615e685a869b50b2.
Reason for revert: b/309680612 races manifesting on macOS.
Original change's description:
> pw_rpc_transport: Close sockets when stopping
>
> Also, when closing sockets, disconnect the underlying sockets by using
> the socket shutdown API. This unblocks socket recv and accept calls.
>
> Bug: 309680612
> Test: Verified socket unit tests pass.
> Test: See details in testing done comment in the code review.
> Change-Id: If9e54bdb88fa36a3f831d32b4549d5a222fd3df8
> Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179591
> Reviewed-by: Carlos Chinchilla <cachinchilla@google.com>
> Commit-Queue: Erik Staats <estaats@google.com>
> Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
# Not skipping CQ checks because original CL landed > 1 day ago.
Bug: 309680612
Change-Id: Ie8f0bc3d6665c9c11c680a964bb6842b5d570c43
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/181130
Pigweed-Auto-Submit: Rob Mohr <mohrr@google.com>
Commit-Queue: Rob Mohr <mohrr@google.com>
Reviewed-by: Rob Mohr <mohrr@google.com>
Reviewed-by: Armando Montanez <amontanez@google.com>
Reviewed-by: Erik Staats <estaats@google.com>
|
|
As the first step of implementing toolchains as proposed in SEED-0113,
this CL introduces pw_cc_flag_set and pw_cc_flag_group. These will be
the core building blocks for specifying flags that will be bound to
toolchains.
As an illustrative example, moves the warning "features" to be
pw_cc_flag_set rules instead.
Bug: b/309533028
Change-Id: I3eaa6ac92511d2973bd5120432878d6953d14793
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179932
Reviewed-by: Ted Pudlik <tpudlik@google.com>
Commit-Queue: Armando Montanez <amontanez@google.com>
Reviewed-by: Kayce Basques <kayce@google.com>
|
|
Change-Id: Ie3ee19f174b5ce02782b65a17c112f954420357c
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/181030
Pigweed-Auto-Submit: Erik Gilling <konkers@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Reviewed-by: Taylor Cramer <cramertj@google.com>
|
|
- First cut at our mission statement
- Sets the stage to gradually add philosophies
Change-Id: I93dffbb56c7f4bbc91bcc7ba788ce65cd5db2b19
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/178910
Reviewed-by: Kayce Basques <kayce@google.com>
Commit-Queue: Keir Mierle <keir@google.com>
|
|
This applies the same fix from pwrev/180826 to another test where the
issue surfaced.
Bug: 297355578
Change-Id: Ic0411a986c59ba58c03fd7803236a076978e4cd3
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180951
Reviewed-by: Ted Pudlik <tpudlik@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Pigweed-Auto-Submit: Alexei Frolov <frolv@google.com>
|
|
Without this, we would get undefined reference errors.
Test: Include "@pigweed//pw_toolchain/arm_gcc:arm_none_eabi_gcc_support"
in cc_binary() deps
Test: Verify __wrap_abort(), __wrap_sread() are used in disasm
Change-Id: I20a600bc787b18cdf3b5b64dd0fc13dc7ae65e61
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180930
Reviewed-by: Ted Pudlik <tpudlik@google.com>
Commit-Queue: Jonathon Reinhart <jrreinhart@google.com>
|
|
This reverts commit 8a42e20a49f2810d8823cd057c81b6b65c970702.
Reason for revert: Broke Mac x86 builders
Original change's description:
> bazel: Use Python 3.11
>
> This matches what we do in GN already.
>
> Also, change the WORKSPACE name from "python3_10" to just "python3"
> (rather than "python3_11"), so that these upgrades don't require us to
> change names all over the place (including in downstream!).
>
> Fixes: b/310293060
> Change-Id: I3479f52a2d8a46850af8a65e14472bc4b6ee1b0d
> Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180534
> Reviewed-by: Armando Montanez <amontanez@google.com>
> Commit-Queue: Ted Pudlik <tpudlik@google.com>
TBR=amontanez@google.com,pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com,tonymd@google.com,tpudlik@google.com
Change-Id: Icc0497f62278dae5af0a0ad6b2abce3a946e51de
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180914
Reviewed-by: Rob Mohr <mohrr@google.com>
Commit-Queue: Ted Pudlik <tpudlik@google.com>
Reviewed-by: Ted Pudlik <tpudlik@google.com>
|
|
Fixes: b/310658991
Change-Id: Id1a8d180e283d2b1ea217b65afb0d6a964069c47
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180871
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Pigweed-Auto-Submit: Ted Pudlik <tpudlik@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Armando Montanez <amontanez@google.com>
|
|
This matches what we do in GN already.
Also, change the WORKSPACE name from "python3_10" to just "python3"
(rather than "python3_11"), so that these upgrades don't require us to
change names all over the place (including in downstream!).
Fixes: b/310293060
Change-Id: I3479f52a2d8a46850af8a65e14472bc4b6ee1b0d
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180534
Reviewed-by: Armando Montanez <amontanez@google.com>
Commit-Queue: Ted Pudlik <tpudlik@google.com>
|
|
The manual cancellation unit test was requesting that the client send
all of its available data, which could sometimes lead to flakes as it
would initiate the closing handshake early. This updates the test to
only prompt the client for a smaller data chunk.
Change-Id: I2ba339fe017431c85714fd7a122c695254770915
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180826
Reviewed-by: Keir Mierle <keir@google.com>
Pigweed-Auto-Submit: Alexei Frolov <frolv@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Reviewed-by: Carlos Chinchilla <cachinchilla@google.com>
|
|
- Add docs banner to main page and subpages
- Fix headers to match style guide on main page
Change-Id: I020e514af38b3f378e1704e5322781dc14e4bb16
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180824
Commit-Queue: Keir Mierle <keir@google.com>
Reviewed-by: Kayce Basques <kayce@google.com>
|
|
Change-Id: Iebb3622b1c4c48d7530b9f23cbc2b32aa0356c74
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180815
Pigweed-Auto-Submit: Ted Pudlik <tpudlik@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Reviewed-by: Jonathon Reinhart <jrreinhart@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
|
|
Adds a sample implementation of the trace service with built on top of
the flat filesystem and transfer services. The trace data is backed by a
persistent buffer, again as an sample implementation.
Also fixes an issue with pw_transfer failing to compile on systems with
different periods.
Note:
- the bazel build doesn't link due to tracing not being supported in the
bazel build(b/260641850), so tracing is disabled in the sample bazel build.
- the cmake build doesn't compile, but it was updated we new targets so
as not to dig the hole any deeper.
Change-Id: I483d049d486647c10be3db8f104ea2419a31a312
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/168834
Commit-Queue: Dave Roth <davidroth@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Armando Montanez <amontanez@google.com>
Reviewed-by: Alexei Frolov <frolv@google.com>
|
|
Change-Id: I3e186fcedebf60ab3101fd8a0633403e15db8652
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/178132
Reviewed-by: Taylor Cramer <cramertj@google.com>
Commit-Queue: Erik Gilling <konkers@google.com>
|
|
From: version:2@1.21.3
To: version:2@1.21.4
Roller-URL: https://ci.chromium.org/b/8764545579011708097
CQ-Do-Not-Cancel-Tryjobs: true
Change-Id: I61d644efa16047aab59fbd69b958867e3e720af0
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180672
Bot-Commit: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
Commit-Queue: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
|
|
From: git_revision:050c2211536b89e0ec9e419cef91c7bc60b339ab
To: git_revision:0280acaad36a2544c150e1246cb344a1b6e6a02d
Roller-URL: https://ci.chromium.org/b/8764598302505471121
CQ-Do-Not-Cancel-Tryjobs: true
Change-Id: I5042b257f1d718474e672b6d3a0128c3500d6a35
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180691
Commit-Queue: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
Bot-Commit: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
|
|
310
From: git_revision:88aad83343c8b7ae5a24b1f0849675b4cbb6cc5c
To: git_revision:9c0316b7067a011583fe245fa764bba382f92d43
311
From: git_revision:88aad83343c8b7ae5a24b1f0849675b4cbb6cc5c
To: git_revision:9c0316b7067a011583fe245fa764bba382f92d43
38
From: git_revision:88aad83343c8b7ae5a24b1f0849675b4cbb6cc5c
To: git_revision:9c0316b7067a011583fe245fa764bba382f92d43
39
From: git_revision:88aad83343c8b7ae5a24b1f0849675b4cbb6cc5c
To: git_revision:9c0316b7067a011583fe245fa764bba382f92d43
Roller-URL: https://ci.chromium.org/b/8764673609459836401
CQ-Do-Not-Cancel-Tryjobs: true
Change-Id: I02362cf14fbf975f191e1b391da864f93ccb9364
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180650
Bot-Commit: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
Commit-Queue: Pigweed Roller <pigweed-roller@pigweed-service-accounts.iam.gserviceaccount.com>
|
|
Bug: 309680612
Test: Verified sample-project-default-build-windows succeeds.
Test: Verified socket unit tests pass.
Test: See details in testing done comment in the code review.
Change-Id: I6d46b8f859c5ec5ad936eb3de73b3a701ac9542d
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180401
Reviewed-by: Carlos Chinchilla <cachinchilla@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Commit-Queue: Erik Staats <estaats@google.com>
|
|
Bug: b/266950138
Change-Id: Ifc643a01ddc49e77c6180711ec73e73911e4740c
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180454
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Armando Montanez <amontanez@google.com>
Pigweed-Auto-Submit: Ted Pudlik <tpudlik@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
|
|
This CL moves `internal::Framework`, `internal::TestInfo`, and `State`
out of pw_perf_test.h and into their own separate files.
It also moves performance_test_generic.cc to
examples/example_perf_test.cc.
Finally, it removes perf_test_test.cc as this file created a no-op unit
test.
Change-Id: Id5e984cdf80bb7ed801be1f81ebb8cba8c05b91b
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/178914
Reviewed-by: Armando Montanez <amontanez@google.com>
Commit-Queue: Aaron Green <aarongreen@google.com>
|
|
This SEED proposes new build rules for declaring Bazel toolchains.
Change-Id: Ief884f3cc4dbb9aebb15e5175f65bac7d0d21cee
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/173453
Commit-Queue: Armando Montanez <amontanez@google.com>
Reviewed-by: Ted Pudlik <tpudlik@google.com>
|
|
Also, when closing sockets, disconnect the underlying sockets by using
the socket shutdown API. This unblocks socket recv and accept calls.
Bug: 309680612
Test: Verified socket unit tests pass.
Test: See details in testing done comment in the code review.
Change-Id: If9e54bdb88fa36a3f831d32b4549d5a222fd3df8
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179591
Reviewed-by: Carlos Chinchilla <cachinchilla@google.com>
Commit-Queue: Erik Staats <estaats@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
|
|
Updates Bazel to 7.0.0-pre.20231011.2.6.
Fixes: b/299671945
Change-Id: I05fc4feda6d1989d208f37072b6ea95011717418
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/178950
Pigweed-Auto-Submit: Armando Montanez <amontanez@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Reviewed-by: Ted Pudlik <tpudlik@google.com>
|
|
Importing test_fixture.py as a relative import doesn't work in the 7.0.0
release candidate of Bazel. This changes the paths to be
project-absolute to unblock updating Bazel.
Bug: b/310038737
Change-Id: Ic20127c259f222f45a0e1345324da1a6e426635f
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180390
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Commit-Queue: Armando Montanez <amontanez@google.com>
Reviewed-by: Ted Pudlik <tpudlik@google.com>
|
|
Change-Id: Ib41c5973dc1a7d01a2dd278df6843e740d6d1a95
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180274
Commit-Queue: Alexei Frolov <frolv@google.com>
Reviewed-by: Ted Pudlik <tpudlik@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
|
|
Fixes an issue where the fixed-width filter field would cause overflow
in the toolbar and hide the action buttons
Change-Id: Ic0e3be0ef6b51e38d4f7f1ed5ef9faaf7ba17cb2
Bug: 308993282
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179470
Reviewed-by: Amy Hu <amyhu@google.com>
Commit-Queue: Luis Flores <lesprit@google.com>
|
|
Change-Id: Ibb1459ba4a92cf244440e00d5e56061b7356d8d5
Fixed: 309546085
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179252
Reviewed-by: Asad Memon <asadmemon@google.com>
Commit-Queue: Amy Hu <amyhu@google.com>
|
|
Change-Id: I4aa20b82fea49db34cac8fbe70bae3a6ed690a12
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180310
Reviewed-by: Rob Mohr <mohrr@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Pigweed-Auto-Submit: Kayce Basques <kayce@google.com>
Reviewed-by: Kayce Basques <kayce@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Reviewed-by: Alan Rosenthal <alanrosenthal@google.com>
|
|
Add documentation for the ASSERT_ and EXPECT_ macros.
Change-Id: I21544ad8e23716b691d2ecc92f2131e9201a1f10
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179873
Commit-Queue: Taylor Cramer <cramertj@google.com>
Pigweed-Auto-Submit: Yuval Peress <peress@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Reviewed-by: Taylor Cramer <cramertj@google.com>
Pigweed-Auto-Submit: Taylor Cramer <cramertj@google.com>
|
|
Mention what CL authors are responsible for.
Change-Id: I2b91c023be379eba5c58c84c4b1d3842aae99f6e
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179171
Reviewed-by: Keir Mierle <keir@google.com>
Commit-Queue: Carlos Chinchilla <cachinchilla@google.com>
|
|
Change-Id: I5bfaf92d5993ace844b680c9ef500f24b60cdf16
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180170
Reviewed-by: Anthony DiGirolamo <tonymd@google.com>
Commit-Queue: Rob Mohr <mohrr@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
|
|
This CL removes the `...Unchecked` methods from
`pw::allocator::Allocator`, and modifies the NVI-style `Do...` methods
to take `Layout` parameters.
Change-Id: I48de73933ea18ae75dbadb1287780c54fe29873b
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/176754
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Commit-Queue: Aaron Green <aarongreen@google.com>
Reviewed-by: Taylor Cramer <cramertj@google.com>
|
|
Makes the steps in working around codependent changes more distinct, and
mentions ACL process at a high level to smooth over requests.
Change-Id: If48588f5be2031435c217ab91123b18623e91f86
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179879
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Pigweed-Auto-Submit: Armando Montanez <amontanez@google.com>
Reviewed-by: Rob Mohr <mohrr@google.com>
|
|
Bug: 309665550
Change-Id: Ia258bd4d640a74befda5b66cf6ec73190ef2825f
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/180030
Pigweed-Auto-Submit: Carlos Chinchilla <cachinchilla@google.com>
Reviewed-by: Ted Pudlik <tpudlik@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
|
|
This used pwrev/171697 as a guide.
Change-Id: Ia9a92fc824035668f39d81cb280ba7b4d05169d4
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/175810
Reviewed-by: Armando Montanez <amontanez@google.com>
Reviewed-by: Ted Pudlik <tpudlik@google.com>
|
|
Fixed: 305022112
Change-Id: I148e2f9a50ba256dd366f0ffcc6a422bac44b82c
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/176867
Commit-Queue: Amy Hu <amyhu@google.com>
Reviewed-by: Asad Memon <asadmemon@google.com>
Reviewed-by: Keir Mierle <keir@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
|
|
Libraries declared in bazel that depend on googletest should have
testonly set to True.
Bug: 309665550
Change-Id: Iecb4024fe13a2992ec3aa9a23493a4d1d2c9063b
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179874
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Ted Pudlik <tpudlik@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Pigweed-Auto-Submit: Carlos Chinchilla <cachinchilla@google.com>
|
|
Libraries declared in bazel that depend on googletest should have
testonly set to True.
Bug: 309665550
Change-Id: Ie484538d469f0b21c8690d7332ba8e9ab8d998f0
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179836
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Reviewed-by: Ted Pudlik <tpudlik@google.com>
Pigweed-Auto-Submit: Carlos Chinchilla <cachinchilla@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
|
|
Add implementation of the _NEAR checks using existing infrastructure.
The same method is used for comparing floats and double.
Error messaging was shortened compared to gTest's to save on buffer
space.
Change-Id: I8b5d490d48f756c61a6b7008e221c522f747fcae
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179770
Reviewed-by: Alexei Frolov <frolv@google.com>
Commit-Queue: Yuval Peress <peress@google.com>
|
|
This CL does the following:
* It move the SimpleAllocator previously described in Block's
class-level comments to an actual source file, and adds a test for
it. The class is now included directly into the module docs.
* It refactors the FakeAllocator to use SimpleAllocator and renames it
to AllocatorForTest. It also moves this from pw_allocator_private/ to
public/pw_allocator/, as these utilities may be useful for testing
custom allocators provided by downstream projects.
* It updates the callers of FakeAllocator to use AllocatorForTest.
Bug: b/306686936
Change-Id: I41953e3b64fa831bd23418496ffa683917944b3f
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/177653
Commit-Queue: Aaron Green <aarongreen@google.com>
Reviewed-by: Taylor Cramer <cramertj@google.com>
Reviewed-by: Keir Mierle <keir@google.com>
|
|
Bug: 288587657
Change-Id: Idb303e2c9b138474777441c0e54bc2044b1076d4
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/178923
Reviewed-by: Keir Mierle <keir@google.com>
Commit-Queue: Amy Hu <amyhu@google.com>
Reviewed-by: Asad Memon <asadmemon@google.com>
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
|
|
This CL moves the dummy definition of `Domain<T>` to match the real
location in FuzzTest. This allows fuzzer helpers to specify a return
type instead of using `auto`.
Change-Id: I49810c0f2fe40c97c5b9673c1c3413690331c008
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/178213
Reviewed-by: Taylor Cramer <cramertj@google.com>
Commit-Queue: Aaron Green <aarongreen@google.com>
|
|
Libraries declared in bazel that depend on googletest should have
testonly set to True.
Bug: 309665550
Change-Id: Iabab250bf7a546ca0ff48362105228f4ff854102
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179612
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Ted Pudlik <tpudlik@google.com>
Pigweed-Auto-Submit: Carlos Chinchilla <cachinchilla@google.com>
Reviewed-by: Nathaniel Brough <nathaniel.brough@gmail.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
|
|
Change-Id: I2c71100d25287c1fac9649d2bd3d340c7cd9fd7d
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179671
Reviewed-by: Keir Mierle <keir@google.com>
Commit-Queue: Chad Norvell <chadnorvell@google.com>
|
|
- Add in missing transitive deps from constraint_hashes_* files
- Normalize (downcase) package names
- Remove upstream only deps
Change-Id: I36f73dae3c7d810043b5356f4337523b6c19ae19
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/179614
Presubmit-Verified: CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com>
Pigweed-Auto-Submit: Anthony DiGirolamo <tonymd@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed-service-accounts.iam.gserviceaccount.com>
Reviewed-by: Chad Norvell <chadnorvell@google.com>
|
|
Change-Id: Idac3ae2726d7ba081771bb672159c68c3467683b
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/168833
Commit-Queue: Dave Roth <davidroth@google.com>
Reviewed-by: Armando Montanez <amontanez@google.com>
|
|
Change-Id: I0d35d21ba2a879ec99b5ea2092ea3726b6ef588f
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/163573
Commit-Queue: Chad Norvell <chadnorvell@google.com>
Reviewed-by: Anthony DiGirolamo <tonymd@google.com>
|