Skip to content
This repository was archived by the owner on Oct 25, 2024. It is now read-only.

Fix build error of libowt.so and pipe fd leak issue #62

Open
wants to merge 2 commits into
base: 79-sdk
Choose a base branch
from
Open

Fix build error of libowt.so and pipe fd leak issue #62

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f67a13a
Remove complete_static_lib to fix build error of libowt.so
liukai1025 Mar 17, 2020
a54715f
Fix pipe fd leak issue
liukai1025 Feb 17, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion BUILD.gn
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -419,7 +419,7 @@ if (!build_with_chromium) {
visibility += [ "//talk/owt" ]
}
sources = []
complete_static_lib = true
# complete_static_lib = true
suppressed_configs += [ "//build/config/compiler:thin_archive" ]
defines = []

Expand Down
4 changes: 2 additions & 2 deletions rtc_base/thread.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -82,8 +82,8 @@ Thread* Thread::Current() {
#ifndef NO_MAIN_THREAD_WRAPPING
// Only autowrap the thread which instantiated the ThreadManager.
if (!thread && manager->IsMainThread()) {
thread = new Thread(SocketServer::CreateDefault());
thread->WrapCurrentWithThreadManager(manager, true);
//thread = new Thread(SocketServer::CreateDefault());
//thread->WrapCurrentWithThreadManager(manager, true);
}
#endif

Expand Down