Remove unneeded save-excursion.

syntax-propertize-function is allowed to move the point and mark.

mark this 1.5.3.git

Merge pull request #864 from Fanael/ninja-mode.el-refactor

Ninja mode.el refactor

Use lexical-binding.

There's no reason not to use lexical-binding when supporting only
Emacs 24+. Its semantics are just that much saner.

Add a Emacs 24 requirement.

Only Emacs >= 24 has prog-mode.

Correctly recognize a comment if the previous line is a comment ending in $.

Avoid putting properties past the end of the buffer.

Don't use dotted list syntax unless necessary.

Remove unnecessary regexp group.

Correctly recognize comments.

Set up a proper syntax table for ninja-mode.

Since quotes are not meant to be treated as string delimiters,
the syntax table is the place to tell Emacs so.

This also means syntactic fontification can be reenabled and the
font-lock keyword entry for comments removed.

Set comment-start buffer-locally, not globally.

Use quasi-quoting instead of list+cons.

Reindent ninja-mode.el using Emacs.

Emacs knows best how to indent Emacs Lisp.

Use double semicolon comments.

Don't leave lone closing parens.

no subshell

don't need subshell to send stderr to /dev/null

Merge pull request #772 from drbo/syntax_bugfix

Added highlighting in Emacs for ${...} variables

Merge pull request #866 from nico/winfix

Fix Windows build after #862. Thanks to @harig for the report.

Fix Windows build after #862. Thanks to @harig for the report.

Merge pull request #862 from nico/browse

Make browse detection consistent with other platform checks.

Merge pull request #863 from nico/nopthread

Stop linking pthread.

Remove now-unused variable test_libs.

Stop linking pthread.

It was only needed by gtest, which is no longer used.
(Intesting note: I checked when the -lpthread flag was added, and it's
been around since the first revision of build.ninja, which used to be
checked in before configure.py existed. Back then, it looks like '@' was
used to dereference built-in variables, and build outputs were also
prefixed by '@'!).

Make browse detection consistent with other platform checks.

Merge pull request #861 from nico/nosub

Remove duplicate import. No behavior change.

Remove duplicate import. No behavior change.

Merge pull request #859 from martine/platform-helper

merge platform_helper into configure script

merge platform_helper into configure script

With this code all in one place, it's easier to spot unused code and
simplification opportunities.

drop leftover references to bootstrap.py

Merge pull request #858 from martine/travis-bootstrap

switch travis to use new --bootstrap mode for building

Merge pull request #813 from cdlm/emacs-mode-require-newline

Emacs mode: inherit from prog-mode

Merge pull request #836 from nocnokneo/patch-1

Mention optional files in installation instructions

switch travis to use new --bootstrap mode for building

Merge pull request #854 from martine/bootstrap

add a --bootstrap mode for configure.py

drop NINJA_BOOTSTRAP define

Merge pull request #857 from martine/travis-python-test

also test ninja_syntax.py in travis

also test ninja_syntax.py in travis

drop bootstrap.py

We now use configure.py --bootstrap.  Direct users of this script to
use the other one.

add a --bootstrap mode for configure.py

Instead of bootstrapping through a separate script, instead make
configure.py able to either generate a build.ninja *or* just execute
all the computed commands to build a ninja binary.

add an "expand" function to ninja_syntax

Implements basic variable expansion for use in configure.py.

Merge pull request #856 from nico/TABS

emacs: Remove an empty line, wrap a comment.

emacs: Remove an empty line, wrap a comment.

Merge pull request #842 from nico/solaris

Fix building on Solaris.

Merge pull request #852 from nico/browse

Make sure configure.py and ninja.cc always agree on if -t browse is incl...

Make sure configure.py and ninja.cc always agree on if -t browse is included.

No behavior change on most platforms. On solaris, -t browse was compiled in
in ninja.cc but browse.cc wasn't compiled in, which probably means that building
on Solaris didn't work. It might be better now.

This also makes browse.cc automatically not included in bootstrap builds;
previously this was done manually through the NINJA_BOOTSTRAP check.

Merge pull request #851 from nico/cleanup

Remove unused variables.

Remove unused variables.

Merge pull request #850 from sgraham/ocd

whitespace/comment/wrap fixes, no intended functionality change

Merge pull request #849 from sgraham/set-err-canon-components

set *err when too many components in CanonicalizePath

whitespace/comment/wrap fixes, no intended functionality change

set *err when too many components in CanonicalizePath

Merge pull request #848 from sgraham/canon-maintain

Maintain bitvector of original slashes from manifest when canonicalizing

properly guard against slash_bits overflow

fix not respecting length

no need to save slash_bits, add comment

save slash_bits in depslog

no need to Decanonicalize on non-Windows

fix CanonicalizePath going past StringPiece length + test

initialize slash_bits on non-win

make all GetNode explicit, add DepsLog canonicalize test

non-win compilation

improve test

assert no slashes in default GetNode

fix order of args to CanonicalizePath

remove CanonicalizePath overloads, test for toplevel behaviour

path decanonicalization when building command

fix multiple sequential slashes

wip on adding tests at higher level, some not right

track back->forward conversions in a bitmask

Merge pull request #822 from tzik/throttle

Throttle the number of pending commands by the parallelism configuration

Merge pull request #825 from Rapptz/master

Suppress warnings from building with MinGW

Fix building on Solaris.

"SunOS" and "Solaris" are the same thing these days, so make them go
down the same code paths. In particular, the browse feature was omitted
on solaris but not sunos5, causing trouble for some folks (see #838).

Merge pull request #841 from sgraham/strchr

Use strchr in \ conversion in CanonicalizePath on Windows

Use strchr in \ conversion in CanonicalizePath on Windows

Merge pull request #840 from sgraham/canon-win-squashed

CanonicalizePath handles \ on Windows

CanonicalizePath handles \ on Windows

Mention optional files in installation instructions

Merge pull request #837 from ppbrown/master

Update line_printer.cc with termios.h

Update line_printer.cc

Include POSIX termios.h

Merge pull request #834 from frasercrmck/master

Add zsh completion for targets in conjunction with -C

Add zsh completion for targets in conjunction with -C

zsh can now list completions for targets in the directory specified by
the -C option

Fix unknown pragma warnings

Throttle the number of pending commands by the parallelism configuration

Merge pull request #823 from nico/conf

Make auto-reconfiguring work if CFLAGS contains more than one flag.

Make auto-reconfiguring work if CFLAGS contains more than one flag.

When using an open-source clang on OS X, one has to pass an isysroot
flag so that it can find system headers (stdio.h), like so:

  CXX=path/to/clang++ CFLAGS="-isysroot $(xcrun -show-sdk-path)" ./configure.py

Previously, configure.py wouldn't quote envvars containing spaces, so
it'd rerun this as

  CXX=path/to/clang++ CFLAGS=-isysroot /sysroot/path ./configure.py

which would then die because /sysroot/path wasn't excecutable.

Merge pull request #820 from nico/ohceedee

Don't print "Recompacting..." message from tests.

Don't print "Recompacting..." message from tests.

Now tests don't print anything. Non-test behavior is unchanged.

Merge pull request #819 from nico/win

Fix building tests on Windows again.

Fix building tests on Windows again.

Turns out gtest was pulling in sys/stat.h, and we were
using stat() through that in tests. This doesn't work
with old MSVCs, so we should probably replace that with
RealDiskInterface in a follow-up.

Merge pull request #818 from nico/lesstest

remove more things not needed without gtest

remove more things not needed without gtest

Merge pull request #817 from nico/warn

fix warning

fix warning

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.