Commit db526e73 authored by Bruce Dawson's avatar Bruce Dawson Committed by Commit Bot

Fix references to non-existent function and variable

All unchecked code will contain errors and apparently, due to quirks of
VC++ template compilation, emulated by clang-cl, the ThreadSpecific<T>
destructor was never *really* compiled. It referenced two functions and
one variable and only one of these three things actually existed. This
was reported by Microsoft who use chromium as a compiler torture test.

Bug: 825352
Change-Id: I5dd1868acda760ef33dde518d72e3d7694fad9a8
Reviewed-on: https://chromium-review.googlesource.com/978849
Commit-Queue: Kent Tamura <tkent@chromium.org>
Reviewed-by: default avatarKent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#546720}
parent 739628e7
......@@ -201,7 +201,7 @@ template <typename T>
inline ThreadSpecific<T>::~ThreadSpecific() {
// Does not invoke destructor functions. They will be called from
// ThreadSpecificThreadExit when the thread is detached.
TlsFree(tlsKeys()[m_index]);
TlsFree(TlsKeys()[index_]);
}
template <typename T>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment