This is an archive of the discontinued LLVM Phabricator instance.

Change print statements to print function calls
ClosedPublic

Authored by zturner on Oct 19 2015, 3:29 PM.

Details

Reviewers
tfiala
Summary
This patch was generating by running `2to3` on the files in the
lldb/test directory.  This patch should be NFC, but it does
introduce the `from __future__ import print_function` line, which
will break future uses of the print statement.

This patch does not address every python 2 / python 3
incompatibility.  Some are more mechanical than others, so I
plan to do this in stages.  The first pass fixes only print statements.
Other incompatibilities will addressed in followup patches.

Diff Detail

Event Timeline

zturner updated this revision to Diff 37805.Oct 19 2015, 3:29 PM
zturner retitled this revision from to Change print statements to print function calls.
zturner updated this object.
zturner added a reviewer: tfiala.
zturner added a subscriber: lldb-commits.
zturner updated this revision to Diff 37808.Oct 19 2015, 3:48 PM

I missed a few files, this gets everything. (Still only print statements)

zturner updated this revision to Diff 37811.Oct 19 2015, 3:50 PM

Sorry, uploaded the wrong patch the second time. This should be the one that gets everything.

tfiala accepted this revision.Oct 19 2015, 3:56 PM
tfiala edited edge metadata.

LGTM, Zachary!

This revision is now accepted and ready to land.Oct 19 2015, 3:56 PM
zturner closed this revision.Oct 19 2015, 4:51 PM

Submitted in rL250763