Switch LineReader from fgets() to just fread().

fgets() needs to search for a \n in the input, and LineReader's
strstr() did the same work again. By using fread() directly, this
duplicate work can be saved.

(I tried using readline() / fgetln() for the same saving, but those
functions are not standard, and were slower than the code in this CL.)

Results from running build_log_perftest, on OS X:
Before: min 369ms  max 372ms  avg 370.4ms
After:  min 243ms  max 247ms  avg 244.8ms

On Windows:
Before: min 796ms  max 904ms  avg 858.0ms
After:  min 359ms  max 375ms  avg 371.4ms

On Linux:
Before: min 161ms  max 169ms  avg 164.8ms
Before: min 130ms  max 137ms  avg 132.6ms

Pull BuildLog line reading into its own class. No performance or behavior change.

Merge pull request #307 from sgraham/nul-input

pass subprocesses handle to nul device rather than null handle

pass subprocesses handle to nul device rather than null handle

Merge pull request #305 from nico/winperftest

Make parser_perftest and build_log_perftest link on Windows.

Make parser_perftest and build_log_perftest link on Windows.

Merge pull request #304 from nico/perftest

Add a BuildLog loading perftest.

Add a BuildLog loading perftest.

include version number

clarify what i mean by "policy"

Merge pull request #301 from nico/logfix

Don't write ninja log header to log on every build on Windows.

Don't write ninja log header to log on every build on Windows.

Merge pull request #297 from nico/colorclang

Produce colored diagnostics when building ninja with clang.

Merge pull request #299 from tfarina/explicit-ctors

views: Mark single-argument constructors as explicit.

views: Mark single-argument constructors as explicit.

This was reported by cpplint as:

python ~/depot_tools/cpplint.py 2>&1 | grep -v "Done processing" | grep explicit

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

Merge pull request #298 from tfarina/build-log-string

No need to convert log_path to c-string. Two places.

No need to convert log_path to c-string. Two places.

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

Produce colored diagnostics when building ninja with clang.

This can be disabled by having -fno-color-diagnostics in CFLAGS,
or setting TERM to dumb.

mention reactos

mention platform support

Merge pull request #296 from nico/confenv

Let configure.py remember its environment for rerunning.

Merge pull request #294 from nico/fastercanonpath

Make CanonicalizePath() 47% faster on mac, 37% faster on linux.

First check for string end, then dereference. (5ms more expensive, heh.)

Do not access os.environ directly.

Let configure.py remember its environment for rerunning.

Without this, CXX is lost when ninja decides that it's time to
regenerate build.ninja, which makes performance tweaking surprising.

Don't walk path components twice. Speeds up CanonicalizePath() 115ms (285ms -> 170ms).

Reorder a few lines, no functionality (or perf) change.

Skip single '/' characters earlier in the loop. 300ms -> 285ms.

Speed up CanonicalizePath() 6.8% (322ms -> 300ms for chrome empty build).

clarify a flag in configure.py

(The -g flag is always on, --debug just adds e.g. libstdc++'s extra
debugging features.)

Related to issue #286.

add $: to docs

Merge pull request #291 from nico/ocd

Fix layout width in ninja_syntax.py.

Merge pull request #290 from nico/metrics

Add .ninja_log load time to metrics.

Merge pull request #284 from nico/fixwinbuild

Fix Windows build.

Merge pull request #283 from tfarina/string-ampersand

Put & operator with the type rather than the variable name.

Fix layout width in ninja_syntax.py.

The last line would sometimes be needlessly longer than the layout
width. One example is line 67 in the build.ninja generated by
ninja's own configure.py: Before this patch, ninja_syntax would
create a 81 character line.

Add .ninja_log load time to metrics.

On my system, it takes 22% of the empty build time for chrome.

Merge pull request #288 from nico/buildfix

ninja_syntax.build() returns a list, not a single string.

ninja_syntax.build() returns a list, not a single string.

Fixes 'multiple rules' warnings on mac/linux.

Merge pull request #285 from nico/winninjatest

Make it possible to write `ninja ninja_test` on windows (as opposed to `ninja ninja_test.exe`). Also `ninja ninja`.

Merge pull request #287 from nico/gtestnortti

Fix ninja_test build with clang, build without rtti on windows.

Explicitly pass -DGTEST_HAS_RTTI to fix building ninja_test with clang.

Build without RTTI on windows (/GR-), saves 1kB and is consisten with linux/mac.

Provide 'ninja' as alias for 'ninja.exe' too. Tidy up.

Provide 'ninja_test' as alias for 'ninja_test.exe' on windows.

Fix Windows build.

Put & operator with the type rather than the variable name.

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

Merge pull request #274 from polrop/max_load_average

Max load average

emacs: hook up ninja-mode to extension

Merge pull request #282 from iwadon/fix-link-to-gplus

Fix a link to Google+

Fix a link to Google+.

rearrange the manual

My goals were:
- shorten the introductory text, hopefully quickly answering the "why?"
  question first.
- put the information relevant to ninja users (e.g. people using CMake
  who don't especially care about ninja) next, at the beginning.
- consolidate the "generating ninja files" text below.

manual: limit width to 45em

I had a local user stylesheet that was doing this for all sites
so I hadn't noticed how bad it was!

suggest ninja -h when someone runs 'ninja help'

doc encoding support

allow UTF-8 in rule descriptions

The lexer already mostly allowed this, except that chars >127 were
being interpreted as negative indexes into the lexer table.

Merge pull request #278 from tychoish/master

Minor Documentation Tweak

reduce custom ninja status patch in minor ways

Elsewhere in the code I avoid sstream and manual buffer management,
so switch this code to behave similarly.  Sorry for being OCD.

Merge branch 'custom_ninja_status' of git://github.com/polrop/ninja

Merge pull request #272 from nico/statstweak

Switch the order of total and avg columns in -d stats output.

Merge pull request #269 from sgraham/explain

add support for -d explain to help debug why rules are running

Merge pull request #228 from pcc/lazily-create-commandrunner

Lazily create the CommandRunner in Builder::Build

Merge branch 'dont-clean-phony' of git://github.com/pcc/ninja

Conflicts:
src/clean.cc

Merge pull request #207 from iwadon/fix-configure.py-windows

Fix configure.py for unpacked gtest on Windows platform.

Refactor and test progress status formatting.

Introduce NINJA_STATUS env var for customization.

Some people may want to display different relevant information about
the progress of the build.  Also it can be usefull to debug jobs
allocation.

Remove trailing white spaces.

docs: clarifying that there can be only one command per rule

Add -l N option to limit the load average.

This is similar to GNU make -l/--load-average option.  It limits the number
of job started if the load average exceed the given value.  It can be very
useful when running ninja on a continuous integration server where we want
to use parallelism as much as possible without overloading the server.

Remove trailing white spaces.

Switch the order of total and avg columns in -d stats output.

add support for -d explain to help debug why rules are running

Merge pull request #252 from qhuo/remove-depfile-when-cleaning

Remove depfiles when running "ninja -t clean <target>" or "ninja -t clean -r <rule>"

Merge pull request #265 from mathstuf/dev/verbose-bootstrap

Allow the bootstrap to be verbose

Merge pull request #253 from qhuo/include-io.h

Fix windows build (add #include <io.h>)

Make |log_| variable private to BuildLog.

This fix the TODO in build_log.h file.

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

Merge pull request #263 from mathstuf/dev/gcc-470-build-fixes

Include unistd.h where needed

Merge pull request #260 from sgraham/rspfile_syntax

add rspfile and rspfile_content to ninja_syntax

Merge pull request #259 from usovalx/distcc_depfile_fix

Explicitly specify target name for gcc depfiles

Merge pull request #256 from iwadon/fix-sign-compare

fix warning: "comparison between signed and unsigned integer expressions"

Merge pull request #257 from PetrWolf/53598255

Add NOMINMAX macro to fix a compilation error on Win32

Allow the bootstrap to be verbose

Include unistd.h where needed

add rspfile and rspfile_content to ninja_syntax

Explicitly specify target name for gcc depfiles

Merge pull request #251 from polrop/depfile-parser-accept-tilde

Depfile parser accept tilde.

fix warning: "comparison between signed and unsigned integer expressions"

Add NOMINMAX macro to fix a compilation error on Win32

Fix windows build (add #include <io.h>)

Remove depfiles when running when running "ninja -t clean <target>"
or "ninja -t clean -r <rule>"

Depfile parser accept tilde.

Merge pull request #250 from qhuo/SetCloseOnExec-for-Windows

Add a Windows version of SetCloseOnExec()

Merge pull request #249 from qhuo/StatBadPath-Win7

Fix StatBadPath for Windows 7

Add a Windows version of SetCloseOnExec()

Fix StatBadPath for Windows 7

The StatBadPath test expects both Windows and Linux to reject a
path name with 512 characters. However, it seems that such path
is actually acceptable to Windows 7. The change constructs a
different path name that is invalid on Windows.

Merge pull request #246 from HeisSpiter/master

Fix resources leaks

Fix resources leaks

Merge pull request #245 from usovalx/unify_depfile_targets

Issue #241 - handle depfiles generated by older versions of GCC

Issue #241 - handle depfiles generated by older versions of GCC

Older versions of GCC would produce broken depfiles when -MT or -MQ is used
    gcc43 -MT foo.o -MMD -MF foo.o.d -o foo.o -c foo.c
will result in the following depfile
    foo.o foo.o: <dependencies>

Parse multiple outputs unifying duplicates and correctly report errors if
they are different.

Merge pull request #235 from fischman/ppoll

pselect->ppoll on linux to raise the process limit roof