diff options
author | Nat Goodspeed <nat@lindenlab.com> | 2015-06-25 17:00:47 -0400 |
---|---|---|
committer | Nat Goodspeed <nat@lindenlab.com> | 2015-06-25 17:00:47 -0400 |
commit | aefdba1230de456784d220982ab8b4dbe3cde17d (patch) | |
tree | 52c56bc79725187e9d4a6244e0621667b695d7a1 /indra/llcommon/llpredicate.cpp | |
parent | 0727519d9307ed09877073ef17b754526ee3f5b1 (diff) |
MAINT-5232: Make LLHTTPClientAdapter not be an LLSingleton.
llhttpclientadapter_test.cpp starts its every test by explicitly instantiating
a local LLHTTPClientAdapter object. This is an abuse of LLSingleton, and if it
had been properly defined (private constructor), it should never have compiled.
Looked at the other way, though, every known reference to LLHTTPClientAdapter
instantiates a local object. Why did someone think it should be an LLSingleton
in the first place? Remove LLSingleton<> as a base class; remove llsingleton.h.
This makes llhttpclientadapter_test.cpp work just fine.
One might also question what value this class adds. It seems to do very little
-- but more significantly, the ONLY references in the source tree are its
declaration, definition and test. Nobody actually uses it anywhere.
Diffstat (limited to 'indra/llcommon/llpredicate.cpp')
0 files changed, 0 insertions, 0 deletions