]> gerrit.simantics Code Review - simantics/platform.git/commitdiff
Possibility to open SCL-modules from plug-in in JAR-file 17/2717/1
authorjsimomaa <jani.simomaa@gmail.com>
Tue, 7 Aug 2018 11:03:55 +0000 (14:03 +0300)
committerHannu Niemistö <hannu.niemisto@semantum.fi>
Tue, 26 Feb 2019 09:20:34 +0000 (11:20 +0200)
The real NPE problem was already fixed in c52bf8a8 by Hannu, but this
adds some SLF4J logging.

gitlab #77

Change-Id: I9ce2e1efca375046b0b5f8a5dd483e1e288d594a

bundles/org.simantics.scl.osgi/src/org/simantics/scl/osgi/internal/BundleModuleSource.java

index fc9799de706d6625e401d47dd3320aa7a900cc91..87b78d6ebf44434c61a252baf8fc339886bc9d1f 100644 (file)
@@ -1,5 +1,6 @@
 package org.simantics.scl.osgi.internal;
 
+
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
@@ -19,11 +20,15 @@ import org.simantics.scl.compiler.module.ImportDeclaration;
 import org.simantics.scl.compiler.module.repository.UpdateListener;
 import org.simantics.scl.compiler.source.EncodedTextualModuleSource;
 import org.simantics.scl.compiler.types.Type;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import gnu.trove.set.hash.THashSet;
 
 public class BundleModuleSource extends EncodedTextualModuleSource implements UpdateListener.Observable {
 
+    private static final Logger LOGGER = LoggerFactory.getLogger(BundleModuleSource.class);
+
     public static final ImportDeclaration[] DEFAULT_IMPORTS = new ImportDeclaration[] {
         new ImportDeclaration("Builtin", ""),
         new ImportDeclaration("StandardLibrary", "")
@@ -75,13 +80,13 @@ public class BundleModuleSource extends EncodedTextualModuleSource implements Up
                 }
                 return digest.digest();
             } catch (NoSuchAlgorithmException e) {
-                e.printStackTrace();
+                LOGGER.error("No SHA1 algorithm available", e);
                 return new byte[0];
             } finally {
                 stream.close();
             }
         } catch(IOException e) {
-            e.printStackTrace();
+            LOGGER.error("Could not compute digest for {}", this);
             return new byte[0];
         }
     }
@@ -151,10 +156,11 @@ public class BundleModuleSource extends EncodedTextualModuleSource implements Up
                 return false;
             return Files.exists(path);
         } catch (IOException e) {
+            LOGGER.debug("Could not check if {} is updateable", this, e);
             return false;
         }
     }
-    
+
     @Override
     public void update(String newSourceText) {
         try {
@@ -163,7 +169,7 @@ public class BundleModuleSource extends EncodedTextualModuleSource implements Up
                 return;
             Files.write(path, newSourceText.getBytes(Charset.forName("UTF-8")));
         } catch(IOException e) {
-            e.printStackTrace();
+            LOGGER.error("Could not update {} with {}", this, newSourceText, e);
         }
         checkUpdates();
     }