Code review comment for lp:~trb143/openlp/servicing

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

  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.

review: Approve

« Back to merge proposal