From 52731e97c04aef214ddea2dd504d2dc7bdf8eae8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Frederik=20=E2=80=9CFreso=E2=80=9D=20S=2E=20Olesen?= Date: Fri, 19 May 2017 19:05:11 +0200 Subject: [PATCH] Remove old, commented out code Some of this seems to be debug code which has been left in, some of it seems to just be old code that was commented out and never put back in and probably just forgotten about. Either way, we use git for a reason, so there's no need for these code snippets to stick around. The code history can be inspected and old code retrieved that way. --- whipper/command/cd.py | 1 - whipper/extern/task/task.py | 1 - whipper/image/cue.py | 1 - whipper/image/image.py | 1 - whipper/image/table.py | 5 ----- whipper/image/toc.py | 3 --- whipper/program/cdparanoia.py | 2 -- whipper/test/test_image_toc.py | 11 ----------- 8 files changed, 25 deletions(-) diff --git a/whipper/command/cd.py b/whipper/command/cd.py index a5e052f..9980bc2 100644 --- a/whipper/command/cd.py +++ b/whipper/command/cd.py @@ -102,7 +102,6 @@ class _CD(BaseCommand): self.runner = task.SyncRunner() # if the device is mounted (data session), unmount it - #self.device = self.parentCommand.options.device self.device = self.options.device sys.stdout.write('Checking device %s\n' % self.device) diff --git a/whipper/extern/task/task.py b/whipper/extern/task/task.py index 30c0ed2..10bbac5 100644 --- a/whipper/extern/task/task.py +++ b/whipper/extern/task/task.py @@ -543,7 +543,6 @@ class SyncRunner(TaskRunner, ITaskListener): sys.stdout.write('\n') sys.stdout.flush() if len(what) > self._longest: - #print; print 'setting longest', self._longest; print self._longest = len(what) def described(self, task, description): diff --git a/whipper/image/cue.py b/whipper/image/cue.py index e8a87ed..31fd0ec 100644 --- a/whipper/image/cue.py +++ b/whipper/image/cue.py @@ -119,7 +119,6 @@ class CueFile(object): state = 'TRACK' trackNumber = int(m.group('track')) - #trackMode = m.group('mode') logger.debug('found track %d', trackNumber) currentTrack = table.Track(trackNumber) diff --git a/whipper/image/image.py b/whipper/image/image.py index b8c1149..815d69f 100644 --- a/whipper/image/image.py +++ b/whipper/image/image.py @@ -203,7 +203,6 @@ class ImageVerifyTask(task.MultiSeparateTask): if taskk.length is None: raise ValueError("Track length was not found; look for " "earlier errors in debug log (set RIP_DEBUG=4)") - # print '%d has length %d' % (trackIndex, taskk.length) index = track.indexes[1] assert taskk.length % common.SAMPLES_PER_FRAME == 0 end = taskk.length / common.SAMPLES_PER_FRAME diff --git a/whipper/image/table.py b/whipper/image/table.py index f09386e..2a2bfbb 100644 --- a/whipper/image/table.py +++ b/whipper/image/table.py @@ -281,8 +281,6 @@ class Table(object): # CD's have a standard lead-in time of 2 seconds # which gets added for CDDB disc id's delta = 2 * common.FRAMES_PER_SECOND - #if self.getTrackStart(1) > 0: - # delta = 0 debug = [str(len(self.tracks))] for track in self.tracks: @@ -293,7 +291,6 @@ class Table(object): n += self._cddbSum(seconds) # the 'real' leadout, not offset by 150 frames - # print 'THOMAS: disc leadout', self.leadout last = self.tracks[-1] leadout = self.getTrackEnd(last.number) + 1 logger.debug('leadout LBA: %d', leadout) @@ -369,7 +366,6 @@ class Table(object): try: offset = values[2 + i] except IndexError: - #print 'track', i - 1, '0 offset' offset = 0 sha.update("%08X" % offset) @@ -727,7 +723,6 @@ class Table(object): # the first cut is the deepest counter = index.counter - #for t in self.tracks: print t, t.indexes logger.debug('absolutizing') while True: track = self.tracks[t - 1] diff --git a/whipper/image/toc.py b/whipper/image/toc.py index 25d00d4..9411a21 100644 --- a/whipper/image/toc.py +++ b/whipper/image/toc.py @@ -311,7 +311,6 @@ class TocFile(object): common.msfToFrames(length)) self._sources.append(counter, absoluteOffset + currentLength, currentFile) - #absoluteOffset += common.msfToFrames(start) currentLength += common.msfToFrames(length) # look for DATAFILE lines @@ -319,7 +318,6 @@ class TocFile(object): if m: filePath = m.group('name') length = m.group('length') - # print 'THOMAS', length logger.debug('FILE %s, length %r', filePath, common.msfToFrames(length)) if not currentFile or filePath != currentFile.path: @@ -332,7 +330,6 @@ class TocFile(object): currentFile = File(filePath, 0, common.msfToFrames(length)) self._sources.append(counter, absoluteOffset + currentLength, currentFile) - #absoluteOffset += common.msfToFrames(start) currentLength += common.msfToFrames(length) diff --git a/whipper/program/cdparanoia.py b/whipper/program/cdparanoia.py index 81e7281..aa5e2f9 100644 --- a/whipper/program/cdparanoia.py +++ b/whipper/program/cdparanoia.py @@ -144,8 +144,6 @@ class ProgressParser: if frameOffset > self.read: delta = frameOffset - self.read if self._nframes and delta != self._nframes: - # print 'THOMAS: Read %d frames more, not %d' % ( - # delta, self._nframes) # my drive either reads 7 or 13 frames pass diff --git a/whipper/test/test_image_toc.py b/whipper/test/test_image_toc.py index 13bcedf..177c5ed 100644 --- a/whipper/test/test_image_toc.py +++ b/whipper/test/test_image_toc.py @@ -62,7 +62,6 @@ class CureTestCase(common.TestCase): self._assertAbsolute(2, 1, 28324) self._assertPath(1, 1, "data.wav") - # self.toc.table.absolutize() self.toc.table.clearFiles() self._assertAbsolute(1, 1, 0) @@ -86,7 +85,6 @@ class CureTestCase(common.TestCase): self._assertRelative(2, 1, None) def testConvertCue(self): - # self.toc.table.absolutize() cue = self.toc.table.cue() ref = self.readCue('cure.cue') common.diffStrings(ref, cue) @@ -155,14 +153,12 @@ class BlocTestCase(common.TestCase): # This disc has a pre-gap, so is a good test for .CUE writing def testConvertCue(self): - #self.toc.table.absolutize() self.failUnless(self.toc.table.hasTOC()) cue = self.toc.table.cue() ref = self.readCue('bloc.cue') common.diffStrings(ref, cue) def testCDDBId(self): - # self.toc.table.absolutize() # cd-discid output: # ad0be00d 13 15370 35019 51532 69190 84292 96826 112527 132448 # 148595 168072 185539 203331 222103 3244 @@ -171,7 +167,6 @@ class BlocTestCase(common.TestCase): def testAccurateRip(self): # we verify it because it has failed in readdisc in the past - # self.toc.table.absolutize() self.assertEquals(self.toc.table.getAccurateRipURL(), 'http://www.accuraterip.com/accuraterip/' 'e/d/2/dBAR-013-001af2de-0105994e-ad0be00d.bin') @@ -216,11 +211,9 @@ class LadyhawkeTestCase(common.TestCase): self.toc = toc.TocFile(self.path) self.toc.parse() self.assertEquals(len(self.toc.table.tracks), 13) - #import code; code.interact(local=locals()) self.failIf(self.toc.table.tracks[-1].audio) def testCDDBId(self): - #self.toc.table.absolutize() self.assertEquals(self.toc.table.getCDDBDiscId(), 'c60af50d') # output from cd-discid: # c60af50d 13 150 15687 31841 51016 66616 81352 99559 116070 133243 @@ -269,7 +262,6 @@ class CapitalMergeTestCase(common.TestCase): self.table.merge(self.toc2.table) def testCDDBId(self): - #self.table.absolutize() self.assertEquals(self.table.getCDDBDiscId(), 'b910140c') # output from cd-discid: # b910140c 12 24320 44855 64090 77885 88095 104020 118245 129255 141765 @@ -336,7 +328,6 @@ class TOTBLTestCase(common.TestCase): self.assertEquals(len(self.toc.table.tracks), 11) def testCDDBId(self): - #self.toc.table.absolutize() self.assertEquals(self.toc.table.getCDDBDiscId(), '810b7b0b') @@ -440,5 +431,3 @@ class SurferRosaTestCase(common.TestCase): self.assertEquals(t.getIndex(1).absolute, 111257) self.assertEquals(t.getIndex(2).relative, 111225 + 3370) self.assertEquals(t.getIndex(2).absolute, 111257 + 3370) - -# print self.toc.table.cue()