Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(463)

Unified Diff: infra/bots/recipe_modules/skia/resources/binary_size_utils.py

Issue 2198173002: Re-organize Skia recipes (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Fix missing dependency Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: infra/bots/recipe_modules/skia/resources/binary_size_utils.py
diff --git a/infra/bots/recipe_modules/skia/resources/binary_size_utils.py b/infra/bots/recipe_modules/skia/resources/binary_size_utils.py
deleted file mode 100644
index c09a65dccdd10019be977a431572d76bd60e58fe..0000000000000000000000000000000000000000
--- a/infra/bots/recipe_modules/skia/resources/binary_size_utils.py
+++ /dev/null
@@ -1,67 +0,0 @@
-# Copyright 2014 The Chromium Authors. All rights reserved.
-# Use of this source code is governed by a BSD-style license that can be
-# found in the LICENSE file.
-
-"""Common utilities for tools that deal with binary size information.
-
-Copied from chromium/src/build/android/pylib/symbols/binary_size_tools.py.
-"""
-
-import logging
-import re
-
-
-def ParseNm(nm_lines):
- """Parse nm output, returning data for all relevant (to binary size)
- symbols and ignoring the rest.
-
- Args:
- nm_lines: an iterable over lines of nm output.
-
- Yields:
- (symbol name, symbol type, symbol size, source file path).
-
- Path may be None if nm couldn't figure out the source file.
- """
-
- # Match lines with size, symbol, optional location, optional discriminator
- sym_re = re.compile(r'^[0-9a-f]{8,} ' # address (8+ hex digits)
- '([0-9a-f]{8,}) ' # size (8+ hex digits)
- '(.) ' # symbol type, one character
- '([^\t]+)' # symbol name, separated from next by tab
- '(?:\t(.*):[\d\?]+)?.*$') # location
- # Match lines with addr but no size.
- addr_re = re.compile(r'^[0-9a-f]{8,} (.) ([^\t]+)(?:\t.*)?$')
- # Match lines that don't have an address at all -- typically external symbols.
- noaddr_re = re.compile(r'^ {8,} (.) (.*)$')
- # Match lines with no symbol name, only addr and type
- addr_only_re = re.compile(r'^[0-9a-f]{8,} (.)$')
-
- for line in nm_lines:
- line = line.rstrip()
- match = sym_re.match(line)
- if match:
- size, sym_type, sym = match.groups()[0:3]
- size = int(size, 16)
- if sym_type in ('B', 'b'):
- continue # skip all BSS for now.
- path = match.group(4)
- yield sym, sym_type, size, path
- continue
- match = addr_re.match(line)
- if match:
- # sym_type, sym = match.groups()[0:2]
- continue # No size == we don't care.
- match = noaddr_re.match(line)
- if match:
- sym_type, sym = match.groups()
- if sym_type in ('U', 'w'):
- continue # external or weak symbol
- match = addr_only_re.match(line)
- if match:
- continue # Nothing to do.
-
-
- # If we reach this part of the loop, there was something in the
- # line that we didn't expect or recognize.
- logging.warning('nm output parser failed to parse: %s', repr(line))
« no previous file with comments | « infra/bots/recipe_modules/skia/pdfium_flavor.py ('k') | infra/bots/recipe_modules/skia/resources/elf_symbolizer.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698