From 190e0139e834e4271268652e058c280787ae69eb Mon Sep 17 00:00:00 2001 From: Mike Bayer Date: Thu, 30 May 2019 11:31:03 -0400 Subject: Enable F841 This is a very useful assertion which prevents unused variables from being set up allows code to be more readable and sometimes even more efficient. test suites seem to be where the most problems are and there do not seem to be documentation examples that are using this, or at least the linter is not taking effect within rst blocks. Change-Id: I2b3341d8dd14da34879d8425838e66a4b9f8e27d --- lib/sqlalchemy/engine/result.py | 2 -- 1 file changed, 2 deletions(-) (limited to 'lib/sqlalchemy/engine') diff --git a/lib/sqlalchemy/engine/result.py b/lib/sqlalchemy/engine/result.py index d0b16a745..740565f5e 100644 --- a/lib/sqlalchemy/engine/result.py +++ b/lib/sqlalchemy/engine/result.py @@ -503,7 +503,6 @@ class ResultMetaData(object): def _merge_textual_cols_by_position( self, context, cursor_description, result_columns ): - dialect = context.dialect num_ctx_cols = len(result_columns) if result_columns else None if num_ctx_cols > len(cursor_description): @@ -558,7 +557,6 @@ class ResultMetaData(object): yield idx, colname, mapped_type, coltype, obj, untranslated def _merge_cols_by_none(self, context, cursor_description): - dialect = context.dialect for ( idx, colname, -- cgit v1.2.1