Ignore:
Timestamp:
2015-04-29T01:44:01+02:00 (4 years ago)
Author:
Don-vip
Message:

fix squid:RedundantThrowsDeclarationCheck + consistent Javadoc for exceptions

Location:
trunk/src/org/openstreetmap/josm/gui/history
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/history/CoordinateInfoViewer.java

    r6883 r8291  
    102102     *
    103103     * @param model the model. Must not be null.
    104      * @throws IllegalArgumentException thrown if model is null
     104     * @throws IllegalArgumentException if model is null
    105105     */
    106     public CoordinateInfoViewer(HistoryBrowserModel model) throws IllegalArgumentException{
     106    public CoordinateInfoViewer(HistoryBrowserModel model) {
    107107        CheckParameterUtil.ensureParameterNotNull(model, "model");
    108108        setModel(model);
  • trunk/src/org/openstreetmap/josm/gui/history/HistoryBrowserModel.java

    r8219 r8291  
    115115     *
    116116     * @param history the history. Must not be null.
    117      * @throws IllegalArgumentException thrown if history is null
     117     * @throws IllegalArgumentException if history is null
    118118     */
    119119    public HistoryBrowserModel(History history) {
     
    247247     * @param pointInTimeType the type of the point in time (must not be null)
    248248     * @return the tag table model
    249      * @exception IllegalArgumentException thrown, if pointInTimeType is null
    250      */
    251     public TagTableModel getTagTableModel(PointInTimeType pointInTimeType) throws IllegalArgumentException {
     249     * @throws IllegalArgumentException if pointInTimeType is null
     250     */
     251    public TagTableModel getTagTableModel(PointInTimeType pointInTimeType) {
    252252        CheckParameterUtil.ensureParameterNotNull(pointInTimeType, "pointInTimeType");
    253253        if (pointInTimeType.equals(PointInTimeType.CURRENT_POINT_IN_TIME))
     
    260260    }
    261261
    262     public DiffTableModel getNodeListTableModel(PointInTimeType pointInTimeType) throws IllegalArgumentException {
     262    public DiffTableModel getNodeListTableModel(PointInTimeType pointInTimeType) {
    263263        CheckParameterUtil.ensureParameterNotNull(pointInTimeType, "pointInTimeType");
    264264        if (pointInTimeType.equals(PointInTimeType.CURRENT_POINT_IN_TIME))
     
    271271    }
    272272
    273     public DiffTableModel getRelationMemberTableModel(PointInTimeType pointInTimeType) throws IllegalArgumentException {
     273    public DiffTableModel getRelationMemberTableModel(PointInTimeType pointInTimeType) {
    274274        CheckParameterUtil.ensureParameterNotNull(pointInTimeType, "pointInTimeType");
    275275        if (pointInTimeType.equals(PointInTimeType.CURRENT_POINT_IN_TIME))
     
    287287     *
    288288     * @param reference the reference history primitive. Must not be null.
    289      * @throws IllegalArgumentException thrown if reference is null
    290      * @throws IllegalStateException thrown if this model isn't a assigned a history yet
     289     * @throws IllegalArgumentException if reference is null
     290     * @throws IllegalStateException if this model isn't a assigned a history yet
    291291     * @throws IllegalArgumentException if reference isn't an history primitive for the history managed by this mode
    292292     *
     
    294294     * @see PointInTimeType
    295295     */
    296     public void setReferencePointInTime(HistoryOsmPrimitive reference) throws IllegalArgumentException, IllegalStateException{
     296    public void setReferencePointInTime(HistoryOsmPrimitive reference) {
    297297        CheckParameterUtil.ensureParameterNotNull(reference, "reference");
    298298        if (history == null)
     
    317317     *
    318318     * @param current the reference history primitive. Must not be {@code null}.
    319      * @throws IllegalArgumentException thrown if reference is {@code null}
    320      * @throws IllegalStateException thrown if this model isn't a assigned a history yet
     319     * @throws IllegalArgumentException if reference is {@code null}
     320     * @throws IllegalStateException if this model isn't a assigned a history yet
    321321     * @throws IllegalArgumentException if reference isn't an history primitive for the history managed by this mode
    322322     *
     
    324324     * @see PointInTimeType
    325325     */
    326     public void setCurrentPointInTime(HistoryOsmPrimitive current) throws IllegalArgumentException, IllegalStateException{
     326    public void setCurrentPointInTime(HistoryOsmPrimitive current) {
    327327        CheckParameterUtil.ensureParameterNotNull(current, "current");
    328328        if (history == null)
     
    364364     * @param type the type of the point in time (must not be null)
    365365     * @return the respective primitive. Can be null.
    366      * @exception IllegalArgumentException thrown, if type is null
    367      */
    368     public HistoryOsmPrimitive getPointInTime(PointInTimeType type) throws IllegalArgumentException {
     366     * @throws IllegalArgumentException if type is null
     367     */
     368    public HistoryOsmPrimitive getPointInTime(PointInTimeType type) {
    369369        CheckParameterUtil.ensureParameterNotNull(type, "type");
    370370        if (type.equals(PointInTimeType.CURRENT_POINT_IN_TIME))
  • trunk/src/org/openstreetmap/josm/gui/history/HistoryLoadTask.java

    r7005 r8291  
    7070     * parent for {@link org.openstreetmap.josm.gui.PleaseWaitDialog}.
    7171     * Must not be <code>null</code>.
    72      * @throws IllegalArgumentException thrown if parent is <code>null</code>
     72     * @throws IllegalArgumentException if parent is <code>null</code>
    7373     */
    7474    public HistoryLoadTask(Component parent) {
     
    8585     * @return this task
    8686     */
    87     public HistoryLoadTask add(long id, OsmPrimitiveType type) throws IllegalArgumentException {
     87    public HistoryLoadTask add(long id, OsmPrimitiveType type) {
    8888        if (id <= 0)
    8989            throw new IllegalArgumentException(MessageFormat.format("Parameter ''{0}'' > 0 expected. Got {1}.", "id", id));
     
    111111     * @param primitive the history item
    112112     * @return this task
    113      * @throws IllegalArgumentException thrown if primitive is null
     113     * @throws IllegalArgumentException if primitive is null
    114114     */
    115115    public HistoryLoadTask add(HistoryOsmPrimitive primitive) {
     
    124124     * @param history the history. Must not be null.
    125125     * @return this task
    126      * @throws IllegalArgumentException thrown if history is null
     126     * @throws IllegalArgumentException if history is null
    127127     */
    128128    public HistoryLoadTask add(History history) {
     
    137137     * @param primitive the OSM primitive. Must not be null. primitive.getId() &gt; 0 required.
    138138     * @return this task
    139      * @throws IllegalArgumentException thrown if the primitive is null
    140      * @throws IllegalArgumentException thrown if primitive.getId() &lt;= 0
     139     * @throws IllegalArgumentException if the primitive is null
     140     * @throws IllegalArgumentException if primitive.getId() &lt;= 0
    141141     */
    142142    public HistoryLoadTask add(OsmPrimitive primitive) {
     
    152152     * <code>primitive.getId() &gt; 0</code> required.
    153153     * @return this task
    154      * @throws IllegalArgumentException thrown if primitives is <code>null</code>
    155      * @throws IllegalArgumentException thrown if one of the ids in the collection &lt;= 0
     154     * @throws IllegalArgumentException if primitives is <code>null</code>
     155     * @throws IllegalArgumentException if one of the ids in the collection &lt;= 0
    156156     */
    157157    public HistoryLoadTask add(Collection<? extends OsmPrimitive> primitives) {
  • trunk/src/org/openstreetmap/josm/gui/history/VersionInfoPanel.java

    r8254 r8291  
    172172     * @param model  the model (must not be null)
    173173     * @param pointInTimeType the point in time this panel visualizes (must not be null)
    174      * @exception IllegalArgumentException thrown, if model is null
    175      * @exception IllegalArgumentException thrown, if pointInTimeType is null
    176      *
     174     * @throws IllegalArgumentException if model is null
     175     * @throws IllegalArgumentException if pointInTimeType is null
    177176     */
    178     public VersionInfoPanel(HistoryBrowserModel model, PointInTimeType pointInTimeType) throws IllegalArgumentException {
     177    public VersionInfoPanel(HistoryBrowserModel model, PointInTimeType pointInTimeType) {
    179178        CheckParameterUtil.ensureParameterNotNull(pointInTimeType, "pointInTimeType");
    180179        CheckParameterUtil.ensureParameterNotNull(model, "model");
Note: See TracChangeset for help on using the changeset viewer.