Remove my $self = shift? #14
Labels
No labels
bug
code clarity
code organization
database
documentation
improvement
new feature
security
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
nu/CharmBoard#14
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
I've noticed this literally EVERYWHERE in the
CharmBoard::Controllernamespace,my $self = shiftat the start of every single subroutine. Maybe this is just confusion on my part since I"m having a hard time getting to grips with my own code after having been away from it for a year, but this looks... Useless, maybe? So if it IS then it should absolutely be removed.Nevermind, I'm a dolt. This is necessary to keep!
$selfto more descriptive names depending on where it gets used #21