← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~trb143/openlp/servicing into lp:openlp

 

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.



References