Review: Approve
  992    # for f in self.imgs:
  993        # fl , nm = os.path.split(str(f))
  994        # c = self.slide_controller.rowCount()
  995        # self.slide_controller.setRowCount(c+1)
  996        # twi = QtGui.QTableWidgetItem(str(f))

I know this is commented out, but looking at those variables, I can't see at a 
glance what they are supposed to be. Please can you make decently named 
variables.
-- 
https://code.launchpad.net/~trb143/openlp/servicing/+merge/6402
Your team openlp.org Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to