Skip to content

[PseudoObjC] Improve how [super ...] calls are displayed #6903

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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jun 18, 2025

Conversation

bdash
Copy link
Contributor

@bdash bdash commented May 30, 2025

  1. Skip displaying the declaration and initialization of the objc_super struct since that's an artifact of how objc_msgSendSuper is called.
  2. Display the message receiver as super rather than &super

@bdash bdash force-pushed the pseudo-objc-super branch 2 times, most recently from d53e0e0 to 86af956 Compare June 10, 2025 06:34
@plafosse plafosse added this to the Helion milestone Jun 10, 2025
bdash added 5 commits June 18, 2025 07:32
objc_opt_self -> [... self], objc_opt_isKindOfClass -> [... isKindOfClass:...], objc_opt_respondsToSelector -> [... respondsToSelector:...]
It was previously explicitly using the default. Retreiving it from the language will make it possible for PseudoObjCFunction to provide its own type printer.
1. Skip displaying the declaration and initialization of the objc_super
   struct since that's an artifact of how objc_msgSendSuper is called.

2. Display the message receiver as `super` rather than `&super`
@0cyn 0cyn force-pushed the pseudo-objc-super branch from 86af956 to 90d5791 Compare June 18, 2025 14:45
@0cyn 0cyn merged commit 90d5791 into Vector35:dev Jun 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants