voussoir/etiquette
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Use .clear() instead of reassigning to empty lists.

Browse source 
I cannot remember if there is a reason I chose to use empty lists.
If there is, I'll rediscover it and write a comment. Until then, it
makes sense to use clear as an explicit clearing of the list.
This commit is contained in:
voussoir 2020-04-15 16:31:09 -07:00
parent 43a665fd91
commit 35895856e6
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
etiquette/photodb.py
View file

@ -765,8 +765,8 @@ class PDBSQLMixin:
else: else:
self.log.debug('Rolling back.') self.log.debug('Rolling back.')
self.sql_execute('ROLLBACK') self.sql_execute('ROLLBACK')
self.savepoints = [] self.savepoints.clear()
self.on_commit_queue = [] self.on_commit_queue.clear()
def savepoint(self, message=None): def savepoint(self, message=None):
savepoint_id = helpers.random_hex(length=16) savepoint_id = helpers.random_hex(length=16)