From 830acff94c95a4966489f8bb484c648fc34fceec Mon Sep 17 00:00:00 2001 From: Pedro Nunes Date: Tue, 8 Feb 2022 17:24:00 -0300 Subject: [PATCH] Changed `log.info` to `log.debug` in six files (#690) * `log.info` changed to `log.debug` in six files * Fix identation * Remove from CHANGELOG.md since no functionality has changed Co-authored-by: Pedro Nunes Co-authored-by: Pieter Marsman --- CHANGELOG.md | 4 +--- pdfminer/cmapdb.py | 2 +- pdfminer/converter.py | 2 +- pdfminer/pdfdocument.py | 16 ++++++++-------- pdfminer/pdfinterp.py | 8 ++++---- pdfminer/pdfpage.py | 4 ++-- pdfminer/psparser.py | 2 +- 7 files changed, 18 insertions(+), 20 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 62b6684..bab05dc 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -20,13 +20,11 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/). ### Changed - Replace warnings.warn with logging.Logger.warning in line with [recommended use](https://docs.python.org/3/howto/logging.html#when-to-use-logging) ([#673](https://github.com/pdfminer/pdfminer.six/pull/673)) +- Switched from nose to pytest, from tox to nox and from Travis CI to GitHub Actions ([#704](https://github.com/pdfminer/pdfminer.six/pull/704)) ### Removed - Unnecessary return statements without argument at the end of functions ([#707](https://github.com/pdfminer/pdfminer.six/pull/707)) -### Changed -- Switched from nose to pytest, from tox to nox and from Travis CI to GitHub Actions ([#704](https://github.com/pdfminer/pdfminer.six/pull/704)) - ## [20211012] ### Added diff --git a/pdfminer/cmapdb.py b/pdfminer/cmapdb.py index f25402c..451aab0 100644 --- a/pdfminer/cmapdb.py +++ b/pdfminer/cmapdb.py @@ -227,7 +227,7 @@ class CMapDB: def _load_data(cls, name: str) -> Any: name = name.replace("\0", "") filename = '%s.pickle.gz' % name - log.info('loading: %r', name) + log.debug('loading: %r', name) cmap_paths = (os.environ.get('CMAP_PATH', '/usr/share/pdfminer/'), os.path.join(os.path.dirname(__file__), 'cmap'),) for directory in cmap_paths: diff --git a/pdfminer/converter.py b/pdfminer/converter.py index b09024e..14da61b 100644 --- a/pdfminer/converter.py +++ b/pdfminer/converter.py @@ -168,7 +168,7 @@ class PDFLayoutAnalyzer(PDFTextDevice): return item.adv def handle_undefined_char(self, font: PDFFont, cid: int) -> str: - log.info('undefined: %r, %r', font, cid) + log.debug('undefined: %r, %r', font, cid) return '(cid:%d)' % cid def receive_layout(self, ltpage: LTPage) -> None: diff --git a/pdfminer/pdfdocument.py b/pdfminer/pdfdocument.py index 4da9927..6be37f4 100644 --- a/pdfminer/pdfdocument.py +++ b/pdfminer/pdfdocument.py @@ -153,7 +153,7 @@ class PDFXRef(PDFBaseXRef): if use_b != b'n': continue self.offsets[objid] = (None, int(pos_b), int(genno_b)) - log.info('xref objects: %r', self.offsets) + log.debug('xref objects: %r', self.offsets) self.load_trailer(parser) def load_trailer(self, parser: PDFParser) -> None: @@ -199,7 +199,7 @@ class PDFXRefFallback(PDFXRef): if line_bytes.startswith(b'trailer'): parser.seek(pos) self.load_trailer(parser) - log.info('trailer: %r', self.trailer) + log.debug('trailer: %r', self.trailer) break line = line_bytes.decode('latin-1') # default pdf encoding m = self.PDFOBJ_CUE.match(line) @@ -268,9 +268,9 @@ class PDFXRefStream(PDFBaseXRef): self.data = stream.get_data() self.entlen = self.fl1+self.fl2+self.fl3 self.trailer = stream.attrs - log.info('xref stream: objid=%s, fields=%d,%d,%d', - ', '.join(map(repr, self.ranges)), - self.fl1, self.fl2, self.fl3) + log.debug('xref stream: objid=%s, fields=%d,%d,%d', + ', '.join(map(repr, self.ranges)), + self.fl1, self.fl2, self.fl3) return def get_trailer(self) -> Dict[str, Any]: @@ -977,7 +977,7 @@ class PDFDocument: prev = line else: raise PDFNoValidXRef('Unexpected EOF') - log.info('xref found: pos=%r', prev) + log.debug('xref found: pos=%r', prev) assert prev is not None return int(prev) @@ -995,7 +995,7 @@ class PDFDocument: (pos, token) = parser.nexttoken() except PSEOF: raise PDFNoValidXRef('Unexpected EOF') - log.info('read_xref_from: start=%d, token=%r', start, token) + log.debug('read_xref_from: start=%d, token=%r', start, token) if isinstance(token, int): # XRefStream: PDF-1.5 parser.seek(pos) @@ -1009,7 +1009,7 @@ class PDFDocument: xref.load(parser) xrefs.append(xref) trailer = xref.get_trailer() - log.info('trailer: %r', trailer) + log.debug('trailer: %r', trailer) if 'XRefStm' in trailer: pos = int_value(trailer['XRefStm']) self.read_xref_from(parser, pos, xrefs) diff --git a/pdfminer/pdfinterp.py b/pdfminer/pdfinterp.py index 2a6d55b..7f60ec3 100644 --- a/pdfminer/pdfinterp.py +++ b/pdfminer/pdfinterp.py @@ -951,7 +951,7 @@ class PDFPageInterpreter: if settings.STRICT: raise PDFInterpreterError('Undefined xobject id: %r' % xobjid) return - log.info('Processing xobj: %r', xobj) + log.debug('Processing xobj: %r', xobj) subtype = xobj.get('Subtype') if subtype is LITERAL_FORM and 'BBox' in xobj: interpreter = self.dup() @@ -980,7 +980,7 @@ class PDFPageInterpreter: return def process_page(self, page: PDFPage) -> None: - log.info('Processing page: %r', page) + log.debug('Processing page: %r', page) (x0, y0, x1, y1) = page.mediabox if page.rotate == 90: ctm = (0, -1, 1, 0, -y0, x1) @@ -1005,8 +1005,8 @@ class PDFPageInterpreter: This method may be called recursively. """ - log.info('render_contents: resources=%r, streams=%r, ctm=%r', - resources, streams, ctm) + log.debug('render_contents: resources=%r, streams=%r, ctm=%r', + resources, streams, ctm) self.init_resources(resources) self.init_state(ctm) self.execute(list_value(streams)) diff --git a/pdfminer/pdfpage.py b/pdfminer/pdfpage.py index 39195a3..5eb8709 100644 --- a/pdfminer/pdfpage.py +++ b/pdfminer/pdfpage.py @@ -109,11 +109,11 @@ class PDFPage: tree_type = tree.get('type') if tree_type is LITERAL_PAGES and 'Kids' in tree: - log.info('Pages: Kids=%r', tree['Kids']) + log.debug('Pages: Kids=%r', tree['Kids']) for c in list_value(tree['Kids']): yield from search(c, tree) elif tree_type is LITERAL_PAGE: - log.info('Page: %r', tree) + log.debug('Page: %r', tree) yield (objid, tree) try: diff --git a/pdfminer/psparser.py b/pdfminer/psparser.py index 43c457d..e691f40 100644 --- a/pdfminer/psparser.py +++ b/pdfminer/psparser.py @@ -200,7 +200,7 @@ class PSBaseParser: if not pos: pos = self.bufpos+self.charpos self.fp.seek(pos) - log.info('poll(%d): %r', pos, self.fp.read(n)) + log.debug('poll(%d): %r', pos, self.fp.read(n)) self.fp.seek(pos0) return