Part of meta-issue #2002650: [meta, no patch] improve maintainability by removing unused local variables

File /core/lib/Drupal/Core/Database/StatementPrefetch.php

Line 458: Unused local variable $column
Line 496: Unused local variable $row

CommentFileSizeAuthor
#1 core-remove-unused-local-variables-statementprefetch.php-2060811-1.patch1.03 KBlegolasbo

Comments

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Status: Active » Needs review
StatusFileSize
new core-remove-unused-local-variables-statementprefetch.php-2060811-1.patch1.03 KB

Attached patch removes the unused variables

dawehner’s picture

dawehner
Primary language German
commented
Priority: Normal » Minor
Status: Needs review » Reviewed & tested by the community

Nice!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Component: other » database system
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks! And nice catch on the now invalid comment. :)

legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Assigned: legolasbo » Unassigned

Automatically closed -- issue fixed for 2 weeks with no activity.