Fix incorrect frames to MSF/HMSF conversion

Tagged a new release (v0.4.1) too.

Signed-off-by: JoeLametta <[email protected]>

README.md

@@ -14,7 +14,7 @@ carefully mimics EAC's generated ones (except for the checksum).
 The logger should be feature complete so future development will consist
 mainly of bugfixes.
 
-If you're looking for the analogous [whipper](https:/whipper-team/whipper) plugin,
+If you're looking for the analogous (legacy) [whipper](https:/whipper-team/whipper) plugin,
 it can be found [here](https:/whipper-team/whipper-plugin-eaclogger).
 
 ## License

eaclogger/__init__.py

@@ -1 +1 @@
-__version__ = '0.4.0'
+__version__ = '0.4.1'

eaclogger/logger/eac.py

@@ -19,7 +19,7 @@ def _framesToMSF(self, frames):
  f = frames % common.FRAMES_PER_SECOND
  frames -= f
  s = (frames / common.FRAMES_PER_SECOND) % 60
- frames -= s * 60
+ frames -= s * common.FRAMES_PER_SECOND
  m = frames / common.FRAMES_PER_SECOND / 60
  return "%2d:%02d.%02d" % (m, s, f)
 
@@ -32,9 +32,9 @@ def _framesToHMSF(self, frames):
  f = frames % common.FRAMES_PER_SECOND
  frames -= f
  s = (frames / common.FRAMES_PER_SECOND) % 60
- frames -= s * 60
- m = frames / common.FRAMES_PER_SECOND / 60
- frames -= m * 60
+ frames -= s * common.FRAMES_PER_SECOND
+ m = (frames / common.FRAMES_PER_SECOND / 60) % 60
+ frames -= m * common.FRAMES_PER_SECOND * 60
  h = frames / common.FRAMES_PER_SECOND / 60 / 60
  return "%2d:%02d:%02d.%02d" % (h, m, s, f)