| Age | Commit message (Collapse) | Author |
|---|
|
The new TUT library build eliminates the ambiguity about ensure_equals(const
char*, ...) versus ensure_equals(const std::string&, ...). Now it's all based
on const std::string&. Remove pointless const char* overloads and ambiguous
forwarding templates.
With clang in Xcode 6, any new datatypes we intend to use with ensure_equals()
must have operator<<(std::ostream&, datatype) declared BEFORE lltut.h
#includes tut.hpp. Reorder code in certain test source files to guarantee that
visibility.
|
|
lltut.h declares a number of ensure_equals() overloads for various data types,
notably the types supported by LLSD. We expect these to be called by tut code.
But the tut code in question is in a template in tut.hpp -- which was
#included BEFORE the overloads were declared. Previous C++ compilers have
evidently made multiple passes, collecting the relevant overloads before
attempting to compile the template bodies. clang does not, complaining that
the overloads must be declared before the tut.hpp template code that
references them. Reordering parts of lltut.h seems to address that problem.
For similar reasons, test programs that use StringVec.h and its operator<<()
must #include StringVec.h before lltut.h.
Add ensure_equals(const std::string&, const LLSD::Binary&, const LLSD::Binary&)
overload. The sloppy mix of (const char*, ...) and (const std::string&, ...)
overloads bothers me, since for many of those ... types we seem to have to
duplicate them.
|
|
test
|
|
|
|
in: 'cout << a() << b()' the order of evaluation of a() and b() is undefined.
|
|
in llsd_new_tut.cpp to let the build proceed - will fix later
|
|
|
|
|
|
|
|
now - operator << issues on clang
|
|
trigger clang warnings
|
|
|
|
code :(
|
|
|
|
|
|
|
|
multiple unused private members
|
|
|
|
|
|
|
|
|
|
|
|
|
|
warnigs/errors (moved to package) AND bracket clang #pragmas in #if LL_DARWIN
|
|
|
|
files - we standardize on Xcode 6
|
|
unused variables, unused const-variables, unused member variables & functions etc. Also removed flags from CMake
|
|
|
|
|
|
|
|
|
|
|
|
|
|
non-virtual dest
|
|
|
|
|
|
commented
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
thought from Nat
|
|
|
|
|
|
|
|
|
