Skip to content
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.

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

Already on GitHub? Sign in to your account

Pdb.do_longlist() double prints? #496

Open
goodboy opened this issue Jun 27, 2022 · 4 comments
Open

Pdb.do_longlist() double prints? #496

goodboy opened this issue Jun 27, 2022 · 4 comments
Labels

Comments

@goodboy
Copy link

goodboy commented Jun 27, 2022

On recent master if you call this method you'll see 2x the output but if I remove the self.lastcmd = 'longlist' line, it works again 馃槀

Are there any tests for this method?
A brief look says no 馃樋

@goodboy
Copy link
Author

goodboy commented Jun 27, 2022

Lol, yeah this explains why new tests I was writing with pexpect weren't workin 馃く

@goodboy
Copy link
Author

goodboy commented Jun 29, 2022

I will try to produce tests for both this and #495 @blueyed

@goodboy
Copy link
Author

goodboy commented Jul 11, 2022

Yah so the version I can get this working by hacking around it and doing:

    if sys.version_info >= (3, 10):
        pdb_obj._printlonglist(False)

as long as I pin to version 0.10.3.dev182+gf4f26d2 this works, but anything later (though haven't tested thoroughly yet) seems to not work the same.

goodboy added a commit to goodboy/tractor that referenced this issue Jul 12, 2022
@goodboy
Copy link
Author

goodboy commented Jul 12, 2022

Lol, so this took me a while to figure out, but it's fancycompleter.

If i upgrade to the 0.9.2 (dev) version (instead of the one that seems to get installed by default this issue is gone i think?

goodboy added a commit to goodboy/tractor that referenced this issue Jul 14, 2022
goodboy added a commit to goodboy/tractor that referenced this issue Jul 15, 2022
goodboy added a commit to goodboy/tractor that referenced this issue Jul 27, 2022
goodboy added a commit to goodboy/tractor that referenced this issue Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants