Skip to content
New issue

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

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

Already on GitHub? # to your account

Remove If True: for not testing scripts #129351

Open
wooffie opened this issue Jan 27, 2025 · 2 comments
Open

Remove If True: for not testing scripts #129351

wooffie opened this issue Jan 27, 2025 · 2 comments
Labels
pending The issue will be closed if no feedback is provided

Comments

@wooffie
Copy link
Contributor

wooffie commented Jan 27, 2025

Can we remove If True: from gdb bindings?

I think it was created for future features, but it was 15 years ago.

Found 2 locations (not testcase for interpreter):

if True:

if True:

Additional information

Found by Linux Verification Center (linuxtesting.org) with SVACE.

Reporter: Burkov Egor (eburkov@rvision.ru).

Organization: R-Vision (support@rvision.ru).

@StanFromIreland
Copy link
Contributor

The first one has a TODO

# TODO: repr() puts everything on one line; pformat can be nicer, but
# can lead to v.long results; this function isolates the choice

The second has nothing listed. I guess we can comment out the section for it.

@encukou
Copy link
Member

encukou commented Jan 27, 2025

These are undocumented and not very discoverable, but useful if you know about them.
I don't know if there's a good way to configure them (or ask GDB for longer output). Maybe we should just document them in the devguide.

@encukou encukou added the pending The issue will be closed if no feedback is provided label Jan 29, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
pending The issue will be closed if no feedback is provided
Projects
None yet
Development

No branches or pull requests

3 participants