Merge pull request #815 from sgraham/gtest-tidy

Don't need to install and configure with gtest any more

Don't need to install and configure with gtest any more

Merge pull request #812 from nico/ninjatest

Use a small, standalone testing framework for ninja instead of googletest.

Don't support ? and : for --gtest-filter, a bit simpler.

Let me know if you use these!

Add support for --gtest_filter. The bots need it, and it is a useful flag.

Use a small, standalone testing framework instead of googletest.

Ninja currently uses googletest for testing. That makes building
ninja_test somewhat annoying since it requires that one passes
--with-gtest PATH to configure. It turns out just implementing the bits
of googletest that ninja uses needs about the same amount of code than
making the --with-gtest flag in configure.py work and making googletest
print test results in a way we want (!)

In addition to making configuration simpler, this also makes compiling
tests much faster: On my system, touching src/build_test.cc (the slowest
file to build in ninja) and rebuilding ninja_tests is twice as fast than
without this patch. Building all is noticeably faster too: 5.6s with
this patch, 9.1s without this patch (38% faster).

The most noticeable things missing: EXPECT_* and ASSERT_* don't support
streaming notes to them with operator<<, and for failing tests the lhs
and rhs are not printed. That's so that this header does not have to
include sstream, which slows down building ninja_test almost 20%.
If this turns out to be annoying, we can maybe add it.

Merge pull request #808 from nico/bettererror

Provide an error message on malformed lets. Fixes #807.

Provide an error message on malformed lets. Fixes #807.

Merge pull request #803 from martine/arch

amend HACKING.md with instructions for Arch Linux

Merge pull request #802 from martine/parens

add some parens to silence a gcc warning

amend HACKING.md with instructions for Arch Linux

add some parens to silence a gcc warning

Merge pull request #798 from kwesolowski/master

Provided Basic Support for -l N option on windows platform.

Remove extra info from Usage()

As now behavior is similar on on platforms, this can probably be removed

Changed implementation to provide load from 0 to ProcessorCount

This makes this implementation more consisten with POSIX load avarage.

Fixed naming convention in GetLoadAverage support functions.

Merge pull request #799 from ryo-on/netbsd-support

Add NetBSD support.

Improved load calculation

Added code to gracefully handle:
1. Call to CalculateProcessorLoad with not incremented ticks (fast calls
to GetSystemTimes can result same results),
2. Smooth/filter load estimation for consecutive calls.

Small usage info fix for windows

Fixes for windows CalculateProcessorLoad

- Fixed bad logic condition,
- Added comment to clarify

Fixed/improved -l N documentation For windows

On windows "system load" is not so obvious term, added line explaining N
argument in -l N

Prepared load (-l N) support for windows.

Inspired by:
http://stackoverflow.com/questions/23143693/retrieving-cpu-load-percent-total-in-windows-with-c

Add NetBSD support.
It works fine under NetBSD/amd64 6.99.47.

Merge pull request #788 from donkopotamus/patch-2

Add highlighting of rule in build statements

Merge pull request #755 from zmodem/working_dir_vs_msvc_tool

Process the -C option before running RUN_AFTER_FLAGS tools.

mark this 1.5.1.git

Merge pull request #792 from nico/statdir

try to fix stat()ing directories on windows in statcache code

try to fix stat()ing directories on windows

add more tests for stat()ing a directory

add a test for stat()ing a directory

Merge pull request #791 from nico/relmore

also note that branch release needs to be pushed

also note that branch release needs to be pushed

Merge pull request #790 from nico/relnotes2

expand RELEASING a bit

expand RELEASING a bit

mark this 1.5.0.git, update RELEASING

Merge pull request #789 from nico/dontwin

Rename -d nowinstatcache to -d nostatcache; might become useful elsewhere (#787)

Rename -d nowinstatcache to -d nostatcache; might become useful elsewhere (#787)

Add highlighting of rule in build statements

Highlight the rule being used in a build statement.  Also add `.` to acceptable characters in a rule name and relax whitespace matching before the name.

Merge pull request #786 from donkopotamus/patch-1

Update ninja-mode for emacs to handle hyphens in rule names

Update ninja-mode for emacs to handle hyphens in rule names

Merge pull request #785 from nico/unused

Fix -Wunused-result warning for chdir on linux.

Fix -Wunused-result warning for chdir on linux.

Merge pull request #784 from nico/statkeep

do not delete files from the logs that still exist on disk

do not delete files from the logs that still exist on disk

This is to keep the possibility of maybe having a tool that deletes
old files in the future, or for having a tool which exposes this
information to generators so they can do that.

See https://github.com/martine/ninja/pull/697#issuecomment-37140762
and the discussion on #762.

Idea from @maximuska!

make Stat() a const method

Merge pull request #783 from sgraham/findexinfobasic

use local definition of FindExInfoBasic for earlier sdks

use local definition of FindExInfoBasic for earlier sdks

Merge pull request #782 from sgraham/warning-on-81sdk

suppress warning on win8.1 sdk

suppress warning on win8.1 sdk

Merge pull request #781 from nico/cleaner

simplify statcache code more

simplify statcache code more

Merge pull request #780 from nico/versioncheck

only use FindExInfoBasic on win7+

only use FindExInfoBasic on win7+

Merge pull request #779 from nico/statcache

Use FindFirstFileEx() for stat()ing files on Windows.

s/hFind/find_handle/

add missing _WIN32 checks

make bool exist everywhere, for simpler calling code

add some statcache tests

Free cache memory once it's no longer used.

Doesn't slow down empty build times measurably, and
saves some memory on non-empty builds.

error checking

on by default

comment

make win-only

simplify more, move behind flag

more minor cleanups

more minor cleanups

minor cleanups

Turn on stat cache. Empty builds of chrome on my laptop 4s -> 1.3s (!)

Add a stat cache. Demo-quality, and disabled atm.

Merge pull request #778 from nico/spellrsp

spellcheck "keeprsp" in -d options

spellcheck "keeprsp" in -d options

Merge pull request #777 from nico/winconf

create a slightly nicer build.ninja on windows

create a slightly nicer build.ninja on windows

Version 1.4 of the vim syntax file was merged.

See https://code.google.com/p/vim/source/detail?r=92751673cc37c9ef4d1ad1ac4d42d36faa67f88f

Merge pull request #770 from debfx/master

Use unversioned \ 1gnukfreebsd platform.

Use unversioned \ 1gnukfreebsd platform.

Merge pull request #769 from nico/plus

Allow + in filenames without escaping

Allow + in filenames without escaping

Due to #690, file.c++ used to be escaped. + seems as safe as -, so allow
it to not be escaped, to keep compile command lines with a fairly common
extension slightly cleaner.

Merge pull request #768 from nico/vimsyn

vim syntax: Correctly highlight $$a as ($$)a instead of $($a).

Merge pull request #767 from nico/esc_deprspfile

Make "depfile=$out.d" work if $out contains escaped characters, rspfile ...

reword manual for depfile/rspfile escaping change

Make "depfile=$out.d" work if $out contains escaped characters, rspfile too.

Fixes #730.  This has always been broken, but due to #690 more paths are now
escaped (e.g. paths containing + characters, like file.c++).  Also see
discussion in #689.

The approach is to give EdgeEnv an enum deciding on whether or not to escape
file names, and provide functions that evaluate depfile and rspfile with that
set that to kNoEscape.  (depfile=$out.d doesn't make sense on edges with
multiple outputs.)

This should be relatively safe, as $in and $out can't be used on edges, only
on rules (#687).

Merge pull request #766 from nico/cleanup

CleanTest cleanups:

CleanTest cleanups:
* $in only makes sense on rules, not edges (see issue #687)
* Remove unneccesary clear() line at end of test

Merge pull request #764 from nico/winconsole

Implement pool=console support on Windows.

win console wip: Fix comments based on review feedback.

win console wip: resolve FIXME

vim syntax: Correctly highlight $$a as ($$)a instead of $($a).

win console wip: ctrl-c should reach commands running in console pools

win console wip: enable test

wip for console pool on windows

Merge pull request #761 from maximuska/proposed/depfile_parser_fixes

Allow paths with '{' '}' in depfiles

Allow paths with '{' '}' in depfiles

Merge pull request #756 from ronnychevalier/rc/zsh_completion

improve zsh completion

Merge pull request #758 from mathstuf/dev/fix-compdb-abort

compdb: check that inputs is not empty

compdb: check that inputs is not empty

Merge pull request #757 from cjdrake/master

Stylistic changes to ninja_syntax.py

Get rid of unused import

Use consistent indentation conventions

improve zsh completion