Fix the TODO in Rule class.

Move various data members to private section and provide accessors for them.

Signed-off-by: Thiago Farina <tfarina@chromium.org>

windows: handle ERROR_PATH_NOT_FOUND

From Frances <frances.buontempo@gmail.com>.

Merge pull request #163 from jsternberg/master

a fix for issue #162

Matching the depfile_parser.in.cc file to properly generate the depfile_parser.cc file that's in version control.

Merge pull request #161 from nico/fix

Let ninja_syntax handle escaped spaces correctly.

Let ninja_syntax handle escaped spaces correctly.

Revert the main loop changes made in 2e7ab7514207ea7faad40faedf3fc9d72b3adf7a,
and add just a few lines to the original main loop to make '$ ' escaping work.
Add several new tests, and make the existing tests pass again.

revert part of c33b198 to fix build of ninja_test

Merge pull request #160 from tfarina/upper-case-error

Capitalize "error" to match with the other utility functions.

Capitalize "error" to match with the other utility functions.

Fatal and Warning functions already output their strings capitalized,
we were just missing the Error function. So capitalize it now.

Signed-off-by: Thiago Farina <tfarina@chromium.org>

Merge pull request #159 from tfarina/protect-build-log-vars

Protect BuildLog data member variables by making them private.

Protect BuildLog data member variables by making them private.

They are not accessed outside of BuildLog and there is even a SetConfig function
to set the |config_| variable.

So better to make them private to BuildLog now while nobody is using it outside.

Signed-off-by: Thiago Farina <tfarina@chromium.org>

Merge pull request #158 from tfarina/alphabetize

Put the list of core source files in alphabetical order.

Put the list of core source files in alphabetical order.

Signed-off-by: Thiago Farina <tfarina@chromium.org>

windows: use GetFileAttributesEx instead of stat

From a Hacker News comment:
"Recent finding, that sped up our systems from 15->3sec on 300,000+ files
filestamp check was to move from _stat to GetFileAttributesEx."

I do recall reading that calls to stat() on Windows were one of the
potential reasons Subversion is so slow on Windows...

gitignore exe files

factor out windows perror equivalent

inline gtest to simplify windows build

fix bad merge

fix mingw cross-compile

rearrange manual to centralize quoting rules

remove weird accidental double-return

Surprised this isn't a compiler warning.

Merge pull request #146 from nornagon/escape-spaces

Escape spaces

adjust depfile test now that parsing is more lax

handle backslashes and isolated colons in depfile parser

The logic was wrong if the input looked like
  foo : bar baz
with a space before the colon.

Test from Frances <frances.buontempo@gmail.com>.

split out depfile parser tests into new file

print 'entering directory' when -C is used

This allows Emacs to track what directory you're in.

Patch from Ami Fischman <fischman@chromium.org>.

Merge pull request #153 from polrop/fix-bootstrap

Fix bootstrap

Fix 'list' tool.

Factor source file listing code in bootstrap script.

Fix bootstrap script.

- Make regexp more robust.
- Exclude .in.cc files.

Use getter instead of private member.

Fix a compilation error.

performance testing note

remove makefile parsing code, use depfile code instead

use re2c to parse depfiles

refactor EdgeEnv

make Rule::generator_ and restat_ private

make Rule::name_ private

remove unused function

make Node::out_edges_ private

make Node::in_edge_ private

make Node::dirty_ private

more whitespace

merge StatCache into State

I think I had originally imagined StatCache would contain more state,
but at this point it's clear it was just managing a single map, which
could just as well be in the already-small State object.

merge FileStat into Node

The two were always one-to-one anyway.  I started adding accessors
to FileStat and then realized most users wanted them on Node and
that forwarding them through was silly.

refactor to remove Node::ready()

add and use getter for Edge::rule_

move build/install to README

Merge pull request #152 from pcc/clean-depfiles

Have the clean tool remove depfiles

Merge pull request #151 from pcc/restat-depfiles

Use the modification time of the depfile when computing restat_mtime

Have the clean tool remove depfiles

Use the modification time of the depfile when computing restat_mtime

Fixes issue #144.

Factor out Edge::EvaluateDepFile

more whitespace in build output

remove reference to nonexistent file

add help output to clean tool

prettify manual for tools

also spell-check the 'clean' tool; abort on misspelling

minor manual touchups

split out tool list into a separate subcommand

Fits the -h output on one screen again, and allows more whitespace
in both the -h output and the tool list.

disable the 'unused parameter' warning

It was firing too often, and hadn't uncovered any bugs.

Merge pull request #150 from luislavena/mingw-no-visibility

visibility=hidden should not be used for Windows

Merge pull request #149 from luislavena/mingw-static

Use g++'s -static option instead of specific libs

visibility=hidden should not be used for Windows

While GCC 4.x supports the usage of -fvisibility=hidden and the related
pragmas, usage of it under MinGW is a noop and should not be used.

Instead, __declspec(dllexport/dllimport) should be used instead to
indicate exportable functions.

Since Ninja builds a static library, no functions are exported and
visibility, at least under MinGW, is not an issue.

Use g++'s -static option instead of specific libs

MinGW g++ recognizes -static as option to avoid the need to specify
individual libraries to be statically linked into the executable.

It also solves the warning of -static-libstdc++ not being recorgnized by
GCC 4.5.2 (TDM build)

fix some minor style issues in RecomputeDirty

when an edge is dirty, mark all outputs dirty

The logic before was like:
  for each output:
    if edge_dirty or output.dirty:
      output.dirty = true
      edge_dirty = true

This was wrong in the case where the second output would case the edge
to be dirty; we needed to go back and mark the first output dirty as
well.  Fixed by taking two passes: one to compute the dirty state,
then a latter sweep to mark all outputs.

Fixes issue 148.

refactor RecomputeOutputDirty to return true/false for dirty

Rather than taking whether the edge is dirty as an input, instead
only consider the arguments and return true/false, allowing the caller
to decide what to do with that information.  (In the restat case we
were faking out the environment to get particular behavior for this.)

Should have no side effects, but this is part of fixing issue 148.

add a test case from issue 148

windows: don't buffer stdout

HACKING: windows notes

bootstrap.sh: windows (mingw) fixes

- Don't build browse.cc in bootstrap.sh.
- Explicitly call out to Python to run configure.py.
- Run ninja-bootstrap with no arguments to build the default target,
which is "ninja.exe" on Windows.

configure.py: windows updates

- Drop unused 'windows' platform
- Make 'mingw' platform actually work on mingw-on-Windows.

windows: clear process launch structs before using them

Update ninja_syntax.py and tests to reflect '$ ' syntax.

Add tests for escaping spaces with '$ '.

Revert "Merge pull request #143 from cipriancraciun/patches/python2"

This reverts commit 38ab41f45ff818b437942b753328a0168914fc86, reversing
changes made to 819d6347b424f583d651b86dd1280605ddb23b88.

Platforms that don't have /usr/bin/python pointing to python2 are broken.

Merge pull request #143 from cipriancraciun/patches/python2

Rename all occurences of `python` to `python2`

Merge pull request #142 from cipriancraciun/patches/custom-cflags-ldflags

Added support for custom CFLAGS and LDFLAGS.

Renamed `python` to `python2`.

Fixed Python 2 scripts prologue (i.e. `#!/usr/bin/env python2`).

Added support for custom CFLAGS and LDFLAGS.

Merge pull request #138 from polrop/fix-warning-in-test

Fix compilation warning in Ninja's test suite.

Fix compilation warning in Ninja's test suite.

Allow '$ ' to escape spaces in identifiers.

Merge pull request #136 from polrop/add-extra-warning

Add -Wextra compilation flags

Remove warning triggered by -Wextra on MinGW.

Add -Wextra warning flag.

Fix triggered warnings:
- unused parameter
- type qualifiers ignored on function return type
- missing initializer for member

Merge pull request #135 from polrop/fix-mingw

Fix mingw.

Merge pull request #133 from polrop/fix-duplication

Fix duplicated list of profilers.

Fix compilation warning with getopt() on MinGW.

Fix compilation warning with getopt_long() on MinGW.

Use more portable way to get the current process ID.

GetProcessId() is available only since Windows XP.  Since
MinGW define WINVER to 0x0400 which is Windows 2000 I think, we have
a compilation error.  Using GetCurrentProcessId() instead of
GetProcessId(GetCurrentProcess()) fix this issue.

Fix duplicated list of profilers.

Merge pull request #132 from nico/spell

Add spelling suggestions for four cases:

Add spelling suggestions for four cases:

1. For targets, when invoking ninja to build a target.
2. For targets, when doing a "query" command.
3. For command names.
4. For the subcommands of the "targets" command.

Also change CmdTargets() to call LookupNode() instead of GetNode() --
since the result was checked for NULL, that's probably what was intended
here originally.

fix TODO: correct dependencies for inline.sh

Not sure why I just didn't write this right the first time; when running
a script for its output, the script is always an implicit input of the
output.

Add a EditDistance() function based on the one in llvm/lib/Support/StringRef.cpp.

Merge pull request #131 from secondplanet/patch-1

Improvements to ninja-mode

Improvements to ninja-mode

* provide ninja-mode feature
* highlight rule names

Merge pull request #127 from tiresdonexits/master

Minor code change to address a warning in g++ 4.6.