From 4ff1c872045cce0a794076fb4dbeac4b55ae54c9 Mon Sep 17 00:00:00 2001 From: Mirko Bonadei Date: Wed, 23 Oct 2019 11:06:25 -0700 Subject: [PATCH] Fix RTC_LOCKABLE RTC_EXPORT order for rtc::Thread. The previous order of macros causes [1]. [1] - https://ci.chromium.org/p/chromium/builders/try/win_chromium_compile_dbg_ng/420871 Bug: webrtc:9419 Change-Id: If261e5c1d4bf058417be2bcc9a7cd26cc210f20b Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/158121 Reviewed-by: Steve Anton Commit-Queue: Mirko Bonadei Cr-Commit-Position: refs/heads/master@{#29592} --- rtc_base/thread.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rtc_base/thread.h b/rtc_base/thread.h index 01a3917155..41052dfb1d 100644 --- a/rtc_base/thread.h +++ b/rtc_base/thread.h @@ -133,7 +133,7 @@ struct _SendMessage { // WARNING! SUBCLASSES MUST CALL Stop() IN THEIR DESTRUCTORS! See ~Thread(). -class RTC_EXPORT RTC_LOCKABLE Thread : public MessageQueue { +class RTC_LOCKABLE RTC_EXPORT Thread : public MessageQueue { public: explicit Thread(SocketServer* ss); explicit Thread(std::unique_ptr ss);