0
Commit Graph

13 Commits

Author SHA1 Message Date
dcheng
f239071ad5 clang update script: add --extra-tools, deprecate --tools flag
plugins and blink_gc_plugin are always mandatory for Chromium builds,
so just always include them. In order to avoid breaking codesearch
bots, --tools is now a synonym for --extra-tools and will be removed.

BUG=none

Review-Url: https://codereview.chromium.org/2609683002
Cr-Commit-Position: refs/heads/master@{#441601}
2017-01-05 06:46:28 +00:00
lukasza
f9b89e7bd4 Split run_tool.py into run_tool.py, extract_edits.py and apply_edits.py
The split will allow generation of edits on multiple configs (e.g. linux
vs windows OR rel vs dbg) and merging the edits before applying them once:
    $ tools/clang/scripts/run_tool.py rewrite_to_chrome_style \
        --generate-compdb --all out/rel >run_tool.linux.rel.out
    $ ...
    $ cat run_tool.*.out \
        | tools/clang/scripts/extract_edits.py \
        | tools/clang/scripts/apply_edits.py
        --generate-compdb --all out/rel >run_tool.linux.rel.out

Test steps:
- tools/clang/translation_unit/test_translation_unit.py
- tools/clang/scripts/test_tool.py rewrite_to_chrome_style
- manually running run_tool | extract_edits | apply_edits pipeline
  on WTF and verifying that it still builds after the rename

BUG=598138
TEST=See "Test steps" above.

Review-Url: https://codereview.chromium.org/2599193002
Cr-Commit-Position: refs/heads/master@{#440881}
2016-12-28 19:45:27 +00:00
danakj
ca6b31b571 Update Windows-specific docs for the clang rewriting tool.
R=dcheng@chromium.org

Review-Url: https://codereview.chromium.org/2596383002
Cr-Commit-Position: refs/heads/master@{#440516}
2016-12-22 22:09:42 +00:00
qyearsley
c0dc6f499d Fix spelling mistakes in //docs.
This includes spelling mistakes like:

accomodate->accommodate
ommited->omitted
unuseable->unusable
dependant->dependent
enviroment->environment
preceed->precede
occurrance->occurrence
occurrances->occurrences
intergrated->integrated
altough->although
appriopriate->appropriate
corresponing->corresponding
dependecies->dependencies
leniant->lenient
overriden->overridden
persisten->persistent
specifyinhg->specifying

As well as capitalization changes like:

javascript->JavaScript
Webkit->WebKit
linux->Linux

A couple other minor things:

the the -> the
trybot -> try bot
etc.

Review-Url: https://codereview.chromium.org/2551513002
Cr-Commit-Position: refs/heads/master@{#436046}
2016-12-02 22:16:53 +00:00
lukasza
1333f1b925 Skip ParmVarDecl nodes without a name.
BUG=598141

Review-Url: https://codereview.chromium.org/2246263002
Cr-Commit-Position: refs/heads/master@{#414877}
2016-08-27 00:27:15 +00:00
danakj
30d0f8c92a clang tools: Add --bootstrap to the documentation for update.py
This lets us build the clang tools with clang not gcc, which appears to
have bugs!

Also work around the gcc bug in one place we know it happens anyways,
just because.

R=dcheng
BUG=580745

Review URL: https://codereview.chromium.org/1647733002

Cr-Commit-Position: refs/heads/master@{#371933}
2016-01-28 00:27:53 +00:00
Daniel Cheng
9ce2a30f5e Fix clang tool refactoring docs formatting.
Argh... whitespace!

BUG=none
R=danakj@chromium.org

Review URL: https://codereview.chromium.org/1592853003 .

Cr-Commit-Position: refs/heads/master@{#369903}
2016-01-16 01:19:45 +00:00
danakj
ef9f1facc0 docs: Fix links/shell with whitespace changes.
Links shouldn't have whitespace inside them, shell snippets
need whitespace around them.

R=dcheng

Review URL: https://codereview.chromium.org/1592883002

Cr-Commit-Position: refs/heads/master@{#369894}
2016-01-16 00:38:18 +00:00
dcheng
ce2375ee82 Update documentation for clang tool refactoring.
BUG=none

Review URL: https://codereview.chromium.org/1577483003

Cr-Commit-Position: refs/heads/master@{#368748}
2016-01-12 01:10:16 +00:00
Nico Weber
a219155337 Update tool docs to use --tools flag
BUG=494442
R=dcheng@chromium.org

Review URL: https://codereview.chromium.org/1493893002 .

Cr-Commit-Position: refs/heads/master@{#362775}
2015-12-02 19:44:22 +00:00
Nico Weber
e250e6adb0 Remove stray references to update.sh from src/
BUG=494442
R=hans@chromium.org

Review URL: https://codereview.chromium.org/1489223003 .

Cr-Commit-Position: refs/heads/master@{#362772}
2015-12-02 19:34:13 +00:00
andybons
6eaa0c0d01 Markdown style fixes for:
code_coverage.md
cocoa_tips_and_tricks.md
closure_compilation.md
clang_format.md
clang_tool_refactoring.md
clang.md

R=nodir
BUG=524256

Review URL: https://codereview.chromium.org/1306233003

Cr-Commit-Position: refs/heads/master@{#345652}
2015-08-26 20:13:51 +00:00
andybons
3322f7611b Per https://groups.google.com/a/chromium.org/forum/#!topic/chromium-dev/irLAQ8f8uGk
Initial migration of wiki content over to src/docs

There will be a follow-up CL to ensure docs are following chromium’s style guide, links are fixed, etc. The file auditing was becoming too much for a single change and per Nico’s suggestion, it seems to be better to do

+ Bulk import with initial prune.
+ Follow-up CLs to clean up the documentation.

So that each CL has its own purpose.

BUG=none

Review URL: https://codereview.chromium.org/1309473002

Cr-Commit-Position: refs/heads/master@{#345186}
2015-08-24 21:39:36 +00:00