Get rid of unused import

Use consistent indentation conventions

Merge pull request #753 from nico/nlacc

Accept \r\n line endings in depfiles.

Accept \r\n line endings in depfiles.

Fixes #752.

Merge pull request #748 from nocnokneo/bash-completion-for-f-and-C

Support completion of arguments to -f and -C options

Fix getopts barfing over the terminal when trying to complete -f and -C

Note that this is only applicable for when bash_completion is
unavailable.

Support completion of arguments to -f and -C options

Note: This is only available for bash_completion users.

Merge pull request #750 from nico/perfwin

Make manifest_parser_perftest build on Windows.

Make manifest_parser_perftest build on Windows.

Merge pull request #747 from nocnokneo/master

Fix bash-completion support for -C option to expand tilde

Style: Fix inconsistent indentation

Fix bash-completion support for -C option to expand tilde

Merge pull request #714 from pcc/console-pool

Introduce the "console" pool

Merge pull request #715 from nico/lazyenv

Allocate per-edge BindingEnvs lazily.

Merge pull request #742 from nico/maniperf

Add a manifest parsing perftest.

Manifest perftest: Try to make it build on Linux.

Manifest perftest: Pull manifest parsing into own function.

Add manifest_parser_perftest to .gitignore.

Manifest perftest: Also measure command evaluation time.

Add a simple manifest parsing perftest.

Merge pull request #741 from nicolasdespres/propagate-file-rename-to-gitignore

Propagate file rename to gitignore.

Propagate file rename to gitignore.

This was introduced by 68a4bb27d1a747b240ea68cd320a51dbd261ed14.

Merge pull request #740 from nico/formatfile

Add a .clang-format file.

Add a .clang-format file.

This isn't meant to be authoritative. It's good enough to let the
"indent current line using clang-format" hotkey do the right thing often
enough to be useful.

Merge pull request #739 from nico/ren

Rename parser_perftest to depfile_parser_perftest.

Rename parser_perftest to depfile_parser_perftest.

Merge pull request #737 from sorbits/bash-completion-fixes

Bash completion fixes

Expand the -C argument via ‘eval’ in bash completion

Previously completion would not work for ‘ninja -C $HOME/Source/foo targ‸’.

We still do not support using tilde in the directory argument.

Fix bash completion when using command options

By quoting the ‘line’ variable we are making it a single word, but ‘getopts’ wants each option as its own word.

Previously bash completion would output an error for a line like: ‘ninja -vn targ‸’.

In addition to removing the quotes (to enable word expansion) I also used it as a regular variable, as that is what it is (not an array).

Merge pull request #734 from sgraham/no-stderr

CLParser shouldn't read stderr

CLParser shouldn't read stderr

Merge pull request #722 from nico/manifestgen

add a script to generate large synthetic manifests

Merge pull request #732 from bratell-at-opera/bratell/fewer_list_copies_20140414

performance: Writer.build should copy less.

performance: Writer.build should copy less.

The build method copies the input lists several times. That
cost about 0.1s in the gyp generation for the Chromium project for
no gain.

Merge pull request #699 from mostynb/avoid_useless_mkdir_failure

don't Fail if trying to mkdir when the dir already exists

Merge pull request #728 from sgraham/both-slashes-on-windows-for-output

Support both slashes on Windows when making output dirs

Support both slashes on Windows when making output dirs

add a script to generate large synthetic manifests

To be used by a manifest parser perf test in a follow-up.

Allocate per-edge BindingEnvs lazily.

In chrome, only 2000 of 22000 build edges have bindings.  A BindingEnv is
64 bytes, so allocating these only when needed saves a bit over 1 MB of
memory.  Since env chains are shorter for lookups, builds also become a
tiny bit faster.

Introduce the "console" pool

This is a pre-defined pool with a depth of 1. It has the special property
that any task in the pool has direct access to the console. This can be
useful for interactive tasks or long-running tasks which produce status
updates on the console (such as test suites).

Merge pull request #710 from purcell/patch-1

ninja-mode: add autoload cookie for the mode itself

ninja-mode: add autoload cookie for the mode itself

Merge branch 'fix-provide-form' of git://github.com/yasuyk/ninja

Conflicts:
misc/ninja-mode.el

Merge branch 'fix-warning' of git://github.com/yasuyk/ninja

Conflicts:
misc/ninja-mode.el

Merge pull request #705 from yasuyk/elpa

emacs: ELPA compatibility

emacs: Place provide form at end of file

Any errors in the code after "provide" will leave the feature
registered, so it should always be at the end.

emacs: Fix a warning

Fix a following byte-compile warning:
ninja-mode.el:18:7:Warning: assignment to free variable `ninja-keywords'

emacs: ELPA compatibility

Merge pull request #702 from nickhutchinson/feature/win32-escaping-fix

Don’t unnecessarily escape backslashes in Win32 paths

Merge pull request #701 from nickhutchinson/feature/msvc-gtest-cflags

configure.py: use /FS flag under vs2013 when compiling gtest

configure.py: use /FS flag under vs2013 when compiling gtest

Don’t unnecessarily escape backslashes in Win32 paths

Under ::CommandLineToArgvW() rules, the backslash character only gets special treatment if it’s immediately followed by a double quote. So, when checking to see if a string needs Win32 escaping, it’s sufficient to check for the presence of a double quote character. This allows paths like "foo\bar" to be recognised as “sensible” paths, which don’t require the full escaping.

don't Fail if trying to mkdir when the dir already exists

Merge pull request #697 from nico/gclogs

Remove old outputs from build log and deps log during recompaction.

Make BuildLogUser reference constant.

Merge pull request #690 from nickhutchinson/feature/escape-paths-properly

More robust escaping of $in, $out paths

More robust escaping of $in, $out paths

In summary: don’t escape if the path doesn’t contain problematic characters, otherwise:
- Shell: enclose string in single quotes, escape embedded single quotes with the magic quote-backslash-quote sequence
- Win32: Escape double quotes by doubling the number of consecutive backslashes that precede them (if any) and adding one more. Finally, double the number of trailing backslashes, and enclose the whole thing in double quotes.

Add a comment.

Rename "IsDead" to "BuildLogUser".

Move duplicated code into a helper function.

Add a test for collection of dead log entries.

Add a test for collection of dead deps entries.

Fix DepsLogTest.Recompact by making sure outputs aren't garbag-collected.

Make tests compile (one fails atm).

Remove dead entries in .ninja_log and .ninja_deps while recompacting.

For .ninja_deps, remove objects that have no in-edges or whose in-edges
do not have a "deps" attribute. (This matches the behaviour of `-t
deps`).

BuildLog doesn't know about state, so let its recompact method take
delegate that decides is a path is life or not, and implement it in
NinjaMain.

Merge pull request #695 from nico/norange

Fix additional range in character class.

Fix additional range in character class.

The range was added in 7ab6dcbdb6447861eefafc47fc3e10f3273cede2, but that
change only tried to add ! to the character class. Fix by moving '-' to the end
of the class.

Fixes #694.

Merge pull request #691 from TheOneRing/fix-msvc13

fixed platform_helper.py msvc_needs_fs test for python 3

fixed platform_helper.py msvc_needs_fs test

Merge pull request #688 from ndmitchell/master

Fix up platform_helper for MSVC

Fix up platform_helper for MSVC, with Python 2.6.8 the arguments have to be in a list, not space separated (unless you set Shell to true)

Merge pull request #683 from nico/ruletest

Add a test that shows that there is a single global namespace for rules.

Add a test that shows that there is a single global namespace for rules.

Merge pull request #682 from nico/pep8

Wrap to 79 colums. No functionality change.

Wrap to 79 colums. No functionality change.

Merge pull request #674 from nico/style

Minor style fixes. No functionality change.

Minor style fixes. No functionality change.

Merge pull request #665 from syntheticpp/localized-showinclude

add deps_prefix for localized /showIncludes' output parsing

add deps_prefix for localized /showIncludes' output parsing

Merge pull request #671 from sgraham/compile-vs2013

Fix compilation on VS2013

Fix compilation on VS2013

mark this 1.4.0.git, update RELEASING

Merge pull request #657 from sgraham/keeprsp2

add -d keeprsp to preserve @rsp files on success on windows

add -d keeprsp to preserve @rsp files on success on windows

Merge pull request #655 from nico/simp

Simplify.

Simplify.

Merge pull request #654 from rgeary1/buildTestFix

Fix for build test for BuildWithDepsLogTest.RestatMissingDepfileDepslog

Fix for build test for BuildWithDepsLogTest.RestatMissingDepfileDepslog

Merge pull request #653 from nico/msg

Make depslog v1->v3 message less scary.

Make depslog v1->v3 message less scary.

See the comment 5 lines up for details.

Merge pull request #650 from nico/simplify

Simplify.

Simplify.

Merge pull request #649 from nico/fff

Delete a line I failed to delete in d7a46654a7be1.

Delete a line I failed to delete in d7a46654a7be1.

Merge pull request #648 from nico/landmaxim

Land the majority of pull request #608.

Fix restat rebuild if deps are missing.

Fixes issue #603.

Apparently, the problem was caused by my fix r "consider target dirty if
depfile is missing" (bcc8ad1), which was not working correctly with
restat rules cleaning nodes.  Switching to deps only triggered an easily
observable issue.

Fix by setting a flag in edges with invalid deps, and not cleaning edges
with that flag set.

Share more code between CleanNode() and RecomputeDirty().

Move a common loop into the new function RecomputeOutputsDirty().
Simplifies things a bit, and makes it harder for the restat path to have
different behavior from the regular path.

No dramatic behavior change (the restat path now also calls
RestatIfNecessary()).

Check depslog timestamp in LoadDepsFromLog(), not in RecomputeOutputDirty().

RecomputeOutputDirty() is called from two places:

1. RecomputeDirty(), which calls LoadDeps().
2. CleanNode(), which always passes 0 for the deps mtime.

So this is no behavior change in either case.

deps_mtime was nonzero only in deps mode, and it got passed all over the
place.  This makes things simpler.

Merge pull request #647 from bmeurer/SimplifyGetProcessorCount

Simplify implementation of GetProcessorCount().

Simplify implementation of GetProcessorCount().

The current implementation is unnecessarily complex, because:

- The BSD derived systems implement sysconf(_SC_NPROCESSORS_ONLN)
  in terms of sysctl({CTL_HW,HW_NCPU}).
- get_nprocs() is a GNU extension, and glibc implements
  sysconf(_SC_NPROCESSORS_ONLN) in terms of get_nprocs().