summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Reduce GetFullPathName callsTakuto Ikuta2017-05-091-2/+4
|
* Fix for reviewTakuto Ikuta2017-05-095-20/+20
|
* Make clparser fastertikuta2017-05-086-79/+130
| | | | | | | | | | This patch improves perfromance of clparser. * Reduce the number of calling GetFullPathName. * Use StringPiece for Split and Join. * Add EqualsCaseInsensitive for StringPiece not to generate new string instance. * Add some utility member in StringPiece class.
* Add string_piece_utilTakuto Ikuta2017-04-265-0/+253
| | | | | | | | | | | | | Following functions are implemented for further performance optimization. * JoinStringPiece * SplitStringPiece * EqualsCaseInsensitiveASCII * ToLowerASCII To improve performance of CLParser, I will introduce above functions into include_normalize-win32.cc.
* Merge pull request #1267 from atetubou/clparser_perftestNico Weber2017-04-173-15/+169
|\ | | | | Add clparser_perftest
| * delete nTakuto Ikuta2017-04-121-0/+0
| |
| * use usTakuto Ikuta2017-04-121-2/+2
| |
| * add gitignoreTakuto Ikuta2017-04-122-0/+1
| |
| * remove util.hTakuto Ikuta2017-04-121-1/+0
| |
| * Add clparser_perftestTakuto Ikuta2017-04-122-15/+169
|/
* Merge pull request #1255 from tchajed/bind-localhostNico Weber2017-03-101-2/+6
|\ | | | | browse: Bind to localhost by default
| * browse: Bind to localhost by defaultTej Chajed2017-03-091-2/+6
|/ | | | | | | | | Previously the browse server would bind to "", which is translated to 0.0.0.0 (all interfaces), and then the hostname as retrieved by socket.gethostname() was presented to the user. The hostname is now "localhost" by default and is configurable, so the original behavior is achieved with `ninja -t browse -a ""`.
* Merge pull request #1231 from colincross/canon_perftest_fixNico Weber2017-02-101-1/+1
|\ | | | | Fix build in canon_perftest_fix
| * Make travis build everythingColin Cross2017-02-021-1/+1
| | | | | | | | | | Tell travis to build "all" instead of just "ninja_test". This would have caught the breakage introduced by #1181.
| * Fix build in canon_perftest_fixColin Cross2017-02-021-1/+1
| | | | | | | | | | | | unsigned int*, make the same change in canon_perftest. Fixes "./ninja all" build.
* | Merge pull request #1235 from refack/patch-1Nico Weber2017-02-101-1/+1
|\ \ | | | | | | Suppress warning 4244 to bootstrap on vs2017
| * | Need this to build on vs2017Refael Ackermann2017-02-041-1/+1
| |/ | | | | | | | | | | | | | | | | | | .\src\clparser.cc(56): note: see reference to function template instantiation '_OutIt std::transform<std::_String_iterator<std::_String_val<std::_Simple_types<char>>>,std::_String_iterator<std::_String_val<std::_Simple_types<char>>>,int(__cdecl *)(int)>(_InIt,_InIt,_OutIt,_Fn1)' being compiled with [ _OutIt=std::_String_iterator<std::_String_val<std::_Simple_types<char>>>, _InIt=std::_String_iterator<std::_String_val<std::_Simple_types<char>>>, _Fn1=int (__cdecl *)(int) ] D:\bin\dev\VS\2017\BuildTools\VC\Tools\MSVC\14.10.24911\include\algorithm(946): warning C4244: '=': conversion from 'int' to 'char', possible loss of data
* | Merge pull request #1232 from nicolasdespres/fix-canon_perftestNico Weber2017-02-101-1/+1
|\ \ | | | | | | Fix compilation error in canon_perftest.
| * | Fix compilation error in canon_perftest.Nicolas Despres2017-02-021-1/+1
| |/ | | | | | | Introduced by 1cc730ddc27df52.
* | Merge pull request #1237 from danw/no_reload_with_restatNico Weber2017-02-101-4/+13
|\ \ | |/ |/| Support restat when rebuilding manifest
| * Support restat when rebuilding manifestDan Willemsen2017-02-081-4/+13
|/ | | | | | | | As a fix for #874, we started reloading the entire manifest even if the manifest was never rebuilt due to a restat rule. But this can be slow, so call State::Reset instead, which also fixes the original crash. Fixes #987
* Merge pull request #1181 from DanielWeber/issue-1161Nico Weber2017-01-2412-48/+57
|\ | | | | Allow more path components
| * Use uint64_t for slash_bits consistentlyDaniel Weber2016-09-015-14/+15
| | | | | | | | The VS compiler complained about possible loss of data (and it was right!)
| * Allow more path componentsDaniel Weber2016-08-229-36/+44
| | | | | | | | | | | | | | - 60 instead of 30 path components - 64 instead of 32 backslashes in a path (windows only) Issue: 1161
* | Merge pull request #1226 from colincross/close-fdsNico Weber2017-01-131-0/+2
|\ \ | | | | | | Close original pipe fd in subprocesses
| * | Close original pipe fd in subprocessesColin Cross2017-01-131-0/+2
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Non-console subprocesses have the write end of a pipe connected to fds 1 and 2 for stdout and stderr, but they also have the it connected to whatever fd was assigned in the ninja process when the pipe was created. Add a call to posix_spawn_file_actions_addclose after the posix_spawn_file_actions_adddup2 calls to close the original fd once it has been dup'd to stdout and stderr. This fixes an issue seen in the Android build, where one of the subprocesses is used to start a background helper process. The background process attempts to close any inherited fds, but if ninja used a very large fd number due to a high parallelism count the background process would not close the fd and ninja would never consider the subprocess finished.
* | Merge pull request #1225 from bippum/patch-1Evan Martin2017-01-121-1/+1
|\ \ | | | | | | fix broken link in hacking.md
| * | fix broken link in hacking.mdBrendan McCarthy2017-01-111-1/+1
|/ /
* | Merge pull request #1220 from phajdan/masterNico Weber2017-01-081-0/+7
|\ \ | | | | | | Fix build with uclibc
| * | Fix build with uclibcPawel Hajdan, Jr2017-01-021-0/+7
|/ / | | | | | | | | | | | | | | Resolves #985 This is based on musl implementation, http://git.musl-libc.org/cgit/musl/commit/?id=20cbd607759038dca57f84ef7e7b5d44a3088574 (thanks to jbergstroem@ for reference)
* | Merge pull request #1213 from nico/copyingNico Weber2016-12-061-1/+1
|\ \ | | | | | | replace copyright placeholder, fixes #1212
| * | replace copyright placeholder, fixes #1212Nico Weber2016-12-061-1/+1
|/ /
* | Merge pull request #1206 from nico/versioninfoNico Weber2016-11-081-12/+7
|\ \ | | | | | | windows: replace deprecated GetVersionEx with recommended replacement
| * | windows: replace deprecated GetVersionEx with recommended replacementNico Weber2016-11-081-12/+7
|/ / | | | | | | | | | | | | | | The recommended replacement VerifyVersionInfo should work with the same SDKs that GetVersionEx worked with (while the wrappers in VersionHelpers.h require a recent SDK). This patch should not change behavior, and it's not supposed to increase build requirements. If this makes things harder to build, please let me know.
* | Merge pull request #1205 from nico/clangclformatNico Weber2016-11-081-1/+1
|\ \ | | | | | | fix a clang-cl -Wformat warning
| * | fix a clang-cl -Wformat warningNico Weber2016-11-081-1/+1
| | |
* | | Merge pull request #1204 from nico/vernumNico Weber2016-11-081-2/+3
|\ \ \ | | | | | | | | fix RELEASING wrt manual.asciidoc process
| * | | fix RELEASING wrt manual.asciidoc processNico Weber2016-11-081-2/+3
|/ / /
* | | Merge pull request #1194 from ilor/depfile-empty-pathNico Weber2016-11-081-1/+3
|\ \ \ | | | | | | | | Improve error message when a depfile contains a bad path
| * | | Improve error message when a depfile contains a bad pathTomasz Śniatowski2016-10-121-1/+3
| | | | | | | | | | | | | | | | | | | | Debugging "ninja: error: empty path" is not fun, so make the error message mention the depfile name.
* | | | Merge pull request #1182 from moosotc/masterNico Weber2016-11-081-2/+3
|\ \ \ \ | | | | | | | | | | Use POSIX_SPAWN_USEVFORK if available
| * | | | Use POSIX_SPAWN_USEVFORK if availablemalc2016-08-261-2/+3
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Existing comments were alluding to it's usage and it makes building ninja itself go a litle bit faster (i.e. taking less wall clock time). FWIW FreeBSD even uses vfork by default c.f. https://www.freebsd.org/cgi/man.cgi?query=posix_spawn
* | | | Merge pull request #1203 from nico/reldocNico Weber2016-11-071-6/+6
|\ \ \ \ | | | | | | | | | | Update RELEASING -- manual.asciidoc no longer has a version number
| * | | | Update RELEASING -- manual.asciidoc no longer has a version numberNico Weber2016-11-071-6/+6
|/ / / /
* | | | Merge pull request #1201 from nico/singlecommandNico Weber2016-11-071-5/+35
|\ \ \ \ | | | | | | | | | | teach -t commands to optionally print only the final command
| * | | | teach -t commands to optionally print only the final commandNico Weber2016-11-061-5/+35
| | | | |
* | | | | mark this 1.7.2.gitNico Weber2016-11-071-1/+1
| | | | |
* | | | | Merge pull request #1202 from nico/ppollbsdNico Weber2016-11-073-10/+9
|\ \ \ \ \ | | | | | | | | | | | | Only run SubprocessTest.SetWithLots on FreeBSD when ppoll() exists.
| * | | | | Only run SubprocessTest.SetWithLots on FreeBSD when ppoll() exists.Nico Weber2016-11-073-10/+9
|/ / / / / | | | | | | | | | | | | | | | Should fix #1189 after #1185.
* | | | | Merge pull request #1198 from jamesr/fix_write_fake_manifestsNico Weber2016-11-071-2/+2
|\ \ \ \ \ | | | | | | | | | | | | Fix syntax error in misc/write_fake_manifests.py
generated by cgit v0.12 at 2024-11-16 00:08:34 (GMT)