Comment #1: /home/edmund/evaluate_projects/SweetHome3D-4.4-src/src/com/eteks/sweethome3d/io/DefaultHomeOutputStream.java: 345-348 [COMMENT #1] Wrap the exception and re-throw // If content is a resource coming from a directory (this should be the case // only when resource isn't in a JAR file during development), retrieve its file name if (resourceUrlContent.isMultiPartResource()) { try { subEntryName = "/" + new File(resourceUrlContent.getURL().toURI()).getName(); } catch (URISyntaxException ex) { ****** COMMENT #1 ****** IOException ex2 = new IOException(); ex2.initCause(ex); throw ex2; } } # Rating on this comment # 1: accurate, adequate, concise and useful at describing the source code. (committable) # 2: not accurate, adequate, concise, or useful, at describing the source code but can be fixed with minor modifications. # (committable but require fixes) # 3: not accurate, adequate, concise, or useful at describing the source code. (not committable) # (Fill in after the keyword: 1, 2 or 3) CLASSIFICATION: 1 ———————————— Comment #2: /home/edmund/evaluate_projects/SweetHome3D-4.4-src/src/com/eteks/sweethome3d/io/ContentDigestManager.java: 187-190 [COMMENT #1] Wrap the exception and re-throw try { MessageDigest messageDigest = MessageDigest.getInstance(DIGEST_ALGORITHM); File contentFile = new File(urlContent.getURL().toURI()); File parentFile = new File(contentFile.getParent()); File [] siblingFiles = parentFile.listFiles(); // Sort files to ensure content files are always listed in the same order Arrays.sort(siblingFiles); for (File siblingFile : siblingFiles) { if (!siblingFile.isDirectory()) { updateMessageDigest(messageDigest, new URLContent(siblingFile.toURI().toURL())); } } return messageDigest.digest(); } catch (URISyntaxException ex) { ****** COMMENT #1 ****** IOException ex2 = new IOException(); ex2.initCause(ex); throw ex2; } # Rating on this comment # 1: accurate, adequate, concise and useful at describing the source code. (committable) # 2: not accurate, adequate, concise, or useful, at describing the source code but can be fixed with minor modifications. # (committable but require fixes) # 3: not accurate, adequate, concise, or useful at describing the source code. (not committable) # (Fill in after the keyword: 1, 2 or 3) CLASSIFICATION: 1 ———————————— Comment #3: /home/edmund/evaluate_projects/SweetHome3D-4.4-src/src/com/eteks/sweethome3d/tools/URLContent.java: 68-71 [COMMENT #1] Wrap the exception and re-throw try { if (new File(jarEntryURL.toURI()).canWrite()) { // Even if cache is actually not used for JAR entries of files, refuse explicitly to use // caches to be able to delete the writable files accessed with jar protocol under Windows, // as suggested in http://bugs.sun.com/bugdatabase/view_bug. do?bug_id=6962459 connection.setUseCaches(false); } } catch (URISyntaxException ex) { ****** COMMENT #1 ****** IOException ex2 = new IOException(); ex2.initCause(ex); throw ex2; } # Rating on this comment # 1: accurate, adequate, concise and useful at describing the source code. (committable) # 2: not accurate, adequate, concise, or useful, at describing the source code but can be fixed with minor modifications. # (committable but require fixes) # 3: not accurate, adequate, concise, or useful at describing the source code. (not committable) # (Fill in after the keyword: 1, 2 or 3) CLASSIFICATION: 1 ———————————— Comment #4: /home/edmund/evaluate_projects/SweetHome3D-4.4-src/src/com/eteks/sweethome3d/io/DefaultHomeOutputStream.java: 199-202 [COMMENT #1] Wrap the exception and re-throw try { if (new File(jarEntryURL.toURI()).canWrite()) { // Even if cache is actually not used for JAR entries of files, refuse explicitly to use // caches to be able to delete the writable files accessed with jar protocol under Windows, // as suggested in http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6962459 connection.setUseCaches(false); } } catch (URISyntaxException ex) { ****** COMMENT #1 ****** IOException ex2 = new IOException(); ex2.initCause(ex); throw ex2; } # Rating on this comment # 1: accurate, adequate, concise and useful at describing the source code. (committable) # 2: not accurate, adequate, concise, or useful, at describing the source code but can be fixed with minor modifications. # (committable but require fixes) # 3: not accurate, adequate, concise, or useful at describing the source code. (not committable) # (Fill in after the keyword: 1, 2 or 3) CLASSIFICATION: 1 ————————————