if __name__ == "__main__": closed = closedRecords("closed.001", "LIS00139") print 'No records', closed.noRecords() recs = closed.getRecords <-- Needs to be closedRecord() ie with parens Otherwise it looks OK to me... Alan g