fix: release table_entry with the move operator and add default constructor for table_entry#39
Conversation
Signed-off-by: Roberto Scolaro <roberto.scolaro21@gmail.com>
|
@therealbobo: The label(s) DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Signed-off-by: Roberto Scolaro <roberto.scolaro21@gmail.com>
f9cb7fb to
8a5f87d
Compare
Signed-off-by: Roberto Scolaro <roberto.scolaro21@gmail.com>
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mstemm, therealbobo The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
LGTM label has been added. DetailsGit tree hash: 5e974c99c30e68c93fee1bf92ae6b10390768deb |
What type of PR is this?
/kind bug
/kind design
Any specific area of the project related to this PR?
What this PR does / why we need it:
The
table_entryclass was missing the default constructor.The
table_entrymove operator didn't clean up the left hand sidetable_entrywhen movingWhich issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: