Fix the prefix removal in patch_orderfile.py.

Fix the prefix removal in patch_orderfile.py.

Committed: https://crrev.com/aeccd0f8b22c8af534cb68d3c4734575b0ca89fb
Cr-Commit-Position: refs/heads/master@{#312137}

[Benoit L, 2015-01-19 18:11:10]

lizeb@chromium.org changed reviewers:
+ azarchs@chromium.org, pasko@chromium.org

[azarchs, 2015-01-19 18:13:22]

https://codereview.chromium.org/859873002/diff/1/tools/cygprofile/patch_order...
File tools/cygprofile/patch_orderfile.py (right):

https://codereview.chromium.org/859873002/diff/1/tools/cygprofile/patch_order...
tools/cygprofile/patch_orderfile.py:103: sys.stderr.write ("WARNING: could not
find symbol " + function + "\n")
From my local run I'd expect roughly 140k warnings to get spit out here.  Please
limit the number of warnings that go to the console.

[azarchs, 2015-01-19 18:27:22]

lgtm

[pasko-google - do not use, 2015-01-19 18:36:49]

pasko@google.com changed reviewers:
+ pasko@google.com

[pasko-google - do not use, 2015-01-19 18:36:50]

lgtm with one optional enhancement

https://codereview.chromium.org/859873002/diff/20001/tools/cygprofile/patch_o...
File tools/cygprofile/patch_orderfile.py (right):

https://codereview.chromium.org/859873002/diff/20001/tools/cygprofile/patch_o...
tools/cygprofile/patch_orderfile.py:110: sys.stderr.write ("symbols found: " +
str(symbols_found) + "\n")
can we also print the amount of missing symbols here (if any)?

[pasko, 2015-01-19 18:39:42]

now lgtm from the right account (but the suggestion holds:)

[Benoit L, 2015-01-19 18:41:41]

On 2015/01/19 18:39:42, pasko wrote:
> now lgtm from the right account (but the suggestion holds:)

Thank you.
All done.

[Benoit L, 2015-01-19 18:42:11]

The CQ bit was checked by lizeb@chromium.org

[commit-bot: I haz the power, 2015-01-19 18:42:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/859873002/40001

[commit-bot: I haz the power, 2015-01-19 19:32:25]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-01-19 19:33:17]

Patchset 3 (id:??) landed as
https://crrev.com/aeccd0f8b22c8af534cb68d3c4734575b0ca89fb
Cr-Commit-Position: refs/heads/master@{#312137}