Add python script that can add "#pragma once" to all our files.

Add python script that can add "#pragma once" to all our files.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=53271

[Nico, 2010-07-21 23:05:08]

Not the prettiest, but seems to work.

[Elliot Glaysher, 2010-07-21 23:07:58]

On 2010/07/21 23:05:08, Nico wrote:
> Not the prettiest, but seems to work.

As scripts go, middle of the road prettiness. LGTM.

[Evan Martin, 2010-07-22 00:44:54]

I feel like this might've been a one-liner sed script.  I'm just not sure of hte
line.

http://codereview.chromium.org/2842064/diff/5001/6001
File tools/pragmaonce/pragmaonce.py (right):

http://codereview.chromium.org/2842064/diff/5001/6001#newcode1
tools/pragmaonce/pragmaonce.py:1: import re
Copyright header plz thx.

http://codereview.chromium.org/2842064/diff/5001/6001#newcode6
tools/pragmaonce/pragmaonce.py:6: #   find chrome -name '*.h' -exec  python
tools/pragmaonce/pragmaonce.py {} \;
nit: extra double space

[Mark Mentovai, 2010-07-22 02:48:38]

http://codereview.chromium.org/2842064/diff/5001/6001
File tools/pragmaonce/pragmaonce.py (right):

http://codereview.chromium.org/2842064/diff/5001/6001#newcode18
tools/pragmaonce/pragmaonce.py:18: m1 = re.match(r'^#ifndef ([A-Z_]+)',
lines[i])
You should likely permit digits too.

[Nico, 2010-07-26 18:54:58]

Doing stuff across multiple lines in sed is very cumbersome as far as I remember
(need to add stuff to registers and stuff – if you know a simpler way, please
tell me).

CL that addresses comments is at http://codereview.chromium.org/3056020

http://codereview.chromium.org/2842064/diff/5001/6001
File tools/pragmaonce/pragmaonce.py (right):

http://codereview.chromium.org/2842064/diff/5001/6001#newcode1
tools/pragmaonce/pragmaonce.py:1: import re
On 2010/07/22 00:44:54, Evan Martin wrote:
> Copyright header plz thx.

Presubmit caught this before committing.

http://codereview.chromium.org/2842064/diff/5001/6001#newcode6
tools/pragmaonce/pragmaonce.py:6: #   find chrome -name '*.h' -exec  python
tools/pragmaonce/pragmaonce.py {} \;
On 2010/07/22 00:44:54, Evan Martin wrote:
> nit: extra double space

Done.

http://codereview.chromium.org/2842064/diff/5001/6001#newcode18
tools/pragmaonce/pragmaonce.py:18: m1 = re.match(r'^#ifndef ([A-Z_]+)',
lines[i])
On 2010/07/22 02:48:38, Mark Mentovai wrote:
> You should likely permit digits too.

Since I don't have a "$" at the end, this happened to work. Added digits and a
$.