Merge pull request #579 from csmith/master

Make existing Log.error calls use app/user markers.

src/com/dmdirc/plugins/ExportedService.java

@@ -22,6 +22,8 @@
 
 package com.dmdirc.plugins;
 
+import com.dmdirc.util.LogUtils;
+
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
 
@@ -76,10 +78,9 @@ public class ExportedService {
     public Object execute(final Object... args) {
         try {
             return method.invoke(object, args);
-        } catch (final LinkageError | IllegalAccessException |
-                IllegalArgumentException | InvocationTargetException le) {
-            LOG.error("Error with exported service calling {}",
-                    method, le);
+        } catch (LinkageError | IllegalAccessException | IllegalArgumentException
+                | InvocationTargetException le) {
+            LOG.error(LogUtils.APP_ERROR, "Error with exported service calling {}", method, le);
             return null;
         }
     }

src/com/dmdirc/updater/checking/DMDircCheckStrategy.java

@@ -28,6 +28,7 @@ import com.dmdirc.interfaces.config.AggregateConfigProvider;
 import com.dmdirc.updater.UpdateChannel;
 import com.dmdirc.updater.UpdateComponent;
 import com.dmdirc.updater.Version;
+import com.dmdirc.util.LogUtils;
 import com.dmdirc.util.io.Downloader;
 
 import java.io.IOException;
@@ -200,7 +201,8 @@ public class DMDircCheckStrategy implements UpdateCheckStrategy {
             return new BaseDownloadableResult(component, new URL(parts[5]),
                     parts[4], new Version(parts[3]));
         } catch (MalformedURLException ex) {
-            LOG.error("Unable to construct URL for update. Parts: {}", parts, ex);
+            LOG.error(LogUtils.APP_ERROR, "Unable to construct URL for update. Parts: {}", parts,
+                    ex);
             return null;
         }
     }

src/com/dmdirc/util/BaseYamlStore.java

@@ -94,7 +94,7 @@ public abstract class BaseYamlStore<T> {
             yamlWriter.write(items.parallelStream().map(this::convertToYaml).collect(toList()));
             yamlWriter.close();
         } catch (IOException ex) {
-            LOG.error("Unable to write to {}", path, ex);
+            LOG.error(LogUtils.USER_ERROR, "Unable to write to {}", path, ex);
         }
     }