Ignore:
Timestamp:
2015-05-02T00:55:19+02:00 (10 years ago)
Author:
Don-vip
Message:

fix potential NPEs and Sonar issues related to serialization

Location:
trunk/src/org/openstreetmap/josm/tools
Files:
6 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/tools/AlphanumComparator.java

    r7937 r8308  
    2323 *
    2424 */
     25import java.io.Serializable;
    2526import java.text.Collator;
    2627import java.util.Comparator;
     
    3839 *
    3940 */
    40 public class AlphanumComparator implements Comparator<String> {
     41public class AlphanumComparator implements Comparator<String>, Serializable {
     42
     43    private static final long serialVersionUID = 1L;
    4144
    4245    private static final AlphanumComparator INSTANCE = new AlphanumComparator();
  • trunk/src/org/openstreetmap/josm/tools/FontsManager.java

    r7402 r8308  
    3939            String url = "resource://data/fonts/"+fontFile;
    4040            try (InputStream i = new CachedFile(url).getInputStream()) {
    41                 Font f = Font.createFont(Font.TRUETYPE_FONT, i);
    42                 if (f == null) {
    43                     throw new RuntimeException("unable to load font: "+fontFile);
    44                 }
    45                 ge.registerFont(f);
     41                ge.registerFont(Font.createFont(Font.TRUETYPE_FONT, i));
    4642            } catch (IOException | FontFormatException ex) {
    4743                throw new RuntimeException(ex);
  • trunk/src/org/openstreetmap/josm/tools/MultikeyActionsHandler.java

    r8285 r8308  
    6767    private class MyAction extends AbstractAction {
    6868
    69         private final MultikeyShortcutAction action;
    70         private final Shortcut shortcut;
     69        private final transient MultikeyShortcutAction action;
     70        private final transient Shortcut shortcut;
    7171
    7272        MyAction(MultikeyShortcutAction action) {
  • trunk/src/org/openstreetmap/josm/tools/PlatformHookWindows.java

    r8099 r8308  
    318318        try (DirectoryStream<Path> ds = Files.newDirectoryStream(fontPath)) {
    319319            for (Path p : ds) {
    320                 fontsAvail.add(p.getFileName().toString().toUpperCase());
     320                Path filename = p.getFileName();
     321                if (filename != null) {
     322                    fontsAvail.add(filename.toString().toUpperCase());
     323                }
    321324            }
    322325            fontsAvail.add(""); // for devanagari
  • trunk/src/org/openstreetmap/josm/tools/Utils.java

    r8304 r8308  
    378378            Main.warn("Unable to create directory "+out.getPath());
    379379        }
    380         for (File f : in.listFiles()) {
    381             File target = new File(out, f.getName());
    382             if (f.isDirectory()) {
    383                 copyDirectory(f, target);
    384             } else {
    385                 copyFile(f, target);
     380        File[] files = in.listFiles();
     381        if (files != null) {
     382            for (File f : files) {
     383                File target = new File(out, f.getName());
     384                if (f.isDirectory()) {
     385                    copyDirectory(f, target);
     386                } else {
     387                    copyFile(f, target);
     388                }
    386389            }
    387390        }
     
    415418        if( path.exists() ) {
    416419            File[] files = path.listFiles();
    417             for (File file : files) {
    418                 if (file.isDirectory()) {
    419                     deleteDirectory(file);
    420                 } else if (!file.delete()) {
    421                     Main.warn("Unable to delete file: "+file.getPath());
     420            if (files != null) {
     421                for (File file : files) {
     422                    if (file.isDirectory()) {
     423                        deleteDirectory(file);
     424                    } else if (!file.delete()) {
     425                        Main.warn("Unable to delete file: "+file.getPath());
     426                    }
    422427                }
    423428            }
  • trunk/src/org/openstreetmap/josm/tools/template_engine/ParseError.java

    r7937 r8308  
    99public class ParseError extends Exception {
    1010
    11     private final Token unexpectedToken;
     11    private final transient Token unexpectedToken;
    1212
    1313    public ParseError(Token unexpectedToken) {
Note: See TracChangeset for help on using the changeset viewer.