Moved relevant TODOs to full-blown issues.

Author: mikolak-net

src/main/scala/net/mikolak/pomisos/audio/SamplePlayer.scala

@@ -2,7 +2,6 @@ package net.mikolak.pomisos.audio
 
 import javax.sound.sampled._
 
-//TODO: refactor to actor
 class SamplePlayer(sampleAtStr: String) {
 
   private lazy val sampleAt = getClass.getResource(sampleAtStr)

src/main/scala/net/mikolak/pomisos/main/PomodoroTableController.scala

@@ -84,7 +84,7 @@ class PomodoroTableController(
       items.onChange(updateComplete())
       cell.item.onChange(updateComplete())
       cell
-    }) //TODO: this should parse w/o .value and be editable
+    })
 
   textColumn.onEditCommit = (e: CellEditEvent[Pomodoro, String]) => {
     dao.saveWith(_.copy(name = e.newValue))(e.rowValue.id)

src/main/scala/net/mikolak/pomisos/main/Tray.scala

@@ -23,7 +23,7 @@ case class Tray(showApp: (() => Unit) @@ AppOpenFunction, exitApp: (() => Unit)
   trayIcon.addActionListener(new ActionListener {
     override def actionPerformed(e: ActionEvent) = Platform.runLater(showApp())
   })
-  //FIXME: systray issue https://bugs.kde.org/show_bug.cgi?id=362941
+  //Blocked: systray issue https://bugs.kde.org/show_bug.cgi?id=362941
   //  trayIcon.addMouseListener(new MouseAdapter {
   //    override def mouseClicked(e: MouseEvent) = {
   //      log.debug("Click")

src/main/scala/net/mikolak/pomisos/prefs/task/TrelloNetworkService.scala

@@ -201,7 +201,6 @@ object TrelloNetworkService {
       internal.update(internal.indexOf(itemToUpload), uploaded)
     }
 
-    //TODO: manage ordering
     internal --= itemsToDeleteLocally
     internal ++= itemsToDownload
 

src/main/scala/net/mikolak/pomisos/quality/ConsecutiveQualityAdjuster.scala

@@ -25,7 +25,6 @@ class ConsecutiveQualityAdjuster(db: ScalaGraphAccess) extends QualityAdjuster {
     if (lastQualities.isEmpty) {
       None
     } else {
-      //TODO clean this up - breaks for LastQualitiesCount identical consecutive values (esp. 10)
       if (lastQualities.map(_.quality).distinct.length == 1) {
         Some(lastQualities.head.quality)
       } else {

src/main/scala/net/mikolak/pomisos/quality/TimeOfDayQualityAdjuster.scala

@@ -21,7 +21,7 @@ class TimeOfDayQualityAdjuster(db: ScalaGraphAccess, clock: Clock) extends Quali
     if (lastQualities.isEmpty) {
       None
     } else {
-      Try { //TODO: fragile for some values
+      Try {
         val OLS = ols(
           lastQualities.map(a => Array(toHour(a.timestamp))).toArray,
           lastQualities.map(_.quality.toDouble).toArray

src/test/scala/net/mikolak/pomisos/quality/QualitySpec.scala

@@ -44,7 +44,7 @@ abstract class QualitySpec
       forAll(genQualityList(Quality.Max, i => choose(Quality.Min, i), _.max(Quality.Min)), minSuccessful(Tries)) { qualityList =>
         whenever(qualityList.size > 10) {
           val prediction = tested.predictWithData(qualityList)
-          if (prediction.nonEmpty) { //TODO: defragile
+          if (prediction.nonEmpty) {
             prediction.value.floor must be <= qualityList.head.quality.toDouble
           }
         }
@@ -58,7 +58,7 @@ abstract class QualitySpec
       forAll(genQualityList(Quality.Min, i => choose(i, Quality.Max), _.min(Quality.Max)), minSuccessful(Tries)) { qualityList =>
         whenever(qualityList.size > 10) {
           val prediction = tested.predictWithData(qualityList)
-          if (prediction.nonEmpty) { //TODO: defragile
+          if (prediction.nonEmpty) {
             prediction.value.ceil must be >= qualityList.head.quality.toDouble
           }
         }