It's faster since it doesn't have a second string to loop over.

Change-Id: I8a5917ad356db38e18cc370276edb76ce3027a17

We have them, so save the subtraction.

Change-Id: Iae5a623b2f77323f6bcf701eeb5fb688f900262a

Change-Id: I8f51197cfce502f07f427a7bb1718d9caeba8465

If the container is empty, the loop will be a no-op anyways, so simplify
the code a bit

Change-Id: I641b93744ff677dd1d8f78a6ad88ea0aa284fa39

Change-Id: I5fc21da7073177768088a7298d84632c43f70bd6

If Relative was non-empty, the parameter is unconditionally casted into
a string anyways, so save the allocation.

Change-Id: I835782fa0307c0122cacf032d4937b5646561d7f

Change-Id: I2f02cf1c150019ecef628a322824a52c86c25431

It's weird that find had one (non-inline) overload. Make overloads for
the ctor and .compile() as well so that other code can drop unnecessary
.c_str() calls.

Change-Id: If0da9811728892bdfd1340f51ec9b33524776958

Rather than doing str[str.size() - 1], save pointer arithmetic and just
use the .rbegin() iterator to get the last element of a container.

Change-Id: Iad5cffce4da4d714f19d93273ee1193a576b8f87

MSVC will warn about performance problems for implicit int -> bool
conversions, but it also makes the code cleaner.

Change-Id: Iedc5fc8ebe56e052303b990005379bed0a8ef76a

Change-Id: Ib725299593d9fce88c73fcee2d640def5759b708

It's not needed and just clutters the code.

Change-Id: I9e69621be79b8bfebe6790edae4cdb86e8389733

Include execinfo.h, cxxabi.h, and dlfcn.h under the same conditions
under which we use the APIs from them.  Move their inclusion out of
OS-specific blocks.

Change-Id: I7723e530e2351d259adddd8e570036a562bade05

Declare a private _Hashtable_node<_Val>::operator= without
implementation to ensure no code tries to assign nodes.
The hash_map uses "pair<const _Key,_Tp>" as the value type
so the const key cannot be assigned.

Change-Id: If772e1588c9504df450c18eb0d36dc024db9f2da

Rolf Eike Beer authored 10 years ago and Brad King committed 10 years ago

Change-Id: Ia74f41c2bb0dc7b164c7a2920acdedcfe17a2899

Change-Id: I5261ad19e3702e646576d70395625ac47adc942e

This was caught by the HP compiler warning:

 warning #4285-D: operator= does not have a check for the source and
 destination addresses being non-identical

Change-Id: I074a33410402b2b872dee03df3d4006ffe32f111

Enable to add extra single quote used by some Watcom tools.

Change-Id: I257bafd1c09c02642e0d793caf7461454bc4b04a

Rolf Eike Beer authored 11 years ago and Brad King committed 11 years ago

Drop the case body when neither CPU_HP_INTEL_EM_1_0 or
CPU_IA64_ARCHREV_0 is defined.

Change-Id: I6e3ab80c74524cb1f0e043f8f7147004142436e1

Gerhard Grimm authored 11 years ago and Brad King committed 11 years ago

Add missing break statements in the QueryHPUXProcessor switch.

Change-Id: I09ada6e51ca5819fedaf29545caadcb3ce5c24ca

Stephen Kelly authored 11 years ago and Brad King committed 11 years ago

Change-Id: I2323e5b6ec35948f6f3ba8967369969e475f4e02

KWSys does not depend on the Apple Clang compiler id being "Clang" so
set the policy to NEW explicitly to avoid the warning and get the NEW
behavior.

Change-Id: I884a82876ced444753a28eec1a2a26cc8c07cdef

IBM XL complains:

 1500-010: (W) WARNING in test6: Infinite loop.  Program may not stop.

Tell it to ignore this because the test does it intentionally.

Change-Id: Ic99ed35067e8ddecea289fffe20d5e119029fa0f

Adrien Destugues authored 11 years ago and Brad King committed 11 years ago

The Haiku API for this was recently changed, to allow more than 8 CPU
cores.

Change-Id: Ia5e0d733cdfabaeffdb7707045f6635734ddac2e

Jiri Malak authored 11 years ago and Brad King committed 11 years ago

Use ISO C header <wctype.h> for towupper instead of relying on the
MS-specific location of towupper in <ctype.h>

Change-Id: If779418eb6752e7b059bcb9e823f64914b4fa101

Change-Id: I4d06782730fca0fd68fb62c418b4e2d95f550625

Matt McCormick authored 11 years ago and Brad King committed 11 years ago

Hide this implementation detail by default.

Change-Id: I436d76681c88459de470d5eec019acfc354d427e

This fixes compile errors on older compilers where basic_filebuf is
not available.

Change-Id: If220925774b5d31b128065603cf365a7a5ec5f42

Compile flags -DUNICODE and -D_UNICODE are added.
This means that using ANSI or wide apis must be explicit.

Change-Id: Ia0fa72f563e1ee8591f6ca3c02b060abcb801a7c

Muli Baron authored 11 years ago and Brad King committed 11 years ago

Using PuTTY with the terminal type set to putty-256color causes CMake
not to output any colors during build. Changing it to one of the
'supported' terminal types fixes the problem. This patch adds the
putty-256color terminal type to the list of supported terminals.

Change-Id: I0f261017f2a47348306a5c8da3e2c0e2f906c161

Also adding a test for this and future improvements to the module.

Change-Id: I54c33129291c9df2fee4bf5dd52360251f6be540

Change-Id: Ifda279df9a68872cf16e580274dff534f6cfd855

Clinton Stimpson authored 11 years ago and Brad King committed 11 years ago

Change-Id: I96564a95f1ee6111edc58bdd85a933ace7502a73

Clinton Stimpson authored 11 years ago and Brad King committed 11 years ago

SystemTools still had uses of kwsys_ios::ofstream that need changed
to kwsys::ofstream to support utf-8 encoding of filenames.

Change-Id: I308e9cc055295881405b1b079d0653dab37b8d3d

Ådne Hovda authored 11 years ago and Brad King committed 11 years ago

Some older versions of HP-UX do not define CPU_IA64_ARCHREV_0, but
CPU_HP_INTEL_EM_1_0 resolves to the same constant (in
/usr/include/sys/unistd.h).

Change-Id: I20f9a161ef62277069a5ba0d339ca09d972a9bae

If a group of processes needs to be terminated recursively the entry in /proc
doesn't need to be kept open during the recursion as all the interesting
information was already read from it. Especially in case of deeper recursion
this lowers the memory pressure.

Change-Id: I95b969f42c8dd68b57cdaf8fc1b705f358f1dc58

Change-Id: I9a6ce9af30dae548e860636ad64164200ba91f49

The WindowsFileExists function was originally created to avoid
duplication by the Cygwin and Windows implementations of FileExists.
Now that we use the ANSI API on Cygwin and the Wide API on Windows, one
of the two overloads is always left unused.  Drop the helper functions
and inline the implementations at their respective (single) call sites.

Change-Id: I4d25f98b8b11f0fdce2f360f5f425e0b159b22e5

The KWSys "String.h" is not needed by "Encoding.hxx" and may not
be available in projects that do not enable the KWSys String API.

Change-Id: I50ff9a89fc3d43b59f6958e09423862bf104d904

Sean McBride authored 11 years ago and Brad King committed 11 years ago

A single quote in an inline assembly comment is confusing gcc:

 .../SystemInformation.cxx:256:24: warning: invalid suffix on literal;
 C++11 requires a space between literal and identifier [-Wliteral-suffix]
 ; optimiser doesn't know about <<CPUID>>, and so doesn't expect
                  ^

Avoid it by changing "doesn't" to "does not".

Change-Id: Ic4caaeaabfdb99aa486ec09b23d670ec1b125069