Revert "Dump state for debugging"

This reverts commit 9b196dc806e57cefd88bbbacd12286447dbf9ad9.

Uninitialized variable! There is always one...

Dump state for debugging

Make edge dump pool name, and skip default pool

cover the nulled pool case

add docs

Add some tests!

begin rationalizing platform for both parsers

block parse method done

all building and tests passing

Dump pools with State

Fix a bug... now chrome build works O_O

and some basic implementation

stub out an api and de-constify Pool

Const ref FTW

Pull graph.cc too

Pull out base changes to state

refactor repeated code in cleaner

delete some obsolete TODOs

trailing whitespace

Merge pull request #453 from curinir/bash_completion

Add support for reading directory for bash completion.

Add support for reading directory for bash completion.

Currently ninja bash completion only works in the directory where the
build.ninja files is located. This adds support for using the -C
argument to ninja.

Merge pull request #451 from ppuryear/browse-python3

browse.py: Support Python 3.

browse.py: Don't truncate error message if ninja -t query fails.

Signed-off-by: Philip Puryear <philippuryear@gmail.com>

browse.py: Fix truncation with an unknown target.

Signed-off-by: Philip Puryear <philippuryear@gmail.com>

browse.py: Python 3 support.

Signed-off-by: Philip Puryear <philippuryear@gmail.com>

fix bad merge

mingw helper: drop unused -r flag

update mingw docs for ubuntu precise

fix whitespace

bootstrap: fail more gracefully if gcc isn't available

Merge pull request #450 from zchothia/zc/py3k

Add support for Python 3

Python scripts: use built-in print function.

Add support for Python 3

Merge pull request #441 from tfarina/missing-virtual

Add missing 'virtual' annotation to ReadFile() override.

Add missing 'virtual' annotation to ReadFile() override.

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

Merge pull request #448 from maximuska/proposed/buildlog-mini-refactor2

Proposed/buildlog mini refactor2

build log: fixing parameter names

Merge pull request #446 from maximuska/proposed/start-end-edge-metrics

build metrics: StartEdge and EndEdge

build metrics: StartEdge and EndEdge

Total build time is also relevant.
The costs of StartEdge and EndEdge turned out to be quite surprising.
Especially if multiple targets are specified on edges.
 (I've tried declaring depfiles targets explicitly)

build log: mini-refactoring to use constructors to initialize entries

build log: moving HashCommand() calculation out of targets loop

build log: Adding test for multiple target edges

Merge pull request #439 from wolfp/metric-recompact

Add metric for log recompacting

Add metric for log recompacting

Merge pull request #438 from wolfp/clean-efficiency

Improve the efficiency of -t clean

Improve the efficiency of -t clean

Prevent each node from being examined for cleaning multiple times, if it
is used in several other nodes

Merge pull request #437 from mo/master

[PATCH] Fixed two spelling errors in ninja

Fix two spelling errors

Merge pull request #436 from tfarina/dry-run-unnamed-namespace

Move DryRunCommandRunner into a unnamed namespace.

Move DryRunCommandRunner into a unnamed namespace.

No functional change.

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

Merge pull request #433 from polrop/fix-help-option

Don't say -h is invalid whereas it is supported.

Don't say -h is invalid whereas it is supported.

getopt_long(3) was reporting "ninja: invalid option -- h" when "ninja -h" was
called.

Regression most probably introduced by 5fdb12ed5cec4e1c853c64026142d088ff5519e1

Merge pull request #428 from maximuska/proposed/rpm-build-fixed

packaging: refactored rpm building

packaging: refactored rpm building

Now passing through a valid SRPM, working in a standard way.

Merge pull request #427 from jonforums/jf/mingw-n-msvc

fix mingw build fail - redux

Merge pull request #426 from sgraham/spaces-in-headers-2

fix spaces in headers for -t msvc

Always include stdio.h

less random comment

review fixes

Silence bothersome warning from -Wextra

Struct initializations such as those in `CLWrapper::Run` of file
`src/msvc_helper-win32.cc` causes MinGW GCC to spew warnings.

Give MinGW builds MSVC build helper superpowers

Note: _WIN32 is used instead of WIN32 to enable builds with MSVC
IDE, Windows SDK non-IDE command line tools, and mingw/mingw-w64
based toolchains

fix spaces in headers for -t msvc

Merge pull request #419 from syntheticpp/rate-buffer-size

Buffer size passed to snprintf is wrong in rate code

Fix subtile buffer size error

The deduced type was char* with size 4 and not char[32] with size 32.
This removes strange output characters on Windows.

build with msvc2012

remove some code duplication

Merge pull request #422 from sgraham/fix-dupe-includes

don't emit duplicate headers for msvc helper

don't emit duplicate headers for msvc helper

Merge pull request #416 from nico/slide

Change rate measurement code.

Merge pull request #415 from nico/getopt

Clean up getopt_long call.

Change rate measurement code.

For %o, remove a superfluous + 0.5: snprintf("%f") rounds already.
Remove some unnecessary code.

For %c, fix a TODO to add a sliding window and update after every
completed edge. Else, with -j50 and several files that take 3s to
compile each, this number would only update every 150s. Also give
the number one decimal place so that this can measure steps slower
than 1s.

Clean up getopt_long call.

Remove now-unimplemented 'V' from getopt_long. Remove 'h', since it's
included in the long options. Order switch cases in the same order as in
the getopt_long argument.

mark msvc-helper as experimental

windows: merge msvc-helper into ninja.exe itself

Now "ninja -t msvc ..." passes the "..." to the msvc helper main.
This drastically simplifies bootstrap and makes ninja a single
binary again.

Merge pull request #414 from riannucci/master

Fix re2c detection in bootstrap which breaks win32 w/ gnuwin32

Fix re2c detection which breaks win32 w/ gnuwin32

scoping workaround for gcc on Windows

From https://github.com/martine/ninja/issues/410.

include termios for solaris

add solaris platform definitions

disable a new warning that popped up on Windows

This pattern is safe as long as you're careful; we don't use it
very much.

add GetProcessorCount() implementation for Solaris

drop special case for msvc bucket count

Calling bucket_count() works locally with MSVC.  I wonder if some
other change in the code fixed this.

Merge pull request #408 from nico/fix

Make sure no stale test file exists before running build log tests.

allow tools to specify when they run (before/after build.ninja load)

This will be necessary to inline msvc-helper as well as so -t graph
can get depfiles.

factor out metrics dump from ninja main

factor out build log load from main

rearrange tool-picking logic

Now "ninja -t list" works from any directory.

show all in graph

reduce indent

move BuildConfig out of globals

remove DiskInterface from globals

remove unfortunate header dependency

This was temporarily added, and now it can be removed.

clarify setter

move BuildLog to DependencyScan

The build log is needed in computing whether an edge is
dirty, so I think it belongs here.  (It's a bit weird
that Builder needs to reach into it to record completed
commands, maybe it will become cleaner with more thought.)

fix --debug for glibc pedantic mode

pass Builder as arg to build-running functions

Reducing use of globals.

remove config from BuildLog, rename members

remove a redundant arg to RecomputeOutputDirty

split out dirty recomputation logic from Edge class

Rather than passing States and DiskInterfaces through all the calls,
put the necessary ambient information in a new DependencyScan object
and move the code accordingly.

Note: I didn't move the source location of the functions to preserve
history, though this does result in a sort of weird order for the
functions in graph.cc.