Skip to content

python 3 compatible syntax tweak #189

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
stonebig wants to merge 1 commit into
base: main
Choose a base branch
from
Open

python 3 compatible syntax tweak #189

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions _pydev_imps/_pydev_SocketServer.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -336,12 +336,12 @@ def handle_error(self, request, client_address):
The default is to print a traceback and continue.

"""
print '-'*40
print 'Exception happened during processing of request from',
print client_address
print ('-'*40)
print ('Exception happened during processing of request from',)
print (client_address)
Comment on lines +339 to +341
Copy link

@cclauss cclauss Mar 14, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
print ('-'*40)
print ('Exception happened during processing of request from',)
print (client_address)
print('-'*40)
print('Exception happened during processing of request from',)
print(client_address)

Copy link
Contributor

@juliangilbey juliangilbey Feb 6, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Surely you wand to replace line 340 with print('...', end='')?

import traceback
traceback.print_exc() # XXX But this goes to stderr!
print '-'*40
print ('-'*40)
Copy link

@cclauss cclauss Mar 14, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
print ('-'*40)
print('-'*40)



class TCPServer(BaseServer):
Expand Down