Skip to content

Stop using use vars#15

Closed
guillemj wants to merge 2 commits intogisle:masterfrom
guillemj:pu/no-use-vars
Closed

Stop using use vars#15
guillemj wants to merge 2 commits intogisle:masterfrom
guillemj:pu/no-use-vars

Conversation

@guillemj
Copy link

This series switches the code away from use vars to modern alternatives, including importing Exporter's import via use.

guillemj added 2 commits July 13, 2017 04:34
This is one of the modern forms to get at the import. Use it instead of
aliasing the symbol into the current scope.
The former is deprecated and has a non-insignificant cost of loading a
module.
@toddr
Copy link

toddr commented Oct 5, 2020

Hi, thanks for the submission. This has already been actioned here Dual-Life@48d12ec and will be in the next release. Please close this case if you can as I don't have access

@guillemj
Copy link
Author

guillemj commented Oct 6, 2020

@toddr ah thanks! Will rebase the branch and submit anything remaining to the Dual-Life repo.

BTW are you planning on also taking over the https://github.com/gisle/digest module into the Dual-Life group? It looks like it's suffering the same fate. I've got a PR there too gisle/digest#4. :)

@guillemj guillemj closed this Oct 6, 2020
@toddr
Copy link

toddr commented Oct 6, 2020

@guillemj yes. I've started the process on that.

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