Skip to content

Commit

Permalink
pw_sync_freertos: Fix ODR violation in tests
Browse files Browse the repository at this point in the history
Change-Id: I0ef58f9b472a58126f342c5edcae9123b1f41483
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/160795
Commit-Queue: Keir Mierle <[email protected]>
Reviewed-by: Ewout van Bekkum <[email protected]>
  • Loading branch information
keir authored and CQ Bot Account committed Jul 28, 2023
1 parent 9a79a64 commit 3c4dc53
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions pw_sync_freertos/timed_thread_notification_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ using pw::thread::Thread;
// the FreeRTOS optimized TimedThreadNotification backend.
#if INCLUDE_vTaskSuspend == 1

class NotificationAcquirer : public thread::ThreadCore {
class TimedNotificationAcquirer : public thread::ThreadCore {
public:
void WaitUntilRunning() { started_notification_.acquire(); }
void Release() { unblock_notification_.release(); }
Expand Down Expand Up @@ -68,7 +68,7 @@ class NotificationAcquirer : public thread::ThreadCore {
};

TEST(TimedThreadNotification, AcquireWithoutSuspend) {
NotificationAcquirer notification_acquirer;
TimedNotificationAcquirer notification_acquirer;
// TODO(b/290860904): Replace TestOptionsThread0 with TestThreadContext.
Thread thread =
Thread(thread::test::TestOptionsThread0(), notification_acquirer);
Expand All @@ -91,7 +91,7 @@ TEST(TimedThreadNotification, AcquireWithoutSuspend) {
}

TEST(TimedThreadNotification, AcquireWithSuspend) {
NotificationAcquirer notification_acquirer;
TimedNotificationAcquirer notification_acquirer;
Thread thread =
Thread(thread::test::TestOptionsThread0(), notification_acquirer);

Expand Down

0 comments on commit 3c4dc53

Please sign in to comment.