Modify

Opened 7 years ago

Closed 7 years ago

Last modified 6 weeks ago

#14794 closed defect (fixed)

Javadoc for every top level class

Reported by: bastiK Owned by: team
Priority: normal Milestone: 17.08
Component: Core Version:
Keywords: javadoc Cc:

Description (last modified by bastiK)

We should have a short description (Javadoc) for the main class or interface in each .java file. Below is a list of all files that are currently missing this basic documentation:

  • src/org/openstreetmap/josm/actions/ActionParameter.java
  • src/org/openstreetmap/josm/actions/AdaptableAction.java
  • src/org/openstreetmap/josm/actions/CloseChangesetAction.java
  • src/org/openstreetmap/josm/actions/CopyCoordinatesAction.java
  • src/org/openstreetmap/josm/actions/ParameterizedAction.java
  • src/org/openstreetmap/josm/actions/ParameterizedActionDecorator.java
  • src/org/openstreetmap/josm/actions/PreferenceToggleAction.java
  • src/org/openstreetmap/josm/actions/ReverseWayAction.java
  • src/org/openstreetmap/josm/actions/SelectAllAction.java
  • src/org/openstreetmap/josm/actions/ShowStatusReportAction.java
  • src/org/openstreetmap/josm/actions/ToggleGPXLinesAction.java
  • src/org/openstreetmap/josm/actions/UnselectAllAction.java
  • src/org/openstreetmap/josm/actions/UpdateDataAction.java
  • src/org/openstreetmap/josm/actions/ZoomToAction.java
  • src/org/openstreetmap/josm/actions/downloadtasks/DownloadNotesUrlBoundsTask.java
  • src/org/openstreetmap/josm/actions/downloadtasks/DownloadNotesUrlIdTask.java
  • src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmIdTask.java
  • src/org/openstreetmap/josm/actions/downloadtasks/PostDownloadHandler.java
  • src/org/openstreetmap/josm/actions/mapmode/DrawSnapHelper.java
  • src/org/openstreetmap/josm/actions/mapmode/ImproveWayAccuracyAction.java
  • src/org/openstreetmap/josm/actions/mapmode/ParallelWayAction.java
  • src/org/openstreetmap/josm/actions/search/PushbackTokenizer.java
  • src/org/openstreetmap/josm/actions/search/SearchAction.java
  • src/org/openstreetmap/josm/actions/upload/ApiPreconditionCheckerHook.java
  • src/org/openstreetmap/josm/actions/upload/CyclicUploadDependencyException.java
  • src/org/openstreetmap/josm/actions/upload/UploadHook.java
  • src/org/openstreetmap/josm/command/ScaleCommand.java
  • src/org/openstreetmap/josm/data/cache/ICachedLoaderJob.java
  • src/org/openstreetmap/josm/data/conflict/IConflictListener.java
  • src/org/openstreetmap/josm/data/coor/QuadTiling.java
  • src/org/openstreetmap/josm/data/gpx/GpxRoute.java
  • src/org/openstreetmap/josm/data/gpx/ImmutableGpxTrackSegment.java
  • src/org/openstreetmap/josm/data/gpx/WayPoint.java
  • src/org/openstreetmap/josm/data/imagery/Shape.java
  • src/org/openstreetmap/josm/data/oauth/OAuthToken.java
  • src/org/openstreetmap/josm/data/osm/ChangesetCacheEvent.java
  • src/org/openstreetmap/josm/data/osm/ChangesetCacheListener.java
  • src/org/openstreetmap/josm/data/osm/DefaultChangesetCacheEvent.java
  • src/org/openstreetmap/josm/data/osm/NodeData.java
  • src/org/openstreetmap/josm/data/osm/OsmUtils.java
  • src/org/openstreetmap/josm/data/osm/RelationMemberData.java
  • src/org/openstreetmap/josm/data/osm/RelationToChildReference.java
  • src/org/openstreetmap/josm/data/osm/SimplePrimitiveId.java
  • src/org/openstreetmap/josm/data/osm/WayData.java
  • src/org/openstreetmap/josm/data/osm/event/ChangesetIdChangedEvent.java
  • src/org/openstreetmap/josm/data/osm/event/DataChangedEvent.java
  • src/org/openstreetmap/josm/data/osm/event/NodeMovedEvent.java
  • src/org/openstreetmap/josm/data/osm/event/PrimitivesAddedEvent.java
  • src/org/openstreetmap/josm/data/osm/event/PrimitivesRemovedEvent.java
  • src/org/openstreetmap/josm/data/osm/event/RelationMembersChangedEvent.java
  • src/org/openstreetmap/josm/data/osm/event/TagsChangedEvent.java
  • src/org/openstreetmap/josm/data/osm/event/WayNodesChangedEvent.java
  • src/org/openstreetmap/josm/data/osm/history/HistoryDataSetListener.java
  • src/org/openstreetmap/josm/data/osm/visitor/paint/PaintColors.java
  • src/org/openstreetmap/josm/data/preferences/CachedProperty.java
  • src/org/openstreetmap/josm/data/preferences/ParametrizedEnumProperty.java
  • src/org/openstreetmap/josm/data/projection/ProjectionChangeListener.java
  • src/org/openstreetmap/josm/data/projection/ProjectionConfigurationException.java
  • src/org/openstreetmap/josm/data/projection/datum/AbstractDatum.java
  • src/org/openstreetmap/josm/data/validation/ValidatorVisitor.java
  • src/org/openstreetmap/josm/gui/GettingStarted.java
  • src/org/openstreetmap/josm/gui/HelpAwareOptionPane.java
  • src/org/openstreetmap/josm/gui/MapSlider.java
  • src/org/openstreetmap/josm/gui/NameFormatterHook.java
  • src/org/openstreetmap/josm/gui/bbox/SlippyMapBBoxChooser.java
  • src/org/openstreetmap/josm/gui/bbox/SourceButton.java
  • src/org/openstreetmap/josm/gui/conflict/pair/ComparePairListCellRenderer.java
  • src/org/openstreetmap/josm/gui/conflict/pair/IConflictResolver.java
  • src/org/openstreetmap/josm/gui/conflict/pair/MergeDecisionType.java
  • src/org/openstreetmap/josm/gui/conflict/pair/PairTable.java
  • src/org/openstreetmap/josm/gui/conflict/pair/tags/MergedTableCellRenderer.java
  • src/org/openstreetmap/josm/gui/conflict/pair/tags/MineTableCellRenderer.java
  • src/org/openstreetmap/josm/gui/conflict/pair/tags/TagMergeTableCellRenderer.java
  • src/org/openstreetmap/josm/gui/conflict/pair/tags/TheirTableCellRenderer.java
  • src/org/openstreetmap/josm/gui/conflict/tags/PasteTagsConflictResolverDialog.java
  • src/org/openstreetmap/josm/gui/conflict/tags/RelationMemberConflictDecision.java
  • src/org/openstreetmap/josm/gui/conflict/tags/RelationMemberConflictDecisionRenderer.java
  • src/org/openstreetmap/josm/gui/conflict/tags/RelationMemberConflictDecisionType.java
  • src/org/openstreetmap/josm/gui/conflict/tags/RelationMemberConflictResolver.java
  • src/org/openstreetmap/josm/gui/conflict/tags/RelationMemberConflictResolverColumnModel.java
  • src/org/openstreetmap/josm/gui/conflict/tags/RelationMemberConflictResolverTable.java
  • src/org/openstreetmap/josm/gui/conflict/tags/TagConflictResolverModel.java
  • src/org/openstreetmap/josm/gui/conflict/tags/TagConflictResolverTable.java
  • src/org/openstreetmap/josm/gui/dialogs/FilterDialog.java
  • src/org/openstreetmap/josm/gui/dialogs/FilterTableModel.java
  • src/org/openstreetmap/josm/gui/dialogs/LatLonDialog.java
  • src/org/openstreetmap/josm/gui/dialogs/MenuItemSearchDialog.java
  • src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetInSelectionListModel.java
  • src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetListCellRenderer.java
  • src/org/openstreetmap/josm/gui/dialogs/changeset/ChangesetListModel.java
  • src/org/openstreetmap/josm/gui/dialogs/properties/RecentTagCollection.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/IMemberModelListener.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/MemberRoleCellEditor.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/MemberTable.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableColumnModel.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableLinkedCellRenderer.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableModel.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableRoleCellRenderer.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/ReferringRelationsBrowserModel.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/SelectionTableModel.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/sort/RelationSortUtils.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/sort/RelationSorter.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/sort/WayConnectionType.java
  • src/org/openstreetmap/josm/gui/dialogs/relation/sort/WayConnectionTypeCalculator.java
  • src/org/openstreetmap/josm/gui/download/DownloadSelection.java
  • src/org/openstreetmap/josm/gui/history/NodeListTableCellRenderer.java
  • src/org/openstreetmap/josm/gui/history/SelectionSynchronizer.java
  • src/org/openstreetmap/josm/gui/io/AbstractUploadTask.java
  • src/org/openstreetmap/josm/gui/io/ConfigurationParameterRequestHandler.java
  • src/org/openstreetmap/josm/gui/io/CredentialDialog.java
  • src/org/openstreetmap/josm/gui/io/MaxChangesetSizeExceededPolicy.java
  • src/org/openstreetmap/josm/gui/io/SaveLayersDialog.java
  • src/org/openstreetmap/josm/gui/io/SaveLayersModel.java
  • src/org/openstreetmap/josm/gui/io/SaveLayersTable.java
  • src/org/openstreetmap/josm/gui/io/SaveLayersTableColumnModel.java
  • src/org/openstreetmap/josm/gui/io/UploadAndSaveProgressRenderer.java
  • src/org/openstreetmap/josm/gui/io/UploadOrSaveState.java
  • src/org/openstreetmap/josm/gui/io/UploadParameterSummaryPanel.java
  • src/org/openstreetmap/josm/gui/io/UploadStrategy.java
  • src/org/openstreetmap/josm/gui/layer/CustomizeColor.java
  • src/org/openstreetmap/josm/gui/layer/GpxLayer.java
  • src/org/openstreetmap/josm/gui/layer/ImageryLayer.java
  • src/org/openstreetmap/josm/gui/layer/JumpToMarkerActions.java
  • src/org/openstreetmap/josm/gui/layer/geoimage/ImageDisplay.java
  • src/org/openstreetmap/josm/gui/layer/geoimage/ImageViewerDialog.java
  • src/org/openstreetmap/josm/gui/layer/geoimage/ThumbsLoader.java
  • src/org/openstreetmap/josm/gui/layer/gpx/CustomizeDrawingAction.java
  • src/org/openstreetmap/josm/gui/layer/gpx/DateFilterPanel.java
  • src/org/openstreetmap/josm/gui/layer/gpx/ImportImagesAction.java
  • src/org/openstreetmap/josm/gui/layer/gpx/MarkersFromNamedPointsAction.java
  • src/org/openstreetmap/josm/gui/mappaint/StyleKeys.java
  • src/org/openstreetmap/josm/gui/mappaint/mapcss/CSSColors.java
  • src/org/openstreetmap/josm/gui/mappaint/mapcss/Instruction.java
  • src/org/openstreetmap/josm/gui/mappaint/styleelement/RepeatImageElement.java
  • src/org/openstreetmap/josm/gui/mappaint/styleelement/StyleElement.java
  • src/org/openstreetmap/josm/gui/oauth/AuthorizationProcedure.java
  • src/org/openstreetmap/josm/gui/oauth/AuthorizationProcedureComboBox.java
  • src/org/openstreetmap/josm/gui/oauth/FullyAutomaticPropertiesPanel.java
  • src/org/openstreetmap/josm/gui/oauth/OsmPrivilegesPanel.java
  • src/org/openstreetmap/josm/gui/preferences/DefaultPreferenceSetting.java
  • src/org/openstreetmap/josm/gui/preferences/DefaultTabPreferenceSetting.java
  • src/org/openstreetmap/josm/gui/preferences/PreferenceDialog.java
  • src/org/openstreetmap/josm/gui/preferences/SourceProvider.java
  • src/org/openstreetmap/josm/gui/preferences/projection/AbstractProjectionChoice.java
  • src/org/openstreetmap/josm/gui/preferences/projection/CustomProjectionChoice.java
  • src/org/openstreetmap/josm/gui/preferences/projection/GaussKruegerProjectionChoice.java
  • src/org/openstreetmap/josm/gui/preferences/projection/LambertCC9ZonesProjectionChoice.java
  • src/org/openstreetmap/josm/gui/preferences/projection/LambertProjectionChoice.java
  • src/org/openstreetmap/josm/gui/preferences/projection/PuwgProjectionChoice.java
  • src/org/openstreetmap/josm/gui/preferences/projection/SwissGridProjectionChoice.java
  • src/org/openstreetmap/josm/gui/preferences/projection/UTMFranceDOMProjectionChoice.java
  • src/org/openstreetmap/josm/gui/preferences/projection/UTMProjectionChoice.java
  • src/org/openstreetmap/josm/gui/progress/AbstractProgressMonitor.java
  • src/org/openstreetmap/josm/gui/progress/CancelHandler.java
  • src/org/openstreetmap/josm/gui/progress/ChildProgress.java
  • src/org/openstreetmap/josm/gui/progress/PleaseWaitProgressMonitor.java
  • src/org/openstreetmap/josm/gui/progress/ProgressException.java
  • src/org/openstreetmap/josm/gui/progress/ProgressTaskId.java
  • src/org/openstreetmap/josm/gui/progress/ProgressTaskIds.java
  • src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetLabel.java
  • src/org/openstreetmap/josm/gui/tagging/presets/TaggingPresetMenu.java
  • src/org/openstreetmap/josm/gui/tagging/presets/items/Roles.java
  • src/org/openstreetmap/josm/gui/widgets/BoundingBoxSelectionPanel.java
  • src/org/openstreetmap/josm/gui/widgets/HistoryChangedListener.java
  • src/org/openstreetmap/josm/gui/widgets/ListPopupMenu.java
  • src/org/openstreetmap/josm/gui/widgets/OsmPrimitivesTableModel.java
  • src/org/openstreetmap/josm/gui/widgets/SearchTextResultListPanel.java
  • src/org/openstreetmap/josm/gui/widgets/SelectAllOnFocusGainedDecorator.java
  • src/org/openstreetmap/josm/gui/widgets/VerticallyScrollablePanel.java
  • src/org/openstreetmap/josm/io/ChangesetQuery.java
  • src/org/openstreetmap/josm/io/DiffResultProcessor.java
  • src/org/openstreetmap/josm/io/FileExporter.java
  • src/org/openstreetmap/josm/io/MissingOAuthAccessTokenException.java
  • src/org/openstreetmap/josm/io/OsmChangeImporter.java
  • src/org/openstreetmap/josm/io/OsmImporter.java
  • src/org/openstreetmap/josm/io/OsmServerReadPostprocessor.java
  • src/org/openstreetmap/josm/io/OsmServerUserInfoReader.java
  • src/org/openstreetmap/josm/io/OsmServerWritePostprocessor.java
  • src/org/openstreetmap/josm/io/StreamProgressUpdater.java
  • src/org/openstreetmap/josm/io/auth/AbstractCredentialsAgent.java
  • src/org/openstreetmap/josm/io/auth/CredentialsAgentException.java
  • src/org/openstreetmap/josm/io/imagery/ImageryReader.java

I will start fixing classes that I understand most, but wouldn't mind some help. Hopefully we can get this list down to zero!

Attachments (2)

TopLevelJavadocCheck.java (2.4 KB ) - added by bastiK 7 years ago.
TopLevelJavadocCheck.2.java (3.0 KB ) - added by bastiK 7 years ago.

Download all attachments as: .zip

Change History (78)

by bastiK, 7 years ago

Attachment: TopLevelJavadocCheck.java added

comment:1 by bastiK, 7 years ago

Attached is a Checkstyle check that finds files with missing top level Javadoc.

comment:2 by Don-vip, 7 years ago

See also Sonar.

comment:3 by bastiK, 7 years ago

In 12148/josm:

see #14794 - javadoc

comment:4 by bastiK, 7 years ago

Description: modified (diff)

comment:5 by michael2402, 7 years ago

Description: modified (diff)

I'll do src/org/openstreetmap/josm/data/osm/

comment:6 by michael2402, 7 years ago

In 12186/josm:

See #14794: Add javadoc for all gpx classes.

comment:7 by michael2402, 7 years ago

In 12189/josm:

See #14794: Javadoc for data.osm package

comment:8 by michael2402, 7 years ago

Description: modified (diff)

comment:9 by michael2402, 7 years ago

In 12190/josm:

See #14794: More javadoc for data.osm package

comment:10 by bastiK, 7 years ago

In 12259/josm:

see #14794 - javadoc

comment:11 by bastiK, 7 years ago

Description: modified (diff)

in reply to:  2 comment:12 by bastiK, 7 years ago

Replying to Don-vip:

See also Sonar.

Good to know! But let's start with something achievable.

Replying to michael2402:

I'll do src/org/openstreetmap/josm/data/osm/

Great!

comment:13 by Don-vip, 7 years ago

In 12284/josm:

see #14794 - javadoc

comment:14 by michael2402, 7 years ago

Description: modified (diff)

comment:15 by michael2402, 7 years ago

In 12300/josm:

See #14794: Add class javadoc for MenuItemSearchDialog

comment:16 by michael2402, 7 years ago

Description: modified (diff)

I found some more files. Your list does not include classes with empty javadoc comments (only /** */) or with only an @author tag.

comment:17 by bastiK, 7 years ago

Description: modified (diff)

by bastiK, 7 years ago

Attachment: TopLevelJavadocCheck.2.java added

in reply to:  16 comment:18 by bastiK, 7 years ago

Replying to michael2402:

I found some more files. Your list does not include classes with empty javadoc comments (only /** */) or with only an @author tag.

Right, I've updated the check.

comment:19 by michael2402, 7 years ago

Description: modified (diff)

comment:20 by michael2402, 7 years ago

In 12371/josm:

See #14794: Class comment for ChangesetInSelectionListModel

comment:21 by michael2402, 7 years ago

In 12372/josm:

See #14794: Document package gui.dialogs.changeset

comment:22 by michael2402, 7 years ago

Description: modified (diff)

comment:23 by michael2402, 7 years ago

In 12376/josm:

See #14794: Document package osm.visitor.paint

comment:24 by michael2402, 7 years ago

In 12377/josm:

See #14794: Document the layer.gpx package

comment:25 by michael2402, 7 years ago

Description: modified (diff)

comment:26 by michael2402, 7 years ago

In 12385/josm:

See #14794: Class comment for CachedProperty

comment:27 by michael2402, 7 years ago

In 12390/josm:

See #14794: Document data.validation package and subpackages.

comment:28 by michael2402, 7 years ago

In 12391/josm:

See #14794: Documentation for the gui package

comment:29 by michael2402, 7 years ago

Description: modified (diff)

comment:30 by bastiK, 7 years ago

In 12452/josm:

see #14794 - javadoc for the josm.gui.io package

comment:31 by bastiK, 7 years ago

In 12453/josm:

see #14794 - remaining javadoc for the josm/data/ packages

comment:32 by bastiK, 7 years ago

Description: modified (diff)

comment:33 by bastiK, 7 years ago

In 12460/josm:

see #14794 - javadoc

comment:34 by bastiK, 7 years ago

Description: modified (diff)

comment:35 by bastiK, 7 years ago

In 12470/josm:

see #14794 - javadoc

comment:36 by bastiK, 7 years ago

Description: modified (diff)

comment:37 by bastiK, 7 years ago

In 12546/josm:

see #14794 - javadoc

comment:38 by bastiK, 7 years ago

In 12581/josm:

see #14794 - javadoc

comment:39 by bastiK, 7 years ago

Description: modified (diff)

comment:40 by bastiK, 7 years ago

In 12582/josm:

see #14794 - add checkstyle check

comment:41 by bastiK, 7 years ago

In 12583/josm:

see #14794 - unnecessary classpath entry

comment:42 by bastiK, 7 years ago

In 12584/josm:

see #14794 - update checkstyle filters

comment:43 by Don-vip, 7 years ago

Keywords: javadoc added
Milestone: 17.08

comment:44 by Don-vip, 7 years ago

build-common.xml in plugins must be updated as well, as the core checkstyle configuration/filter files are reused.

comment:45 by Don-vip, 7 years ago

This also breaks Eclipse checkstyle, as we need now to create a custom plugin: http://eclipse-cs.sourceforge.net/#!/custom-checks

comment:46 by Don-vip, 7 years ago

In 12586/josm:

see #14794 - update to checkstyle 8.0 (to match the version used by Eclipse CS) + add new project to build custom Eclipse plugin (now required)

comment:47 by Don-vip, 7 years ago

plugins build-common.xml updated in [o33491].

in reply to:  47 comment:48 by bastiK, 7 years ago

Replying to Don-vip:

plugins build-common.xml updated in [o33491].

Thank, you beat me to it! (not that I would mind :) )

comment:49 by Don-vip, 7 years ago

Well I was already into it because of https://github.com/checkstyle/checkstyle/issues/4612 :) Also I have some experience in custom Checkstyle plugins as I maintain one at work ;)

comment:50 by Don-vip, 7 years ago

Damn, this is unexpected:

BUILD FAILED
C:\SVN\josm\plugins\build-common.xml:528: Unable to process files: [C:\SVN\josm\plugins\ColumbusCSV\src\org\openstreetmap\josm\plugins\columbusCSV\ColumbusCSVImporter.java, C:\SVN\josm\plugins\ColumbusCSV\src\org\openstreetmap\josm\plugins\columbusCSV\ColumbusCSVPlugin.java, C:\SVN\josm\plugins\ColumbusCSV\src\org\openstreetmap\josm\plugins\columbusCSV\ColumbusCSVPreferences.java, C:\SVN\josm\plugins\ColumbusCSV\src\org\openstreetmap\josm\plugins\columbusCSV\ColumbusCSVReader.java, C:\SVN\josm\plugins\ColumbusCSV\src\org\openstreetmap\josm\plugins\columbusCSV\ColumbusCSVUtils.java, C:\SVN\josm\plugins\ColumbusCSV\src\org\openstreetmap\josm\plugins\columbusCSV\WayPointHelper.java]
        at com.puppycrawl.tools.checkstyle.ant.CheckstyleAntTask.processFiles(CheckstyleAntTask.java:366)
...
Caused by: com.puppycrawl.tools.checkstyle.api.CheckstyleException: Exception was thrown while processing C:\SVN\josm\plugins\ColumbusCSV\src\org\openstreetmap\josm\plugins\columbusCSV\ColumbusCSVImporter.java
        at com.puppycrawl.tools.checkstyle.Checker.processFiles(Checker.java:295)
        at com.puppycrawl.tools.checkstyle.Checker.process(Checker.java:213)
        at com.puppycrawl.tools.checkstyle.ant.CheckstyleAntTask.processFiles(CheckstyleAntTask.java:360)
        ... 18 more
Caused by: java.lang.NullPointerException
        at org.openstreetmap.josm.TopLevelJavadocCheck.hasProperText(TopLevelJavadocCheck.java:74)
        at org.openstreetmap.josm.TopLevelJavadocCheck.hasJavadoc(TopLevelJavadocCheck.java:70)
        at org.openstreetmap.josm.TopLevelJavadocCheck.visitToken(TopLevelJavadocCheck.java:90)
        at com.puppycrawl.tools.checkstyle.TreeWalker.notifyVisit(TreeWalker.java:368)
        at com.puppycrawl.tools.checkstyle.TreeWalker.processIter(TreeWalker.java:505)
        at com.puppycrawl.tools.checkstyle.TreeWalker.walk(TreeWalker.java:310)
        at com.puppycrawl.tools.checkstyle.TreeWalker.processFiltered(TreeWalker.java:187)
        at com.puppycrawl.tools.checkstyle.api.AbstractFileSetCheck.process(AbstractFileSetCheck.java:78)
        at com.puppycrawl.tools.checkstyle.Checker.processFile(Checker.java:316)
        at com.puppycrawl.tools.checkstyle.Checker.processFiles(Checker.java:286)
        ... 20 more

It seems your check is not happy with this class:

/**
 * This program is free software: you can redistribute it and/or modify it under 
 * the terms of the GNU General Public License as published by the 
 * Free Software Foundation, either version 3 of the License, or 
 * (at your option) any later version. 
 * 
 * This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; 
 * without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. 
 * See the GNU General Public License for more details. 
 * 
 * You should have received a copy of the GNU General Public License along with this program. 
 * If not, see <http://www.gnu.org/licenses/>.
 */

package org.openstreetmap.josm.plugins.columbusCSV;

import static org.openstreetmap.josm.tools.I18n.tr;

import java.io.File;
import java.io.IOException;

import org.openstreetmap.josm.Main;
import org.openstreetmap.josm.actions.AutoScaleAction;
import org.openstreetmap.josm.actions.ExtensionFileFilter;
import org.openstreetmap.josm.data.gpx.GpxData;
import org.openstreetmap.josm.gui.layer.GpxLayer;
import org.openstreetmap.josm.gui.layer.markerlayer.MarkerLayer;
import org.openstreetmap.josm.gui.progress.NullProgressMonitor;
import org.openstreetmap.josm.gui.progress.ProgressMonitor;
import org.openstreetmap.josm.io.FileImporter;
import org.openstreetmap.josm.io.IllegalDataException;

/**
 * 
 * @author Oliver Wieland <oliver.wieland@online.de>
 * 
 */
public class ColumbusCSVImporter extends FileImporter {
...
}
Last edited 7 years ago by Don-vip (previous) (diff)

comment:51 by bastiK, 7 years ago

In 12587/josm:

see #14794 - fix NPE

in reply to:  50 comment:52 by bastiK, 7 years ago

Replying to Don-vip:

Damn, this is unexpected:
[...]

Just a small NPE-bug.

The ant output is not particularly helpful, but this gave a stacktrace:

$ java -classpath build2:tools/checkstyle/checkstyle-all.jar com.puppycrawl.tools.checkstyle.Main -c tools/checkstyle/josm_checks.xml ../plugins/ColumbusCSV/src/org/openstreetmap/josm/plugins/columbusCSV/ColumbusCSVImporter.java 
Last edited 7 years ago by bastiK (previous) (diff)

comment:53 by Don-vip, 7 years ago

I got it in debug mode (cd plugins/ColumbusCSV && and -d checkstyle).

comment:54 by Don-vip, 7 years ago

In 12597/josm:

see #14794 - checkstyle/javadoc

comment:55 by bastiK, 7 years ago

In 12615/josm:

see #14794 - javadoc

comment:56 by bastiK, 7 years ago

In 12646/josm:

see #14794 - javadoc

comment:57 by michael2402, 7 years ago

In 12657/josm:

See #14794: Add javadoc for gui/dialogs/relation package.

comment:58 by michael2402, 7 years ago

In 12658/josm:

See #14794: Add javadoc for gui/dialogs/relation/sort package.

comment:59 by michael2402, 7 years ago

In 12660/josm:

See #14794: Add javadoc for gui/conflict/tags package.

comment:60 by michael2402, 7 years ago

Resolution: fixed
Status: newclosed

In 12661/josm:

Fix #14794: Add javadoc for gui/conflict/pair package.

comment:61 by Don-vip, 7 years ago

Congrats! :)

in reply to:  61 comment:62 by michael2402, 7 years ago

Replying to Don-vip:

Congrats! :)

Just got annoyed by all those jenkins mails ;-)

comment:63 by bastiK, 7 years ago

Vincent, you are very strict with us, making the build fail when there are checkstyle violations ... ;)

comment:64 by Don-vip, 7 years ago

I dit it before you applied the new checkstyle rule ;)
There is currently a tiny tolerance: up to 1 Java warning is allowed :D

comment:65 by michael2402, 7 years ago

Hmm, then I should probably try to get the one out of Undo/Redo, so that others have a chance to get that tolerance (in hope that you don't just decrease the tolerance then :D)

comment:66 by Don-vip, 7 years ago

:)

comment:67 by Don-vip, 7 years ago

MultikeyActionsHandler and MultikeyShortcutAction have no javadoc. Yet Checkstyle is happy?

comment:68 by Don-vip, 7 years ago

In 12797/josm:

see #14794 - fix checkstyle filters - the whole tools package was ignored

comment:69 by Don-vip, 7 years ago

In 12798/josm:

see #14794 - checkstyle

comment:70 by Don-vip, 7 years ago

These classes need javadoc:

/org/openstreetmap/josm/tools/MultikeyActionsHandler.java
/org/openstreetmap/josm/tools/MultikeyShortcutAction.java
/org/openstreetmap/josm/tools/template_engine/CompoundTemplateEntry.java
/org/openstreetmap/josm/tools/template_engine/Condition.java
/org/openstreetmap/josm/tools/template_engine/ParseError.java
/org/openstreetmap/josm/tools/template_engine/SearchExpressionCondition.java
/org/openstreetmap/josm/tools/template_engine/StaticText.java
/org/openstreetmap/josm/tools/template_engine/TemplateEngineDataProvider.java

comment:71 by Don-vip, 7 years ago

In 12799/josm:

see #15229 - see #15182 - see #14794 - move Multi* GUI classes from tools to gui.util

comment:72 by Don-vip, 7 years ago

In 12802/josm:

see #14794 - checkstyle (unit tests)

comment:73 by Don-vip, 7 years ago

In 12807/josm:

see #15229 - see #15182 - see #14794 - checkstyle, unit tests

comment:74 by Don-vip, 7 years ago

I've increased the number of allowed checkstyle warnings to let us time to document the template engine classes.

comment:75 by bastiK, 6 years ago

In 13003/josm:

see #14794 - add missing top level javadoc; minor refactoring for Condition

comment:76 by taylor.smock, 6 weeks ago

See #23465: follow up to replace the custom check with a standardized check from checkstyle (MissingJavadocType).

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.