Skip to content

FIx: Remove superfluous arg to __bp_set_ret_value - #139

Draft
glensc wants to merge 1 commit into
rcaloras:masterfrom
glensc:patch-1
Draft

glensc wants to merge 1 commit into
rcaloras:masterfrom
glensc:patch-1

Conversation

@glensc

@glensc glensc commented Feb 15, 2023 •

Copy link
Copy Markdown

__bp_set_ret_value() takes just one argument:

__bp_set_ret_value takes just one argument
@glensc

glensc commented Feb 15, 2023

Copy link
Copy Markdown
Author

Seems it's trickery to set $_:

perhaps it then needs better code documentation, i.e for __bp_set_ret_value() function.

@glensc
glensc marked this pull request as draft February 15, 2023 23:22
@rcaloras

Copy link
Copy Markdown
Owner

@glensc, thanks for submitting the PR.

Seems it's trickery to set $:
Yes, it effectively propagates the $
variable so it can continue to be used. Feel free to close the PR or edit the the comments if you like.

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.

2 participants