Ignore:
Timestamp:
2018-08-10T00:01:05+02:00 (6 years ago)
Author:
donvip
Message:

fix some warnings

Location:
applications/editors/josm/plugins/ImportImagePlugin
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/ImportImagePlugin/.settings/org.eclipse.jdt.core.prefs

    r32699 r34445  
    33org.eclipse.jdt.core.compiler.annotation.missingNonNullByDefaultAnnotation=ignore
    44org.eclipse.jdt.core.compiler.annotation.nonnull=org.eclipse.jdt.annotation.NonNull
     5org.eclipse.jdt.core.compiler.annotation.nonnull.secondary=
    56org.eclipse.jdt.core.compiler.annotation.nonnullbydefault=org.eclipse.jdt.annotation.NonNullByDefault
     7org.eclipse.jdt.core.compiler.annotation.nonnullbydefault.secondary=
    68org.eclipse.jdt.core.compiler.annotation.nullable=org.eclipse.jdt.annotation.Nullable
     9org.eclipse.jdt.core.compiler.annotation.nullable.secondary=
    710org.eclipse.jdt.core.compiler.annotation.nullanalysis=disabled
    811org.eclipse.jdt.core.compiler.codegen.inlineJsrBytecode=enabled
    … …  
    1316org.eclipse.jdt.core.compiler.debug.localVariable=generate
    1417org.eclipse.jdt.core.compiler.debug.sourceFile=generate
     18org.eclipse.jdt.core.compiler.doc.comment.support=enabled
     19org.eclipse.jdt.core.compiler.problem.APILeak=warning
    1520org.eclipse.jdt.core.compiler.problem.annotationSuperInterface=warning
    1621org.eclipse.jdt.core.compiler.problem.assertIdentifier=error
    … …  
    3641org.eclipse.jdt.core.compiler.problem.incompleteEnumSwitch=warning
    3742org.eclipse.jdt.core.compiler.problem.indirectStaticAccess=ignore
     43org.eclipse.jdt.core.compiler.problem.invalidJavadoc=warning
     44org.eclipse.jdt.core.compiler.problem.invalidJavadocTags=enabled
     45org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsDeprecatedRef=disabled
     46org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsNotVisibleRef=enabled
     47org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsVisibility=public
    3848org.eclipse.jdt.core.compiler.problem.localVariableHiding=ignore
    3949org.eclipse.jdt.core.compiler.problem.methodWithConstructorName=warning
    … …  
    4252org.eclipse.jdt.core.compiler.problem.missingEnumCaseDespiteDefault=disabled
    4353org.eclipse.jdt.core.compiler.problem.missingHashCodeMethod=ignore
     54org.eclipse.jdt.core.compiler.problem.missingJavadocComments=warning
     55org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsOverriding=disabled
     56org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsVisibility=public
     57org.eclipse.jdt.core.compiler.problem.missingJavadocTagDescription=all_standard_tags
     58org.eclipse.jdt.core.compiler.problem.missingJavadocTags=warning
     59org.eclipse.jdt.core.compiler.problem.missingJavadocTagsMethodTypeParameters=disabled
     60org.eclipse.jdt.core.compiler.problem.missingJavadocTagsOverriding=disabled
     61org.eclipse.jdt.core.compiler.problem.missingJavadocTagsVisibility=public
    4462org.eclipse.jdt.core.compiler.problem.missingOverrideAnnotation=ignore
    4563org.eclipse.jdt.core.compiler.problem.missingOverrideAnnotationForInterfaceMethodImplementation=enabled
    … …  
    5068org.eclipse.jdt.core.compiler.problem.nonExternalizedStringLiteral=ignore
    5169org.eclipse.jdt.core.compiler.problem.nonnullParameterAnnotationDropped=warning
     70org.eclipse.jdt.core.compiler.problem.nonnullTypeVariableFromLegacyInvocation=warning
    5271org.eclipse.jdt.core.compiler.problem.nullAnnotationInferenceConflict=error
    5372org.eclipse.jdt.core.compiler.problem.nullReference=warning
    … …  
    5675org.eclipse.jdt.core.compiler.problem.overridingPackageDefaultMethod=warning
    5776org.eclipse.jdt.core.compiler.problem.parameterAssignment=ignore
     77org.eclipse.jdt.core.compiler.problem.pessimisticNullAnalysisForFreeTypeVariables=warning
    5878org.eclipse.jdt.core.compiler.problem.possibleAccidentalBooleanAssignment=ignore
    5979org.eclipse.jdt.core.compiler.problem.potentialNullReference=ignore
    … …  
    7292org.eclipse.jdt.core.compiler.problem.syntacticNullAnalysisForFields=disabled
    7393org.eclipse.jdt.core.compiler.problem.syntheticAccessEmulation=ignore
     94org.eclipse.jdt.core.compiler.problem.terminalDeprecation=warning
    7495org.eclipse.jdt.core.compiler.problem.typeParameterHiding=warning
    7596org.eclipse.jdt.core.compiler.problem.unavoidableGenericTypeProblems=enabled
    … …  
    7899org.eclipse.jdt.core.compiler.problem.undocumentedEmptyBlock=ignore
    79100org.eclipse.jdt.core.compiler.problem.unhandledWarningToken=warning
     101org.eclipse.jdt.core.compiler.problem.unlikelyCollectionMethodArgumentType=warning
     102org.eclipse.jdt.core.compiler.problem.unlikelyCollectionMethodArgumentTypeStrict=disabled
     103org.eclipse.jdt.core.compiler.problem.unlikelyEqualsArgumentType=info
    80104org.eclipse.jdt.core.compiler.problem.unnecessaryElse=ignore
    81105org.eclipse.jdt.core.compiler.problem.unnecessaryTypeCheck=ignore
    … …  
    85109org.eclipse.jdt.core.compiler.problem.unusedDeclaredThrownExceptionIncludeDocCommentReference=enabled
    86110org.eclipse.jdt.core.compiler.problem.unusedDeclaredThrownExceptionWhenOverriding=disabled
     111org.eclipse.jdt.core.compiler.problem.unusedExceptionParameter=ignore
    87112org.eclipse.jdt.core.compiler.problem.unusedImport=warning
    88113org.eclipse.jdt.core.compiler.problem.unusedLabel=warning

  • applications/editors/josm/plugins/ImportImagePlugin/src/org/openstreetmap/josm/plugins/ImportImagePlugin/ImageLayer.java

    r34219 r34445  
    223223            } catch (ArrayIndexOutOfBoundsException e) {
    224224                // TODO: prevents this to happen when displaying GeoTIFF images (see #7902)
    225                 e.printStackTrace();
     225                Logging.error(e);
    226226            }
    227227

  • applications/editors/josm/plugins/ImportImagePlugin/src/org/openstreetmap/josm/plugins/ImportImagePlugin/LayerPropertiesDialog.java

    r34219 r34445  
    88import java.awt.GridBagLayout;
    99import java.awt.Rectangle;
     10import java.awt.event.KeyAdapter;
    1011import java.awt.event.KeyEvent;
    1112import java.io.File;
    … …  
    3132import org.geotools.referencing.CRS;
    3233import org.opengis.referencing.FactoryException;
    33 import org.opengis.referencing.NoSuchAuthorityCodeException;
    3434import org.opengis.referencing.crs.CoordinateReferenceSystem;
    3535import org.openstreetmap.josm.tools.Logging;
    … …  
    109109    }
    110110
    111 
    112111    /**
    113112     * This method initializes this
    … …  
    118117        this.setContentPane(getMainPanel());
    119118        this.setPreferredSize(new Dimension(404, 485));
    120 
    121119    }
    122120
    … …  
    353351            okButton.setBounds(new Rectangle(134, 5, 136, 31));
    354352            okButton.setText("OK");
    355             okButton.addActionListener(new java.awt.event.ActionListener() {
    356                 @Override
    357         public void actionPerformed(java.awt.event.ActionEvent e) {
    358                     setVisible(false);
    359                     dispose();
    360                 }
    361             });
     353            okButton.addActionListener(e -> {
     354                        setVisible(false);
     355                        dispose();
     356                    });
    362357        }
    363358        return okButton;
    … …  
    374369            searchField.setBounds(new Rectangle(13, 111, 282, 20));
    375370            searchField.setToolTipText("Enter keywords or EPSG codes");
    376             searchField.addKeyListener(new java.awt.event.KeyAdapter() {
     371            searchField.addKeyListener(new KeyAdapter() {
    377372                @Override
    378                 public void keyTyped(java.awt.event.KeyEvent e) {
     373                public void keyTyped(KeyEvent e) {
    379374
    380375                    for (Iterator<String> iterator = supportedCRS.iterator(); iterator.hasNext();) {
    … …  
    430425            useDefaultCRSButton.setBounds(new Rectangle(253, 54, 118, 28));
    431426            useDefaultCRSButton.setText("Apply Default");
    432             useDefaultCRSButton.addActionListener(new java.awt.event.ActionListener() {
    433                 @Override
    434         public void actionPerformed(java.awt.event.ActionEvent e) {
    435                     try {
    436 
    437                         setCursor(new Cursor(Cursor.WAIT_CURSOR));
    438                         if (PluginOperations.defaultSourceCRS != null) {
    439                             imageLayer.resample(PluginOperations.defaultSourceCRS);
    440                         } else {
    441                             JOptionPane.showMessageDialog(getContentPane(),
    442                                     "<html>No default reference system available.<br>Please select one from the list</html>");
     427            useDefaultCRSButton.addActionListener(e -> {
     428                        try {
     429                            setCursor(new Cursor(Cursor.WAIT_CURSOR));
     430                            if (PluginOperations.defaultSourceCRS != null) {
     431                                imageLayer.resample(PluginOperations.defaultSourceCRS);
     432                            } else {
     433                                JOptionPane.showMessageDialog(getContentPane(),
     434                                        "<html>No default reference system available.<br>Please select one from the list</html>");
     435                            }
     436                        } catch (FactoryException | IOException e1) {
     437                            Logging.error(e1);
     438                        } finally {
     439                            setCursor(new Cursor(Cursor.DEFAULT_CURSOR));
    443440                        }
    444 
    445                     } catch (NoSuchAuthorityCodeException e1) {
    446                         // TODO Auto-generated catch block
    447                         e1.printStackTrace();
    448                     } catch (FactoryException e1) {
    449                         // TODO Auto-generated catch block
    450                         e1.printStackTrace();
    451                     } catch (IOException e2) {
    452                         // TODO Auto-generated catch block
    453                         e2.printStackTrace();
    454                     } finally {
    455                         setCursor(new Cursor(Cursor.DEFAULT_CURSOR));
    456                     }
    457                 }
    458             });
     441                    });
    459442        }
    460443        return useDefaultCRSButton;
    … …  
    473456            applySelectedCRSButton.setHorizontalTextPosition(SwingConstants.TRAILING);
    474457            applySelectedCRSButton.setText("<html>Apply<br>Selection</html>");
    475             applySelectedCRSButton.addActionListener(new java.awt.event.ActionListener() {
    476                 @Override
    477         public void actionPerformed(java.awt.event.ActionEvent e) {
    478 
    479                     String selection = crsJList.getSelectedValue();
    480                     String code = selection.substring(selection.indexOf("[-") + 2, selection.indexOf("-]"));
    481 
    482                     CoordinateReferenceSystem newRefSys = null;
    483                     try {
    484                         newRefSys = CRS.decode(code, eastingFirstCheckBox.isSelected());
    485 
    486                         setCursor(new Cursor(Cursor.WAIT_CURSOR));
    487 
    488                         imageLayer.resample(newRefSys);
    489 
    490                     } catch (NoSuchAuthorityCodeException e1) {
    491                         // TODO Auto-generated catch block
    492                         e1.printStackTrace();
    493                     } catch (FactoryException e1) {
    494                         // TODO Auto-generated catch block
    495                         e1.printStackTrace();
    496                     } catch (IOException e2) {
    497                         // TODO Auto-generated catch block
    498                         e2.printStackTrace();
    499                     } finally {
    500                         setCursor(new Cursor(Cursor.DEFAULT_CURSOR));
    501                     }
    502                 }
    503             });
     458            applySelectedCRSButton.addActionListener(e -> {
     459
     460                        String selection = crsJList.getSelectedValue();
     461                        String code = selection.substring(selection.indexOf("[-") + 2, selection.indexOf("-]"));
     462
     463                        try {
     464                            CoordinateReferenceSystem newRefSys = CRS.decode(code, eastingFirstCheckBox.isSelected());
     465                            setCursor(new Cursor(Cursor.WAIT_CURSOR));
     466                            imageLayer.resample(newRefSys);
     467                        } catch (FactoryException | IOException e1) {
     468                            Logging.error(e1);
     469                        } finally {
     470                            setCursor(new Cursor(Cursor.DEFAULT_CURSOR));
     471                        }
     472                    });
    504473        }
    505474        return applySelectedCRSButton;
    … …  
    516485            setSelectedCRSAsDefaultButton.setBounds(new Rectangle(315, 300, 69, 61));
    517486            setSelectedCRSAsDefaultButton.setText("<html>Set as<br>Default</html>");
    518             setSelectedCRSAsDefaultButton
    519                     .addActionListener(new java.awt.event.ActionListener() {
    520                         @Override
    521                         public void actionPerformed(java.awt.event.ActionEvent e) {
    522 
    523                             if (crsJList.getSelectedValue() != null) {
    524                                 String selection = crsJList.getSelectedValue();
    525                                 String code = selection.substring(selection.indexOf("[-") + 2, selection.indexOf("-]"));
    526 
    527                                 try {
    528                                     PluginOperations.defaultSourceCRS = CRS.decode(code, eastingFirstCheckBox.isSelected());
    529                                     PluginOperations.defaultSourceCRSDescription = selection;
    530 
    531                                     ImportImagePlugin.pluginProps.setProperty("default_crs_eastingfirst",
    532                                             "" + eastingFirstCheckBox.isSelected());
    533                                     ImportImagePlugin.pluginProps.setProperty("default_crs_srid", code);
    534                                     try (FileWriter fileWriter = new FileWriter(new File(ImportImagePlugin.PLUGINPROPERTIES_PATH))) {
    535                                         ImportImagePlugin.pluginProps.store(fileWriter, null);
    536                                     }
    537 
    538                                     defaultCRSLabel.setText(selection);
    539 
    540                                 } catch (IOException | FactoryException e2) {
    541                                     Logging.error(e2);
     487            setSelectedCRSAsDefaultButton.addActionListener(e -> {
     488                        if (crsJList.getSelectedValue() != null) {
     489                            String selection = crsJList.getSelectedValue();
     490                            String code = selection.substring(selection.indexOf("[-") + 2, selection.indexOf("-]"));
     491
     492                            try {
     493                                PluginOperations.defaultSourceCRS = CRS.decode(code, eastingFirstCheckBox.isSelected());
     494                                PluginOperations.defaultSourceCRSDescription = selection;
     495
     496                                ImportImagePlugin.pluginProps.setProperty("default_crs_eastingfirst",
     497                                        "" + eastingFirstCheckBox.isSelected());
     498                                ImportImagePlugin.pluginProps.setProperty("default_crs_srid", code);
     499                                try (FileWriter fileWriter = new FileWriter(new File(ImportImagePlugin.PLUGINPROPERTIES_PATH))) {
     500                                    ImportImagePlugin.pluginProps.store(fileWriter, null);
    542501                                }
    543                             } else {
    544                                 JOptionPane.showMessageDialog(getContentPane(), "Please make a selection from the list.");
     502
     503                                defaultCRSLabel.setText(selection);
     504
     505                            } catch (IOException | FactoryException e2) {
     506                                Logging.error(e2);
    545507                            }
     508                        } else {
     509                            JOptionPane.showMessageDialog(getContentPane(), "Please make a selection from the list.");
    546510                        }
    547511                    });
Note: See TracChangeset for help on using the changeset viewer.