Use the GetShortPathNameW return value to allocate a buffer of the
required size.  The previous assumption that the "short" path will
always require less space than the original path is incorrect.

CMake-Issue: 15892
Inspired-by: Jan van Dorsten <dorsten@boehme-weihs.de>
Change-Id: I5845752710560b3e71a2f5eaf334dd676f5b3a9d

Jan van Dorsten authored 9 years ago and Brad King committed 9 years ago

Use a `std::string` instead of `new char`.

Change-Id: I88afe09990c6dd881c813df963ccd0031dcea301

The ICC frontend on OSX is clang, and therefore the "__clang__" define exists,
but it doesn't support the clang diagnostic pragma syntax.

Change-Id: I6947bd22c77af8066b85a6b00be7cf6019e7d06e

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

When peeking in a file for a byte order mark, if the file is short
enough to reach end-of-file, clear the istream flags.

CMake-Issue: 15890
Change-Id: I1fa3a1b94b47ad46b568b00b7b82b5380cbfcb21

This header is no longer used by any KWSys clients.

Change-Id: I9362a7053b2f24f6bacf2bbe89dbfceeaabe70ae

This header is no longer used by any KWSys clients.

Change-Id: Ifca2c6aec5d4b3068774d8398719376d61706d53

This was used only for an old VS 6 version that did not support
conversion of `unsigned __int64` to `double`.  Support for this compiler
has been dropped from KWSys clients that used the macro.

Change-Id: Ia61beaa0251fc2b6d77632af3e610c08f64d11eb

FileIsDirectory() will return false if the given path does not exist, so the
call to FileExists() is redundant.

Change-Id: I46b3dde9150d6f24a436c4d2f9dd6e2d21c57638

SystemTools::FileExists() can already check this case. These instances were
forgotten in commit 0adafb51 (SystemTools: use
FindProgram() overload that checks for directory). The summary says
"FindProgram()", but it meant to be "FileExists()".

Change-Id: I5ef2919d915fffe3a70e3f47d0c49e18951e66eb

Neither this variable nor the original one are modified before the copy gets
later overwritten, so just use the original variable to entirely avoid needless
copy operations.

Change-Id: I67438e774d68c797e77dc4f4a2b50bbad131a9da

This vector will only be filled in an #ifdef, so put the only user of it outside
that first #ifdef into an according one. This means Un*x platforms do not need
to carry that dead code around anymore.

Change-Id: Ia5071c36ed253e7cea63ccfa516add61799877f6

Change-Id: I59c150ef46c87bea36bab2822ebd0643aa7b7c2b

Change-Id: I45efddc1638965d17f0207cdf900a44aef8bf0e8

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

The list of possible extensions is only filled inside an #ifdef, so the whole
usage of this list can be moved into that #ifdef's, too. The loop can be
completely avoided if it's already known that the list will not be filled at
all.

Change-Id: I89a87115babf68f9bf99d4ad523cb0851c252cd9

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

This converts all constructs like

  FileExists(f) && !SystemTools::FileIsDirectory(f)

to

  FileExists(f, true)

which behaves the same.

Change-Id: Iadb26fb792db032a396bc54c58f7b902c5e054b2

Dmitry Marakasov authored 9 years ago and Brad King committed 9 years ago

Microseconds range from 0 to 999999, so 1000000 is an overflow which
should be processed as well.

Change-Id: I59873504987d6aa416e87aa7f5cd208983217ace

Marek Vasut authored 9 years ago and Brad King committed 9 years ago

The NIOS2 CPU is little-endian.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Ley Foon Tan <lftan@altera.com>
Cc: Thomas Chou <thomas@wytron.com.tw>
Cc: Walter Goossens <waltergoossens@home.nl>
Change-Id: Ifd1bc890b2fb74168b934641c70800a029f7f0b8

Jan Niklas Hasse authored 9 years ago and Brad King committed 9 years ago

See http://bixense.com/clicolors/ for documentation of this convention.
This is useful e.g. for build servers or IDE outputs.

Change-Id: I648edb4e09deda12a33328a7248dd3daf920be38

Teach the Touch and FileTimeCompare methods to use the `st_mtimespec`
member of `struct stat` on OS X.

Change-Id: Ib7dcca41eee571af23509536fe4768a68d98a81a

Construct the access time component-wise at runtime rather than in an
initializer list.

Change-Id: I0cdcb341b2b9e49506ebe6363729b4a662e38307

Do not expose our platform introspection result publicly.

Change-Id: I9929f31293adcb628b32d9d7f11b120c44046354

Terrell Russell authored 9 years ago and Brad King committed 9 years ago

Change-Id: I1375f56aacc561714d2cb0095478a763079ee657

Include the <shellapi.h> header to get CommandLineToArgvW even
when <windows.h> does not include it due to WIN32_LEAN_AND_MEAN.

Change-Id: If42241621c30b76c7b49cd64e044a19403f9da09

Vitaly Baranov authored 9 years ago and Brad King committed 9 years ago

The GetPath method appends paths to an output vector and also converts
paths to forward slashes.  However, the method should not modify entries
in the output vector that existed prior to the call.  Fix it to only fix
slashes in the entries it added to the vector.  Also add a test case
covering this behavior.

Change-Id: Ie098c3eddbe63ed59f0d393aa6c43ea6c395011d

Domen Vrankar authored 9 years ago and Brad King committed 9 years ago

Teach these functions to copy directories as directories instead
of files.  Create the destination directory and set permissions,
but do not recurse into the directory structure because these
functions are meant for individual filesystem entries.

Change-Id: I20fae2e159f86285c02c3fd894609e2e51cafdac

Change-Id: I3cc74292e44eb13e8d7c278b67d7270119e2ae4f

Gilles Khouzam authored 9 years ago and Brad King committed 9 years ago

Change-Id: I552e0840b5c84187b67a781621c09b485d3e6048

Gilles Khouzam authored 9 years ago and Brad King committed 9 years ago

We pass a OSVERSIONINFOEXA structure so call GetVersionExA explicitly to
avoid ever calling GetVersionExW with a structure of the wrong size by
mistake.

Change-Id: I892ae0497ef3ca4b38cd2c1c1870d165b0bd8713

Change-Id: Ie06816e1a2a0537c23fb646985a8017087dcf702

Change-Id: I0da93a55af243fe798b9dd5bda4ee30c1bc3fcf9

Drop the KWSYS_STL_HAS_* checks needed only for hashtable compatibility
with old STL versions.

Change-Id: I164a40669881439f5853a72c60ab2065498a69c0

Drop "kwsys/stl/*" and "kwsys_stl::" and use the standard headers and
namespace instead.  Drop the now-unused KWSYS_STL_HAVE_STD check.

Change-Id: I6105fd32cd8f0e487e03f58927976e824a53f0d8

Remove the KWSYS_STL_STRING_HAVE_{ISTREAM,OSTREAM,NEQ_CHAR} checks
and drop their uses.

Change-Id: I1e95dbbd724a7e08db67f71533da381b9d943f53

Remove the checks for KWSYS_IOS_USE_ANSI and KWSYS_IOS_HAVE_STD and uses
of the results.

Change-Id: I67d756a70a08636175a9d131c6b3e27a3f74b175

Change-Id: I930fdf45832e09b01b653a21ed0be159bebbf45d

Change-Id: Icdc68acbac4b393ff753598dff331ac22df68b73

Use the standard headers and namespace instead.  We no longer support
compilers without them.

Change-Id: Iafb19d28462e9302df06eecc8ae42b48a9577a21

If honor_umask is set, the umask is queried and applied to the
given permissions when calling SetPermissions.

Change-Id: I302231a2ae7f0c4610eb47fdb256fa02934ecd8c

Added a new TestFileAccess function that can be used to test for
access to a file or directory using the requested combination of
permissions.  This is basically a wrapper of the POSIX access()
function.

Change-Id: I3e1769a2438cc2047a97850c5f312e36ed695799

The FileExists function always ensures that a file is readable on
POSIX systems by using access() with R_OK.  This was due to bug #809.
Publicly state in the header file that this is being done.

Also simplify the implementation so that redundant code handling the
isFile parameter is eliminated.

Change-Id: Ie07bec7aa5dd2bca07bb3244c01c0a29587aa437