reactos/ninja
1
0
Fork 0
mirror of https://github.com/reactos/ninja.git synced 2025-01-23 11:44:44 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

performance testing note

Browse Source
This commit is contained in:
Evan Martin 2011-12-07 12:30:59 -08:00
parent 8aae49d6f7
commit b922101633
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
HACKING
View File

@ -9,6 +9,15 @@ Test-driven development:
Remember to build "all" before committing to verify the other source
still works!
Testing performance impact of changes:
If you have a Chrome build handy, it's a good test case.
Otherwise, https://github.com/martine/ninja/downloads has a copy of
the Chrome build files (and depfiles). You can untar that, then run
"ninja chrome". I often do something like:
(for i in `seq 5`; do time -p ninja chrome) 2>&1 | grep real > old
(for i in `seq 5`; do time -p ninja-new chrome) 2>&1 | grep real > new
and then compare those two lists of timings either by eye or with R.
Coding guidelines:
- Function name are camelcase.
- Member methods are camelcase, expect for trivial getters which are