Permalink
Browse files

clarify TODO

  • Loading branch information...
1 parent e48b83a commit 006daba9533eed539b3476c1658b43669ff1a8cd @chocolateboy chocolateboy committed Nov 26, 2012
Showing with 5 additions and 5 deletions.
  1. +5 −5 src/main/java/net/pms/io/ProcessWrapperImpl.java
@@ -148,11 +148,11 @@ public void run() {
}
}
- // TODO fix the callers of this code to use simpler mechanisms to
- // execute short-running commands (e.g. vlc -version) so
- // that this class is only used to run long-running tasks i.e.
- // transcodes. in that case, we won't need separate stdout and stderr
- // and can merge them by calling:
+ // XXX A cleaner way to execute short-running commands (e.g. vlc -version)
+ // is being developed. When that's done, this class can be used solely
+ // for the long-running tasks i.e. transcodes. At that point, we won't need
+ // separate stdout and stderr and can merge them by uncommenting the
+ // following line:
// pb.redirectErrorStream(true);
process = pb.start();
PMS.get().currentProcesses.add(process);

0 comments on commit 006daba

Please sign in to comment.