Ticket #3377: 1_less_warnings_josm.patch

File 1_less_warnings_josm.patch, 14.4 KB (added by xeen, 16 years ago)

for JOSM.

  • test/functional/org/openstreetmap/josm/gui/conflict/properties/PropertiesMergerTest.java

     
    2525    protected void populate() {
    2626        Node my = new Node(1);
    2727        my.setCoor(new LatLon(1,1));
    28         my.deleted = true;
     28        my.delete(true);
    2929
    3030        Node their = new Node(2);
    3131        their.setCoor(new LatLon(10,10));

  • src/org/openstreetmap/josm/io/DiffResultReader.java

     
    117117        Long[] nv = versions.get(key);
    118118        if (nv != null) {
    119119            processed.add(r);
    120             if (!r.deleted) {
     120            if (!r.isDeleted()) {
    121121                r.id = nv[0]; r.version = nv[1].intValue();
    122122            }
    123123        }

  • src/org/openstreetmap/josm/actions/ShowStatusReportAction.java

     
    7272
    7373                while ((line = input.readLine()) != null) {
    7474                    // Skip potential private information
    75                     if (line.trim().toLowerCase().startsWith("osm-server.username"))
     75                    if (line.trim().toLowerCase().startsWith("osm-server.username")) {
    7676                        continue;
    77                     if (line.trim().toLowerCase().startsWith("osm-server.password"))
     77                    }
     78                    if (line.trim().toLowerCase().startsWith("osm-server.password")) {
    7879                        continue;
    79                     if (line.trim().toLowerCase().startsWith("marker.show"))
     80                    }
     81                    if (line.trim().toLowerCase().startsWith("marker.show")) {
    8082                        continue;
     83                    }
    8184
    8285                    text.append(line);
    8386                    text.append("\n");
    … …  
    9699        JScrollPane sp = new JScrollPane(ta);
    97100        sp.setPreferredSize(new Dimension(600, 500));
    98101
    99         int result = new ExtendedDialog(Main.parent, tr(tr("Status Report")), sp,
    100                 new String[] {tr("Copy to clipboard and close"), tr("Close") },
    101                 new String[] {"copy.png", "cancel.png" }).getValue();
     102        ExtendedDialog ed = new ExtendedDialog(Main.parent,
     103                tr("Status Report"),
     104                new String[] {tr("Copy to clipboard and close"), tr("Close") });
     105        ed.setButtonIcons(new String[] {"copy.png", "cancel.png" });
     106        ed.setContent(sp);
     107        ed.showDialog();
    102108
    103         if(result != 1) return;
     109        if(ed.getValue() != 1) return;
    104110        try {
    105111            Toolkit.getDefaultToolkit().getSystemClipboard().setContents(
    106112                    new StringSelection(text.toString()), new ClipboardOwner() {
    107113                        public void lostOwnership(Clipboard clipboard, Transferable contents) {}
    108114                    }
    109              );
     115            );
    110116        }
    111117        catch (RuntimeException x) {}
    112118    }

  • src/org/openstreetmap/josm/actions/MergeNodesAction.java

     
    154154        // Complain to the user if the ways don't have equal memberships.
    155155        for (HashSet<Node> nodelinks : backlinks.values()) {
    156156            if (!nodelinks.containsAll(allNodes)) {
    157                 int option = new ExtendedDialog(Main.parent,
     157                ExtendedDialog ed = new ExtendedDialog(Main.parent,
    158158                        tr("Merge nodes with different memberships?"),
    159                         tr("The selected nodes have differing relation memberships.  "
    160                                 + "Do you still want to merge them?"),
    161                                 new String[] {tr("Merge Anyway"), tr("Cancel")},
    162                                 new String[] {"mergenodes.png", "cancel.png"}).getValue();
    163                 if (option == 1) {
     159                        new String[] {tr("Merge Anyway"), tr("Cancel")});
     160                ed.setButtonIcons(new String[] {"mergenodes.png", "cancel.png"});
     161                ed.setContent(tr("The selected nodes have differing relation memberships.  "
     162                        + "Do you still want to merge them?"));
     163                ed.showDialog();
     164
     165                if (ed.getValue() == 1) {
    164166                    break;
    165167                }
    166168                return null;

  • src/org/openstreetmap/josm/Main.java

     
    337337        // init default coordinate format
    338338        //
    339339        try {
    340             CoordinateFormat format = CoordinateFormat.valueOf(Main.pref.get("coordinates"));
     340            //CoordinateFormat format = CoordinateFormat.valueOf(Main.pref.get("coordinates"));
    341341            CoordinateFormat.setCoordinateFormat(CoordinateFormat.valueOf(Main.pref.get("coordinates")));
    342342        } catch (IllegalArgumentException iae) {
    343343            CoordinateFormat.setCoordinateFormat(CoordinateFormat.DECIMAL_DEGREES);
    … …  
    411411            dialog.getModel().populate(layersWithUnmodifiedChanges);
    412412            dialog.setVisible(true);
    413413            switch(dialog.getUserAction()) {
    414                 case CANCEL: return false;
    415                 case PROCEED: return true;
    416                 default: return false;
     414            case CANCEL: return false;
     415            case PROCEED: return true;
     416            default: return false;
    417417            }
    418418        }
    419419        return true;
    … …  
    445445                numUnsavedLayers,
    446446                numUnsavedLayers
    447447        );
    448         int result = new ExtendedDialog(parent, tr("Unsaved Changes"),
    449                 new JLabel(msg),
    450                 new String[] {tr("Save and Exit"), tr("Discard and Exit"), tr("Cancel")},
    451                 new String[] {"save.png", "exit.png", "cancel.png"}).getValue();
    452448
    453         switch(result) {
    454             case 2: /* discard and exit */ return true;
    455             case 3: /* cancel */ return false;
     449        ExtendedDialog ed = new ExtendedDialog(parent,
     450                tr("Unsaved Changes"),
     451                new String[] {tr("Save and Exit"), tr("Discard and Exit"), tr("Cancel")});
     452        ed.setButtonIcons(new String[] {"save.png", "exit.png", "cancel.png"});
     453        ed.setContent(new JLabel(msg));
     454        ed.showDialog();
     455
     456        switch(ed.getValue()) {
     457        case 2: /* discard and exit */ return true;
     458        case 3: /* cancel */ return false;
    456459        }
    457460        boolean savefailed = false;
    458461        for (OsmDataLayer l : map.mapView.getLayersOfType(OsmDataLayer.class)) {

  • src/org/openstreetmap/josm/command/DeletedStateConflictResolveCommand.java

     
    6868                layer.data.unlinkReferencesToPrimitive(conflict.getMy());
    6969            }
    7070        } else if (decision.equals(MergeDecisionType.KEEP_THEIR)) {
    71             if (conflict.getTheir().deleted) {
     71            if (conflict.getTheir().isDeleted()) {
    7272                layer.data.unlinkReferencesToPrimitive(conflict.getMy());
    7373                conflict.getMy().delete(true);
    7474            } else {
    75                 conflict.getMy().deleted = conflict.getTheir().deleted;
     75                conflict.getMy().delete(conflict.getTheir().isDeleted());
    7676            }
    7777        } else
    7878            // should not happen

  • src/org/openstreetmap/josm/data/APIDataSet.java

     
    3939        toAdd.clear();
    4040        toUpdate.clear();
    4141        toDelete.clear();
    42         if (ds == null)
    43             return;
     42
    4443        for (OsmPrimitive osm :ds.allPrimitives()) {
    4544            if (osm.get("josm/ignore") != null) {
    4645                continue;

  • src/org/openstreetmap/josm/gui/conflict/pair/ListMergeModel.java

     
    5555 * @see ListMerger
    5656 */
    5757public abstract class ListMergeModel<T> extends Observable {
    58     private static final Logger logger = Logger.getLogger(ListMergeModel.class.getName());
     58    //private static final Logger logger = Logger.getLogger(ListMergeModel.class.getName());
    5959
    6060    public static final String FROZEN_PROP = ListMergeModel.class.getName() + ".frozen";
    6161

  • src/org/openstreetmap/josm/gui/conflict/pair/ListMerger.java

     
    4444 * @see ListMergeModel
    4545 */
    4646public abstract class ListMerger<T> extends JPanel implements PropertyChangeListener, Observer {
    47     private static final Logger logger = Logger.getLogger(ListMerger.class.getName());
     47    //private static final Logger logger = Logger.getLogger(ListMerger.class.getName());
    4848
    4949    protected JTable myEntriesTable;
    5050    protected JTable mergedEntriesTable;

  • src/org/openstreetmap/josm/gui/conflict/pair/nodes/NodeListTableCellRenderer.java

     
    2828 *
    2929 */
    3030public  class NodeListTableCellRenderer extends JLabel implements TableCellRenderer {
    31     static private final Logger logger = Logger.getLogger(NodeListTableCellRenderer.class.getName());
    32     private static DecimalFormat COORD_FORMATTER = new DecimalFormat("###0.0000");
     31    //static private final Logger logger = Logger.getLogger(NodeListTableCellRenderer.class.getName());
     32    //private static DecimalFormat COORD_FORMATTER = new DecimalFormat("###0.0000");
    3333    public final static Color BGCOLOR_SELECTED = new Color(143,170,255);
    3434    public final static Color BGCOLOR_EMPTY_ROW = new Color(234,234,234);
    3535    public final static Color BGCOLOR_FROZEN = new Color(234,234,234);

  • src/org/openstreetmap/josm/gui/FileDrop.java

     
    88import java.io.IOException;
    99import java.io.PrintStream;
    1010import java.io.Reader;
     11import java.util.List;
     12
     13import javax.swing.BorderFactory;
    1114
    1215import org.openstreetmap.josm.actions.OpenFileAction;
    1316
    … …  
    7073    {   this(
    7174            null,  // Logging stream
    7275            c,     // Drop target
    73             javax.swing.BorderFactory.createMatteBorder( 2, 2, 2, 2, defaultBorderColor ), // Drag border
     76            BorderFactory.createMatteBorder( 2, 2, 2, 2, defaultBorderColor ), // Drag border
    7477            true, // Recursive
    7578            new FileDrop.Listener()
    7679            {
    … …  
    340343                    log( out, "FileDrop: file list accepted." );
    341344
    342345                    // Get a useful list
    343                     java.util.List fileList = (java.util.List)
     346                    List fileList = (java.util.List)
    344347                    tr.getTransferData(java.awt.datatransfer.DataFlavor.javaFileListFlavor);
    345348
    346349                    // Convert list to array
    347                     final java.io.File[] files = (File[]) fileList.toArray();;
     350                    final File[] files = (File[]) fileList.toArray();
    348351
    349352                    // Alert listener to drop.
    350353                    if( listener != null ) {

  • src/org/openstreetmap/josm/gui/dialogs/LayerListDialog.java

     
    6161 *
    6262 */
    6363public class LayerListDialog extends ToggleDialog {
    64     static private final Logger logger = Logger.getLogger(LayerListDialog.class.getName());
     64    //static private final Logger logger = Logger.getLogger(LayerListDialog.class.getName());
    6565
    6666    /** the unique instance of the dialog */
    6767    static private LayerListDialog instance;
    … …  
    316316                dialog.getModel().populate(layersWithUnmodifiedChanges);
    317317                dialog.setVisible(true);
    318318                switch(dialog.getUserAction()) {
    319                     case CANCEL: return false;
    320                     case PROCEED: return true;
    321                     default: return false;
     319                case CANCEL: return false;
     320                case PROCEED: return true;
     321                default: return false;
    322322                }
    323323            }
    324324            return true;

  • src/org/openstreetmap/josm/gui/MapStatus.java

     
    172172                        if (osms == null) {
    173173                            continue;
    174174                        }
    175                         if (osms != null && osms.equals(osmStatus) && ms.modifiers == oldModifiers) {
     175                        if (osms.equals(osmStatus) && ms.modifiers == oldModifiers) {
    176176                            continue;
    177177                        }
    178178