clean-up

pom.xml

@@ -70,7 +70,7 @@
         <dependency>
             <groupId>commons-io</groupId>
             <artifactId>commons-io</artifactId>
-            <version>2.7</version>
+            <version>2.11.0</version>
             <scope>compile</scope>
         </dependency>
 
@@ -83,7 +83,7 @@
         <dependency>
             <groupId>org.controlsfx</groupId>
             <artifactId>controlsfx</artifactId>
-            <version>11.1.0</version>
+            <version>11.1.1</version>
             <exclusions>
                 <exclusion>
                     <groupId>org.openjfx</groupId>
@@ -95,13 +95,13 @@
         <dependency>
             <groupId>com.google.code.gson</groupId>
             <artifactId>gson</artifactId>
-            <version>2.8.6</version>
+            <version>2.8.9</version>
         </dependency>
 
         <dependency>
             <groupId>com.google.guava</groupId>
             <artifactId>guava</artifactId>
-            <version>30.1.1-jre</version>
+            <version>31.0.1-jre</version>
         </dependency>
 
         <dependency>

src/main/java/uk/yermak/audiobookconverter/AudiobookConverter.java

@@ -16,8 +16,6 @@
 import org.controlsfx.control.Notifications;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-import uk.yermak.audiobookconverter.AppProperties;
-import uk.yermak.audiobookconverter.Utils;
 import uk.yermak.audiobookconverter.Version;
 import uk.yermak.audiobookconverter.fx.ConversionContext;
 import uk.yermak.audiobookconverter.fx.JfxEnv;

src/main/java/uk/yermak/audiobookconverter/ConversionGroup.java

@@ -139,15 +139,13 @@ public void launch(ListView<ProgressComponent> progressQueue, ProgressComponent
                 Chapter chapter = chapters.get(i);
                 String finalDesination = outputDestination;
                 if (chapters.size() > 1) {
-                    finalDesination = finalDesination.replace("." + format.toString(), ", Chapter " + (i + 1) + "." + format.toString());
+                    finalDesination = finalDesination.replace("." + format.toString(), ", Chapter " + (i + 1) + "." + format);
                 }
                 String finalName = new File(finalDesination).getName();
                 logger.debug("Adding conversion for chapter {}", finalName);
 
                 ConversionProgress conversionProgress = this.start(chapter, finalDesination);
-                Platform.runLater(() -> {
-                    progressQueue.getItems().add(0, new ProgressComponent(conversionProgress));
-                });
+                Platform.runLater(() -> progressQueue.getItems().add(0, new ProgressComponent(conversionProgress)));
 
             }
         } else {
@@ -156,15 +154,13 @@ public void launch(ListView<ProgressComponent> progressQueue, ProgressComponent
                 Part part = parts.get(i);
                 String finalDesination = outputDestination;
                 if (parts.size() > 1) {
-                    finalDesination = finalDesination.replace("." + format.toString(), ", Part " + (i + 1) + "." + format.toString());
+                    finalDesination = finalDesination.replace("." + format.toString(), ", Part " + (i + 1) + "." + format);
                 }
                 String finalName = new File(finalDesination).getName();
                 logger.debug("Adding conversion for part {}", finalName);
 
                 ConversionProgress conversionProgress = this.start(part, finalDesination);
-                Platform.runLater(() -> {
-                    progressQueue.getItems().add(0, new ProgressComponent(conversionProgress));
-                });
+                Platform.runLater(() -> progressQueue.getItems().add(0, new ProgressComponent(conversionProgress)));
             }
         }
 

src/main/java/uk/yermak/audiobookconverter/ConversionJob.java

@@ -55,7 +55,6 @@ public void run() {
 
         String tempFile = Utils.getTmp(jobId, outputDestination.hashCode(), conversionGroup.getWorkfileExtension());
 
-        File metaFile = null;
         try {
 //            conversion.getOutputParameters().updateAuto(conversion.getMedia());
 
@@ -94,8 +93,6 @@ public void run() {
             StringWriter sw = new StringWriter();
             e.printStackTrace(new PrintWriter(sw));
             error(e.getMessage() + "; " + sw.getBuffer().toString());
-        } finally {
-            FileUtils.deleteQuietly(metaFile);
         }
     }
 

src/main/java/uk/yermak/audiobookconverter/Environment.java

@@ -20,14 +20,14 @@ protected String getAppPath() {
         }
 
         @Override
-        protected File getConfigFilePath(File file) {
+        protected File getConfigFilePath() {
             return new File(getAppPath(), "Contents/app/path.properties");
         }
     },
 
     LINUX {
         @Override
-        protected File getConfigFilePath(File file) {
+        protected File getConfigFilePath() {
             return new File("../lib/app/path.properties");
 
         }
@@ -83,14 +83,13 @@ protected String getAppPath() {
         return "";
     }
 
-    protected File getConfigFilePath(File file) {
+    protected File getConfigFilePath() {
         return new File("app/path.properties");
     }
 
     synchronized Properties loadAppProperties() {
         if (properties.isEmpty()) {
-            File file = null;
-            file = getConfigFilePath(file);
+            File file = getConfigFilePath();
 
             if (file.exists()) {
                 try (FileInputStream in = new FileInputStream(file)) {

src/main/java/uk/yermak/audiobookconverter/FFMediaLoader.java

@@ -203,8 +203,7 @@ static void parseCue(MediaInfoBean mediaInfo, String cue) {
     private static long parseCueTime(String substring) {
         String cleanText = cleanText(substring);
         String[] split = cleanText.split(":");
-        long time = 1000 * (Integer.parseInt(split[0]) * 60 + Integer.parseInt(split[1])) + Integer.parseInt(split[2]) * 1000 / 75;
-        return time;
+        return 1000 * (Integer.parseInt(split[0]) * 60 + Integer.parseInt(split[1])) + Integer.parseInt(split[2]) * 1000 / 75;
     }
 
     private static String cleanText(String text) {

src/main/java/uk/yermak/audiobookconverter/FFMpegConcatenator.java

@@ -30,7 +30,6 @@ public class FFMpegConcatenator {
     private final ProgressCallback callback;
     private ProgressParser progressParser;
     private List<String> tmpFiles = new ArrayList<>();
-    private String fileListFileName;
 
     public FFMpegConcatenator(ConversionJob conversionJob, String outputFileName, MetadataBuilder metadataBuilder, List<MediaInfo> media, ProgressCallback callback) {
         this.conversionJob = conversionJob;
@@ -49,14 +48,12 @@ protected static File prepareFiles(long jobId, List<MediaInfo> media, String wor
 
     public void concat() throws IOException, InterruptedException {
         if (conversionJob.getStatus().isOver()) return;
-        fileListFileName = prepareFiles(conversionJob.jobId, media, conversionJob.getConversionGroup().getWorkfileExtension()).getAbsolutePath();
+        String fileListFileName = prepareFiles(conversionJob.jobId, media, conversionJob.getConversionGroup().getWorkfileExtension()).getAbsolutePath();
 
         while (ProgressStatus.PAUSED.equals(conversionJob.getStatus())) Thread.sleep(1000);
         callback.reset();
         try {
-            progressParser = new TcpProgressParser(progress -> {
-                callback.converted(progress.out_time_ns / 1000000, progress.total_size);
-            });
+            progressParser = new TcpProgressParser(progress -> callback.converted(progress.out_time_ns / 1000000, progress.total_size));
             progressParser.start();
         } catch (URISyntaxException e) {
         }
@@ -83,15 +80,15 @@ public void concat() throws IOException, InterruptedException {
             while (!conversionJob.getStatus().isOver() && !finished) {
                 finished = process.waitFor(500, TimeUnit.MILLISECONDS);
             }
-            logger.debug("Concat Out: {}", out.toString());
-            logger.error("Concat Error: {}", err.toString());
+            logger.debug("Concat Out: {}", out);
+            logger.error("Concat Error: {}", err);
 
             if (process.exitValue() != 0) {
                 throw new ConversionException("Concatenation exit code " + process.exitValue() + "!=0", new Error(err.toString()));
             }
 
             if (!new File(outputFileName).exists()) {
-                throw new ConversionException("Concatenation failed, no output file:" + out.toString(), new Error(err.toString()));
+                throw new ConversionException("Concatenation failed, no output file:" + out, new Error(err.toString()));
             }
         } catch (Exception e) {
             logger.error("Error during concatination of files:", e);

src/main/java/uk/yermak/audiobookconverter/FFMpegNativeConverter.java

@@ -69,8 +69,8 @@ public String call() throws Exception {
             while (!conversionJob.getStatus().isOver() && !finished) {
                 finished = process.waitFor(500, TimeUnit.MILLISECONDS);
             }
-            logger.debug("ffmpeg out: {}", out.toString());
-            logger.warn("ffmpeg err: {}", err.toString());
+            logger.debug("ffmpeg out: {}", out);
+            logger.warn("ffmpeg err: {}", err);
 
             DurationVerifier.ffMpegUpdateDuration(mediaInfo, outputFileName);
             return outputFileName;

src/main/java/uk/yermak/audiobookconverter/FFMpegOptimizer.java

@@ -18,15 +18,13 @@ public class FFMpegOptimizer {
     private String tempFile;
     private final String outputFileName;
 
-    private final ProgressCallback callback;
     private ProgressParser progressParser;
 
 
     public FFMpegOptimizer(ConversionJob conversionJob, String tempFile, String outputFileName, ProgressCallback callback) {
         this.conversionJob = conversionJob;
         this.tempFile = tempFile;
         this.outputFileName = outputFileName;
-        this.callback = callback;
     }
 
 
@@ -45,7 +43,7 @@ public void moveResultingFile() {
         }
     }
 
-    private  void optimize() throws IOException, InterruptedException {
+    private  void optimize() throws InterruptedException {
         if (conversionJob.getStatus().isOver()) return;
         while (ProgressStatus.PAUSED.equals(conversionJob.getStatus())) Thread.sleep(1000);
 //        callback.reset();
@@ -88,15 +86,15 @@ private  void optimize() throws IOException, InterruptedException {
             while (!conversionJob.getStatus().isOver() && !finished) {
                 finished = process.waitFor(500, TimeUnit.MILLISECONDS);
             }
-            logger.debug("Optimize Out: {}", out.toString());
-            logger.error("Optimize Error: {}", err.toString());
+            logger.debug("Optimize Out: {}", out);
+            logger.error("Optimize Error: {}", err);
 
             if (process.exitValue() != 0) {
                 throw new ConversionException("Optimisation exit code " + process.exitValue() + "!=0", new Error(err.toString()));
             }
 
             if (!new File(Utils.getTmp(conversionJob.jobId, outputFileName.hashCode()+1, conversionJob.getConversionGroup().getWorkfileExtension())).exists()) {
-                throw new ConversionException("Optimisation failed, no output file:" + out.toString(), new Error(err.toString()));
+                throw new ConversionException("Optimisation failed, no output file:" + out, new Error(err.toString()));
             }
         } catch (Exception e) {
             logger.error("Error during optimisation of resulting file:", e);

src/main/java/uk/yermak/audiobookconverter/FFmpegArtWorkExtractor.java

@@ -46,8 +46,8 @@ public ArtWork call() throws Exception {
             while (!conversionGroup.isOver() && !finished) {
                 finished = process.waitFor(500, TimeUnit.MILLISECONDS);
             }
-            FFMediaLoader.logger.debug("ArtWork Out: {}", out.toString());
-            FFMediaLoader.logger.error("ArtWork Error: {}", err.toString());
+            FFMediaLoader.logger.debug("ArtWork Out: {}", out);
+            FFMediaLoader.logger.error("ArtWork Error: {}", err);
 
             ArtWorkBean artWorkBean = new ArtWorkBean(poster);
             Platform.runLater(() -> {

src/main/java/uk/yermak/audiobookconverter/MediaInfo.java

@@ -22,9 +22,9 @@ public interface MediaInfo extends Organisable {
 
     void setChapter(Chapter chapter);
 
-    public Chapter getChapter();
+    Chapter getChapter();
 
-    public int getUID();
+    int getUID();
 }
 
 

src/main/java/uk/yermak/audiobookconverter/MediaInfoProxy.java

@@ -1,7 +1,6 @@
 package uk.yermak.audiobookconverter;
 
 import java.util.List;
-import java.util.Objects;
 import java.util.concurrent.Future;
 
 public class MediaInfoProxy implements MediaInfo {

src/main/java/uk/yermak/audiobookconverter/Mp4v2ArtBuilder.java

@@ -33,7 +33,7 @@ public void coverArt(String outputFileName) throws IOException, InterruptedExcep
         }
     }
 
-    public void updateSinglePoster(ArtWork poster, int index, String outputFileName) throws IOException, InterruptedException {
+    public void updateSinglePoster(ArtWork poster, int index, String outputFileName) {
         Process process = null;
         try {
             ProcessBuilder artProcessBuilder = new ProcessBuilder(Environment.MP4ART,
@@ -53,15 +53,15 @@ public void updateSinglePoster(ArtWork poster, int index, String outputFileName)
             while (!conversionJob.getStatus().isOver() && !finished) {
                 finished = process.waitFor(500, TimeUnit.MILLISECONDS);
             }
-            logger.debug("mp4art out: {}", out.toString());
-            logger.warn("mp4art err: {}", err.toString());
+            logger.debug("mp4art out: {}", out);
+            logger.warn("mp4art err: {}", err);
 
             if (process.exitValue() != 0) {
                 throw new ConversionException("ArtWork failed with code " + process.exitValue() + "!=0", new Error(err.toString()));
             }
 
             if (!new File(outputFileName).exists()) {
-                throw new ConversionException("ArtWork failed, no output file:" + out.toString(), new Error(err.toString()));
+                throw new ConversionException("ArtWork failed, no output file:" + out, new Error(err.toString()));
             }
         } catch (Exception e) {
             logger.error("Failed to apply art work", e);

src/main/java/uk/yermak/audiobookconverter/Utils.java

@@ -162,9 +162,7 @@ public static String renderPart(Part part, Map<String, Function<Part, Object>> c
         ST partTemplate = new ST(g, partFormat);
 
 
-        context.forEach((key, value) -> {
-            partTemplate.add(key, value.apply(part));
-        });
+        context.forEach((key, value) -> partTemplate.add(key, value.apply(part)));
 
         String result = partTemplate.render();
         char[] toRemove = new char[]{':', '\\', '/', '>', '<', '|', '?', '*', '"'};

src/main/java/uk/yermak/audiobookconverter/fx/BookInfoController.java

@@ -85,7 +85,7 @@ private void initialize() {
         year.textProperty().addListener(o -> AudiobookConverter.getContext().getBookInfo().year().set(year.getText()));
         comment.textProperty().addListener(o -> AudiobookConverter.getContext().getBookInfo().comment().set(comment.getText()));
 
-        AudiobookConverter.getContext().addContextDetachListener(observable -> Platform.runLater(() -> clearTags()));
+        AudiobookConverter.getContext().addContextDetachListener(observable -> Platform.runLater(this::clearTags));
     }
 
     private void updateTags(ObservableList<MediaInfo> media, boolean clear) {

src/main/java/uk/yermak/audiobookconverter/fx/ConversionContext.java

@@ -137,9 +137,7 @@ public void movePosterUp(final Integer selected) {
     }
 
     public void addPosterIfMissingWithDelay(ArtWork artWork) {
-        Platform.runLater(() -> {
-            addPosterIfMissing(artWork);
-        });
+        Platform.runLater(() -> addPosterIfMissing(artWork));
     }
 
     public void addPosterIfMissing(ArtWork artWork) {

src/main/java/uk/yermak/audiobookconverter/fx/DialogHelper.java

@@ -55,7 +55,7 @@ public static List<String> selectFilesDialog() {
 
         Arrays.stream(FILE_EXTENSIONS).map(String::toUpperCase).forEach(filetypes::add);
 
-        fileChooser.setTitle("Select " + filetypes.toString() + " files for conversion");
+        fileChooser.setTitle("Select " + filetypes + " files for conversion");
 
         fileChooser.getExtensionFilters().add(new FileChooser.ExtensionFilter("Audio", Arrays.asList(toSuffixes("*.", FILE_EXTENSIONS))));
 
@@ -67,8 +67,7 @@ public static List<String> selectFilesDialog() {
             File parentFile = firstFile.getParentFile();
             AppProperties.setProperty("source.folder", parentFile.getAbsolutePath());
         }
-        List<String> fileNames = collectFiles(files);
-        return fileNames;
+        return collectFiles(files);
     }
 
     public static List<String> selectFolderDialog() {
@@ -81,14 +80,13 @@ public static List<String> selectFolderDialog() {
 
         Arrays.stream(FILE_EXTENSIONS).map(String::toUpperCase).forEach(filetypes::add);
 
-        directoryChooser.setTitle("Select folder with " + filetypes.toString() + " files for conversion");
+        directoryChooser.setTitle("Select folder with " + filetypes + " files for conversion");
         File selectedDirectory = directoryChooser.showDialog(window);
 
         if (selectedDirectory == null) return null;
         AppProperties.setProperty("source.folder", selectedDirectory.getAbsolutePath());
 
-        List<String> fileNames = collectFiles(Collections.singleton(selectedDirectory));
-        return fileNames;
+        return collectFiles(Collections.singleton(selectedDirectory));
     }
 
     static List<String> collectFiles(Collection<File> files) {

src/main/java/uk/yermak/audiobookconverter/fx/OutputController.java

@@ -90,12 +90,9 @@ private void initialize() {
             AudiobookConverter.getContext().setSpeed(Double.valueOf(newValue));
         });
         force.getSelectionModel().select(0);
-        force.valueProperty().addListener(new ChangeListener<String>() {
-            @Override
-            public void changed(ObservableValue<? extends String> observableValue, String oldValue, String newValue) {
-                if (newValue == null) return;
-                AudiobookConverter.getContext().getOutputParameters().setForce(FORCE.equals(newValue));
-            }
+        force.valueProperty().addListener((observableValue, oldValue, newValue) -> {
+            if (newValue == null) return;
+            AudiobookConverter.getContext().getOutputParameters().setForce(FORCE.equals(newValue));
         });
 
         outputFormatBox.getItems().addAll(Format.values());

src/main/java/uk/yermak/audiobookconverter/fx/SubTracksDialog.java

@@ -7,13 +7,11 @@
 import javafx.scene.control.Spinner;
 import javafx.scene.layout.GridPane;
 import javafx.stage.Window;
-import javafx.util.Callback;
 import javafx.util.Pair;
 import org.controlsfx.control.ToggleSwitch;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import javax.imageio.plugins.tiff.GeoTIFFTagSet;
 import java.io.IOException;
 import java.lang.invoke.MethodHandles;