GP-0: Cleaning up certain types of javadoc errors

GPL/DMG/src/dmg/java/mobiledevices/dmg/ghidra/GDataConverterBE.java

@@ -13,9 +13,6 @@ package mobiledevices.dmg.ghidra;
 public class GDataConverterBE implements GDataConverter {
 	public static final GDataConverterBE INSTANCE = new GDataConverterBE();
 
-	/**
-	 * 
-	 */
 	private static final long serialVersionUID = 1L;
 
 	/**

GPL/DMG/src/dmg/java/mobiledevices/dmg/ghidra/GDataConverterLE.java

@@ -11,9 +11,7 @@ package mobiledevices.dmg.ghidra;
 
 public class GDataConverterLE implements GDataConverter {
 	public static GDataConverterLE INSTANCE = new GDataConverterLE();
-	/**
+
-	 * 
-	 */
 	private static final long serialVersionUID = 1L;
 
 	/**

Ghidra/Debug/Framework-TraceModeling/src/main/java/ghidra/trace/model/data/TraceBasedDataTypeManager.java

@@ -48,9 +48,6 @@ public interface TraceBasedDataTypeManager extends ProgramBasedDataTypeManager {
 	 */
 	@SuppressWarnings("unchecked")
 	default <T extends DataType> T resolveType(T dataType, DataTypeConflictHandler handler) {
-		/**
-		 * 
-		 */
 		return (T) resolve(dataType, handler);
 	}
 

Ghidra/Features/BSim/src/main/java/ghidra/features/bsim/query/facade/SFOverviewInfo.java

@@ -35,7 +35,7 @@ public class SFOverviewInfo {
 	 * Constructs an overview request with default parameters.
 	 * @param functions required--a set of functions (at least one) for which an overview will be 
 	 * 					computed.  All functions must be from the same program.
-	 * @throws IllegalArgumentException if <tt>functions</tt> is <tt>null</tt>/empty or functions
+	 * @throws IllegalArgumentException if {@code functions} is {@code null}/empty or functions
 	 * are from multiple programs.  
 	 */
 	public SFOverviewInfo(Set<FunctionSymbol> functions) {

Ghidra/Features/BSim/src/main/java/ghidra/features/bsim/query/facade/SFQueryInfo.java

@@ -48,7 +48,7 @@ public class SFQueryInfo {
 	 * Constructs a query request with default parameters.
 	 * @param functions required--a set of functions (at least one) for which similar functions
 	 *                  will searched.  All functions must be from the same program.
-	 * @throws IllegalArgumentException if <tt>functions</tt> is <tt>null</tt>/empty or functions
+	 * @throws IllegalArgumentException if {@code functions} is {@code null}/empty or functions
 	 * are from multiple programs.  
 	 */
 	public SFQueryInfo(Set<FunctionSymbol> functions) {

Ghidra/Features/BSim/src/main/java/ghidra/features/bsim/query/facade/SimilarFunctionQueryService.java

@@ -118,7 +118,7 @@ public class SimilarFunctionQueryService implements AutoCloseable {
 	}
 
 	/**
-	 * Query the given server with the parameters provider by <tt>queryInfo</tt>.
+	 * Query the given server with the parameters provider by {@code queryInfo}.
 	 * 
 	 * @param queryInfo a query info object containing the settings for the query
 	 * @param listener is the listener to be informed of the query status and incremental results 

Ghidra/Features/Base/ghidra_scripts/InstructionSearchScript.java

@@ -33,9 +33,6 @@ public class InstructionSearchScript extends GhidraScript {
 		testLoadAddresses();
 	}
 
-	/**
-	 * 
-	 */
 	@SuppressWarnings("unused")
 	private void testSearcher() {
 		AddressFactory addressFactory = currentProgram.getAddressFactory();
@@ -65,9 +62,6 @@ public class InstructionSearchScript extends GhidraScript {
 		}
 	}
 
-	/**
-	 * 
-	 */
 	@SuppressWarnings("unused")
 	private void testLoadString() {
 		InstructionSearchApi searcher = new InstructionSearchApi();
@@ -76,9 +70,6 @@ public class InstructionSearchScript extends GhidraScript {
 		searcher.loadInstructions(bytes, state.getTool());
 	}
 
-	/**
-	 * 
-	 */
 	private void testLoadAddresses() {
 		InstructionSearchApi searcher = new InstructionSearchApi();
 

Ghidra/Features/Base/src/main/java/ghidra/app/cmd/data/AbstractCreateStructureCmd.java

@@ -76,7 +76,7 @@ public abstract class AbstractCreateStructureCmd implements Command<Program> {
 
 	/**
 	 * Initializes the structure that is represented by the provided
-	 * <tt>structureInfo</tt> object.  This involves populating the new 
+	 * {@code structureInfo} object.  This involves populating the new 
 	 * structure with data and then returning the data type object that 
 	 * represents the newly created structure.
 	 * 

Ghidra/Features/Base/src/main/java/ghidra/app/context/DataLocationListContext.java

@@ -42,7 +42,7 @@ public interface DataLocationListContext {
 
 	/**
 	 * Returns a list of the locations of the current {@link Data} objects in the current action context that pass the given filter.
-	 * <P>
+	 * 
 	 * @param filter a filter to apply to the current context's Data list, <code>null</code>
 	 * implies all elements match.
 	 * @return  a list of the locations of the current {@link Data} objects in the current action context that pass the given filter.

Ghidra/Features/Base/src/main/java/ghidra/app/merge/datatypes/DataTypeMergeManager.java

@@ -2208,9 +2208,6 @@ public class DataTypeMergeManager implements MergeResolver {
 		}
 	}
 
-	/**
-	 * 
-	 */
 	private void processDataTypesAdded() throws CancelledException {
 		for (Long element : myDtAddedList) {
 			currentMonitor.checkCancelled();

Ghidra/Features/Base/src/main/java/ghidra/app/merge/datatypes/SourceArchiveMergePanel.java

@@ -85,9 +85,6 @@ class SourceArchiveMergePanel extends JPanel {
 		buttonGroup.add(originalRB);
 	}
 
-	/**
-	 * 
-	 */
 	int getSelectedOption() {
 		if (latestRB.isSelected()) {
 			return DataTypeMergeManager.OPTION_LATEST;

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/analysis/AnalysisTaskList.java

@@ -1,6 +1,5 @@
 /* ###
  * IP: GHIDRA
- * REVIEWED: YES
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -16,18 +15,15 @@
  */
 package ghidra.app.plugin.core.analysis;
 
+import java.util.*;
+import java.util.concurrent.CopyOnWriteArrayList;
+
 import ghidra.app.services.Analyzer;
 import ghidra.framework.options.Options;
 import ghidra.program.model.address.Address;
 import ghidra.program.model.address.AddressSetView;
 import ghidra.program.model.listing.Program;
 
-import java.util.*;
-import java.util.concurrent.CopyOnWriteArrayList;
-
-/**
- * 
- */
 public class AnalysisTaskList {
 	private List<AnalysisScheduler> tasks;
 	private AutoAnalysisManager analysisMgr;

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/analysis/FindReferencesTableModel.java

@@ -1,6 +1,5 @@
 /* ###
  * IP: GHIDRA
- * REVIEWED: YES
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -16,6 +15,9 @@
  */
 package ghidra.app.plugin.core.analysis;
 
+import java.util.List;
+
+import docking.widgets.table.TableColumnDescriptor;
 import ghidra.app.util.query.AlignedObjectBasedPreviewTableModel;
 import ghidra.framework.plugintool.PluginTool;
 import ghidra.framework.plugintool.ServiceProvider;
@@ -28,13 +30,6 @@ import ghidra.util.exception.CancelledException;
 import ghidra.util.table.field.*;
 import ghidra.util.task.TaskMonitor;
 
-import java.util.List;
-
-import docking.widgets.table.TableColumnDescriptor;
-
-/**
- * 
- */
 public class FindReferencesTableModel extends
 		AlignedObjectBasedPreviewTableModel<ReferenceAddressPair> {
 

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/bookmark/BookmarkPlugin.java

@@ -513,7 +513,7 @@ public class BookmarkPlugin extends ProgramPlugin implements PopupActionProvider
 
 	/**
 	 * Returns a list of actions to delete bookmarks that are in the code unit surrounding the given
-	 * address. The list of actions will not exceed <tt>maxActionsCount</tt>
+	 * address. The list of actions will not exceed {@code maxActionsCount}
 	 * 
 	 * @param primaryAddress The address required to find the containing code unit.
 	 * @param maxActionsCount The maximum number of actions to include in the returned list.
@@ -544,7 +544,7 @@ public class BookmarkPlugin extends ProgramPlugin implements PopupActionProvider
 	 * @param primaryAddress The address required to find the containing code unit.
 	 * @param type The bookmark type to retrieve.
 	 * @param navigator The BookmarkNavigator used to determine whether there are bookmarks inside
-	 *            the code unit containing the given <tt>primaryAddress</tt>.
+	 *            the code unit containing the given {@code primaryAddress}.
 	 * @return a list of actions to delete bookmarks that are in the code unit surrounding the given
 	 *         address <b>for the given <i>type</i> of bookmark</b>.
 	 */
@@ -579,8 +579,8 @@ public class BookmarkPlugin extends ProgramPlugin implements PopupActionProvider
 	}
 
 	/**
-	 * Adds the actions in <tt>newActionList</tt> to <tt>actionList</tt> while the size of
+	 * Adds the actions in {@code newActionList} to {@code actionList} while the size of
-	 * <tt>actionList</tt> is less than the given {@link #MAX_DELETE_ACTIONS}.
+	 * {@code actionList} is less than the given {@link #MAX_DELETE_ACTIONS}.
 	 * 
 	 * @param actionList The list to add to
 	 * @param newActionList The list containing items to add

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/compositeeditor/CompositeEditorPanel.java

@@ -1449,7 +1449,7 @@ public abstract class CompositeEditorPanel extends JPanel
 	 * list of traversal components.  Once one of the radio buttons is focused, the up and down 
 	 * arrow keys can be used to navigate the radio buttons.  With this traversal policy, pressing 
 	 * Tab when on these buttons will move to the next traversal component.
-	 * <P>
+	 *
 	 * @see #getFocusComponents()
 	 */
 	private class CompFocusTraversalPolicy extends FocusTraversalPolicy {

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/data/CreateStructureDialog.java

@@ -415,7 +415,7 @@ public class CreateStructureDialog extends ReusableDialogComponentProvider {
 	/**
 	 * Shows a dialog that allows the user to create a new structure.
 	 * <p>
-	 * This method expects that <tt>program</tt> and <tt>structure</tt> be
+	 * This method expects that {@code program} and {@code structure} be
 	 * non-null.
 	 *
 	 * @param  program The current program which will be used to obtain current

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/disassembler/DisassembledViewPlugin.java

@@ -543,7 +543,7 @@ public class DisassembledViewPlugin extends ProgramPlugin implements DomainObjec
 		/**
 		 * Constructs a new info instance based upon the given address.
 		 * <p>
-		 * Note: A NullPointerException will be logged if <tt> address</tt> is
+		 * Note: A NullPointerException will be logged if {@code address} is
 		 *       null.
 		 * 
 		 * @param  address The address that this info is based upon.

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/instructionsearch/model/MaskSettings.java

@@ -37,9 +37,6 @@ public class MaskSettings {
 		this.maskAddresses = maskAddresses;	
 	}
 
-	/**
-	 * 
-	 */
 	public void clear() {
 		maskOperands = false;
 		maskScalars = false;

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/instructionsearch/ui/ControlPanelWidget.java

@@ -29,9 +29,6 @@ import javax.swing.border.*;
  */
 public abstract class ControlPanelWidget extends JPanel {
 
-	/**
-	 * 
-	 */
 	public ControlPanelWidget( String title) {
 
 		this.setLayout(new FlowLayout());

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/instructionsearch/ui/SearchDirectionWidget.java

@@ -59,9 +59,6 @@ public class SearchDirectionWidget extends ControlPanelWidget {
 	 * PROTECTED METHODS
 	 ********************************************************************************************/
 
-	/**
-	 * 
-	 */
 	@Override
 	protected JPanel createContent() {
 

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/navigation/locationreferences/LocationDescriptor.java

@@ -262,8 +262,8 @@ public abstract class LocationDescriptor {
 	}
 
 	/**
-	 * Returns a generic {@link ProgramLocation} based upon the <tt>program</tt> and
+	 * Returns a generic {@link ProgramLocation} based upon the {@code program} and
-	 * <tt>homeAddress</tt> of this <tt>LocationDescriptor</tt>.  Subclasses should override this
+	 * {@code homeAddress} of this {@code LocationDescriptor}.  Subclasses should override this
 	 * method to return more specific addresses.
 	 *
 	 * @return a generic ProgramLocation.
@@ -314,7 +314,7 @@ public abstract class LocationDescriptor {
 	 * @param obj The object associated with the text being rendered (e.g., CodeUnit).
 	 * @param fieldFactoryClass The class that created the field being rendered.
 	 * @param highlightColor The color to use for highlighting.
-	 * @return An array of highlights to render for the given <tt>text</tt>
+	 * @return An array of highlights to render for the given {@code text}
 	 */
 	abstract Highlight[] getHighlights(String text, Object obj,
 			Class<? extends FieldFactory> fieldFactoryClass, Color highlightColor);

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/navigation/locationreferences/LocationReference.java

@@ -98,7 +98,7 @@ public class LocationReference implements Comparable<LocationReference> {
 	/**
 	 * Returns the address where the item described by this object is used.  For example, for
 	 * data types, the address is where a data type is applied; for references, this value is the
-	 * <tt>from</tt> address.
+	 * {@code from} address.
 	 *
 	 * @return  the address where the item described by this object is used.
 	 */

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/navigation/locationreferences/LocationReferencesService.java

@@ -38,7 +38,7 @@ public interface LocationReferencesService {
 	 * location.
 	 * @param location The location for which to show references.
 	 * @param navigatable The navigatable in which the references should be shown
-	 * @throws NullPointerException if <tt>location</tt> is null.
+	 * @throws NullPointerException if {@code location} is null.
 	 */
 	public void showReferencesToLocation(ProgramLocation location, Navigatable navigatable);
 

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/navigation/locationreferences/ReferenceUtils.java

@@ -173,7 +173,7 @@ public final class ReferenceUtils {
 	 *
 	 * @param accumulator the results storage.
 	 * @param dataType The datatype for which to find references.
-	 * @param fieldName optional field name for which to search; the <tt>dataType</tt> must be a
+	 * @param fieldName optional field name for which to search; the {@code dataType} must be a
 	 * {@link Composite} to search for a field.
 	 * @param program The program from within which to find references.
 	 * @param monitor A task monitor to be updated as data is searched; if this is null, then a
@@ -198,7 +198,7 @@ public final class ReferenceUtils {
 	 *
 	 * @param accumulator the results storage.
 	 * @param dataType The datatype for which to find references.
-	 * @param fieldName optional field name for which to search; the <tt>dataType</tt> must be a
+	 * @param fieldName optional field name for which to search; the {@code dataType} must be a
 	 * {@link Composite} to search for a field.
 	 * @param program The program from within which to find references.
 	 * @param discoverTypes if true, the {@link DataTypeReferenceFinder} service will be used to

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/navigation/locationreferences/VariableTypeLocationDescriptor.java

@@ -61,7 +61,7 @@ class VariableTypeLocationDescriptor extends DataTypeLocationDescriptor {
 	/**
 	 * Overridden to catch special cases when processing variable locations.  This method will 
 	 * call to the {@link super#getHighlights(String, Object, Class, Color)} 
-	 *  <tt>super</tt> implementation if no variable highlights are found.
+	 * {@code super} implementation if no variable highlights are found.
 	 * 
 	 * @see DataTypeLocationDescriptor#getHighlights(String, Object, Class, Color)
 	 */

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/programtree/DnDTreeCellRenderer.java

@@ -90,7 +90,7 @@ class DnDTreeCellRenderer extends DefaultTreeCellRenderer {
 	/**
 	 * Enables and disables the rendering of HTML content in this renderer.  If enabled, this
 	 * renderer will interpret HTML content when the text this renderer is showing begins with
-	 * <tt>&lt;html&gt;</tt>
+	 * {@code <html>}
 	 *
 	 * @param enable true to enable HTML rendering; false to disable it
 	 */

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/references/ExternalReferencesProvider.java

@@ -49,7 +49,6 @@ import resources.Icons;
 
 /**
  * ComponentProvider that displays a table of External Programs.
- * <p>
  */
 public class ExternalReferencesProvider extends ComponentProviderAdapter {
 	private static Icon ADD_ICON = Icons.ADD_ICON;

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/stackeditor/StackEditorModel.java

@@ -446,9 +446,6 @@ public class StackEditorModel extends CompositeEditorModel {
 		setRelOffsetSelection(offsetSelection);
 	}
 
-	/**
-	 * 
-	 */
 	private OffsetPairs getRelOffsetSelection() {
 		OffsetPairs offsets = new OffsetPairs();
 		int num = selection.getNumRanges();
@@ -469,9 +466,6 @@ public class StackEditorModel extends CompositeEditorModel {
 		return offsets;
 	}
 
-	/**
-	 * 
-	 */
 	private void setRelOffsetSelection(OffsetPairs offsets) {
 		FieldSelection newSelection = new FieldSelection();
 		int num = offsets.getNumPairs();

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/string/StringTableModel.java

@@ -33,7 +33,6 @@ import ghidra.util.task.TaskMonitor;
 
 /**
  * Table model for the Search -> For Strings... result dialog.
- * <p>
  */
 public class StringTableModel extends AddressBasedTableModel<FoundString> {
 

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/strings/ViewStringsPlugin.java

@@ -47,8 +47,6 @@ import resources.ResourceManager;
 /**
  * Plugin that provides the "Defined Strings" table, where all the currently defined
  * string data in the program is listed.
- * <p>
- *
  */
 //@formatter:off
 @PluginInfo(

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/symboltree/nodes/OrganizationNode.java

@@ -94,12 +94,12 @@ public class OrganizationNode extends SymbolTreeNode {
 	 *  abcc
 	 *  g
 	 * </pre>
-	 * <p>
+	 *
 	 * @param list list of child nodes of to breakup into smaller groups
 	 * @param maxGroupSize the max number of nodes to allow before trying to organize into
 	 * smaller groups
 	 * @param monitor the TaskMonitor to be checked for canceling this operation
-	 * @return the given <tt>list</tt> sub-grouped as outlined above
+	 * @return the given {@code list} sub-grouped as outlined above
 	 * @throws CancelledException if the operation is cancelled
 	 */
 	public static List<GTreeNode> organize(List<GTreeNode> list, int maxGroupSize,

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/terminal/vt/VtHandler.java

@@ -88,7 +88,7 @@ import ghidra.util.Msg;
  * <p>
  * For example, the Cursor Position sequence is documented as:
  * <p>
- * <tt>CSI <em>n</em> ; <em>m</em> H</tt>
+ * {@code CSI <em>n</em> ; <em>m</em> H}
  * <p>
  * Supposing {@code n} is 13 and {@code m} is 40, this sequence would be encoded as the string
  * {@code "\033[13;40H"}. The parser will handle decoding the CSI, parameters, and final byte
@@ -682,7 +682,7 @@ public interface VtHandler {
 	}
 
 	/**
-	 * Parse a sequence of integers in the form <tt><em>n</em> ; <em>m</em> ;</tt> ....
+	 * Parse a sequence of integers in the form {@code <em>n</em> ; <em>m</em> ;} ....
 	 * 
 	 * <p>
 	 * This is designed to replace the {@link String#split(String)} and

Ghidra/Features/Base/src/main/java/ghidra/app/plugin/match/SubroutineMatch.java

@@ -31,9 +31,7 @@ public class SubroutineMatch {
 	private Address[] progAAddrs;
 	private Address[] progBAddrs;
 	private String reason;
-	/**
+
-	 * 
-	 */
 	public SubroutineMatch(String reason) {
 		progAAddrs = new Address[0];
 		progBAddrs = new Address[0];

Ghidra/Features/Base/src/main/java/ghidra/app/script/GhidraScript.java

@@ -1515,7 +1515,6 @@ public abstract class GhidraScript extends FlatProgramAPI {
 	 * 		<li>In the headless environment this method will set the {@link #currentSelection}
 	 * 			variable to the given value and update the GhidraState's selection variable.</li>
 	 * </ol>
-	 * <p>
 	 * 
 	 * @param addressSet the set of addresses to include in the selection.  If this value is null,
 	 * the current selection will be cleared and the variables set to null.
@@ -1558,7 +1557,6 @@ public abstract class GhidraScript extends FlatProgramAPI {
 	 * 		<li>In the headless environment this method will set the {@link #currentHighlight}
 	 * 			variable to	the given value and update the GhidraState's highlight variable.</li>
 	 * </ol>
-	 * <p>
 	 *
 	 * @param addressSet the set of addresses to include in the highlight.  If this value is null,
 	 * the current highlight will be cleared and the variables set to null.
@@ -2986,7 +2984,6 @@ public abstract class GhidraScript extends FlatProgramAPI {
 	 * <p>
 	 * Note that in both headless and GUI modes, you may specify "PI" or "E" and get the
 	 * corresponding floating point value to 15 decimal places.
-	 * <p>
 	 *
 	 * @param title the title of the dialog (in GUI mode) or the first part of the variable name
 	 * 			(in headless mode or when using .properties file)

Ghidra/Features/Base/src/main/java/ghidra/app/services/MarkerService.java

@@ -37,7 +37,6 @@ import ghidra.program.model.listing.Program;
  * Navigation Margin. The higher the priority, the more likely the marker/bar will be displayed on
  * the top. Areas will always be lower than marker priorities.
  * </p>
- * <a name="usage"></a> <u>Recommended Usage</u><br>
  * <u>Recommended Usage</u><br>
  * The service used to work independently of {@link Program}s. In order to work effectively this
  * service has been changed to associate created markers with individual programs. Thus, it is up to

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/BinaryReader.java

@@ -80,7 +80,6 @@ public class BinaryReader {
 
 	/**
 	 * Reads and returns an object from the current position in the specified input stream.
-	 * <p>
 	 * 
 	 * @param <T> the type of object that will be returned
 	 */
@@ -579,7 +578,6 @@ public class BinaryReader {
 	 * advancing the current index by the length of the string that was found.
 	 * <p>
 	 * Note: this method no longer trims() the returned String.
-	 * <p>
 	 * 
 	 * @return the US-ASCII string at the current index
 	 * @exception IOException if an I/O error occurs
@@ -596,7 +594,7 @@ public class BinaryReader {
 	 * a string from a fixed length field that is padded with trailing null chars)
 	 * <p>
 	 * Note: this method no longer trims() the returned String.
-	 * <p>
+	 * 
 	 * @param length number of bytes to read
 	 * @return the US-ASCII string at the current index
 	 */
@@ -607,7 +605,6 @@ public class BinaryReader {
 	/**
 	 * Reads a null-terminated UTF-16 Unicode string at the current index, 
 	 * advancing the current index by the length of the string that was found.
-	 * <p>
 	 * 
 	 * @return UTF-16 string at the current index
 	 * @exception IOException if an I/O error occurs
@@ -619,7 +616,6 @@ public class BinaryReader {
 	/**
 	 * Reads a fixed length UTF-16 Unicode string at the current index,
 	 * advancing the current index by the length of the string that was found.
-	 * <p>
 	 *
 	 * @param charCount number of UTF-16 characters to read (not bytes)
 	 * @return the UTF-16 Unicode string at the current index
@@ -632,7 +628,6 @@ public class BinaryReader {
 	/**
 	 * Reads a null-terminated UTF-8 string at the current index, 
 	 * advancing the current index by the length of the string that was found.
-	 * <p>
 	 * 
 	 * @return UTF-8 string at the current index
 	 * @exception IOException if an I/O error occurs
@@ -644,7 +639,6 @@ public class BinaryReader {
 	/**
 	 * Reads a fixed length UTF-8 string the current index,
 	 * advancing the current index by the length of the string that was found.
-	 * <p>
 	 *
 	 * @param length number of bytes to read
 	 * @return the UTF-8 string at the current index
@@ -658,7 +652,7 @@ public class BinaryReader {
 	 * Reads a null terminated string starting at the current index, 
 	 * using a specific {@link Charset}, advancing the current index by the length of 
 	 * the string that was found.
-	 * <p>
+	 * 
 	 * @param charset {@link Charset}, see {@link StandardCharsets}
 	 * @param charLen number of bytes in each character
 	 * @return the string
@@ -679,7 +673,7 @@ public class BinaryReader {
 	 * <p>
 	 * Trailing null terminator characters will be removed.  (suitable for reading
 	 * a string from a fixed length field that is padded with trailing null chars)
-	 * <p>
+	 * 
 	 * @param charCount the number of charLen character elements to read
 	 * @param charset {@link Charset}, see {@link StandardCharsets}
 	 * @param charLen number of bytes in each character
@@ -851,7 +845,6 @@ public class BinaryReader {
 	 * the first null character.
 	 * <p>
 	 * Note: this method no longer trims() the returned String.
-	 * <p>
 	 * 
 	 * @param index starting position of the string
 	 * @return US-ASCII string, excluding the trailing null terminator character
@@ -868,7 +861,7 @@ public class BinaryReader {
 	 * a string from a fixed length field that is padded with trailing null chars)
 	 * <p>
 	 * Note: this method no longer trims() the returned String.
-	 * <p>
+	 * 
 	 * @param index where the string begins
 	 * @param length number of bytes to read
 	 * @return the US-ASCII string
@@ -883,7 +876,7 @@ public class BinaryReader {
 	 * the pre-specified {@link #setLittleEndian(boolean) endianness}.
 	 * <p>
 	 * The end of the string is denoted by a two-byte (ie. short) <code>null</code> character.
-	 * <p>
+	 * 
 	 * @param index where the UTF-16 Unicode string begins
 	 * @return the UTF-16 Unicode string
 	 * @exception IOException if an I/O error occurs
@@ -899,7 +892,7 @@ public class BinaryReader {
 	 * <p>
 	 * Trailing null terminator characters will be removed.  (suitable for reading
 	 * a string from a fixed length field that is padded with trailing null chars)
-	 * <p>
+	 * 
 	 * @param index the index where the UTF-16 Unicode string begins
 	 * @param charCount the number of UTF-16 character elements to read.
 	 * @return the UTF-16 Unicode string
@@ -911,7 +904,7 @@ public class BinaryReader {
 
 	/**
 	 * Reads a null-terminated UTF-8 string starting at <code>index</code>.
-	 * <p>
+	 * 
 	 * @param index where the UTF-8 string begins
 	 * @return the string
 	 * @exception IOException if an I/O error occurs
@@ -926,7 +919,7 @@ public class BinaryReader {
 	 * <p>
 	 * Trailing null terminator characters will be removed.  (suitable for reading
 	 * a string from a fixed length field that is padded with trailing null chars)
-	 * <p>
+	 * 
 	 * @param index the index where the UTF-8 string begins
 	 * @param length the number of bytes to read
 	 * @return the string
@@ -942,7 +935,7 @@ public class BinaryReader {
 	 * <p>
 	 * Trailing null terminator characters will be removed.  (suitable for reading
 	 * a string from a fixed length field that is padded with trailing null chars)
-	 * <p>
+	 * 
 	 * @param index the index where the string begins
 	 * @param charCount the number of charLen character elements to read
 	 * @param charset {@link Charset}, see {@link StandardCharsets}
@@ -965,7 +958,7 @@ public class BinaryReader {
 	/**
 	 * Reads a null-terminated string starting at <code>index</code>, using a specific
 	 * {@link Charset}.
-	 * <p>
+	 * 
 	 * @param index where the string begins
 	 * @param charset {@link Charset}, see {@link StandardCharsets}
 	 * @param charLen number of bytes in each character

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/ByteProviderInputStream.java

@@ -30,14 +30,13 @@ public class ByteProviderInputStream extends InputStream {
 	 * An {@link InputStream} that reads from a {@link ByteProvider}, and <b>DOES</b>
 	 * {@link ByteProvider#close() close()} the underlying ByteProvider when
 	 * closed itself.
-	 * <p> 
 	 */
 	public static class ClosingInputStream extends ByteProviderInputStream {
 		/**
 		 * Creates an {@link InputStream} that reads from a {@link ByteProvider}, that
 		 * <b>DOES</b> {@link ByteProvider#close() close()} the underlying ByteProvider when
 		 * closed itself.
-		 * <p>
+		 * 
 		 * @param provider the {@link ByteProvider} to read from (and close)
 		 */
 		public ClosingInputStream(ByteProvider provider) {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/ByteProviderPaddedInputStream.java

@@ -39,7 +39,7 @@ public class ByteProviderPaddedInputStream extends InputStream {
 	 * The source ByteProvider is not closed when this stream is closed.
 	 * <p>
 	 * The total number of bytes that can be read from this instance will be length + padCount.
-	 * <p>
+	 * 
 	 * @param provider the {@link ByteProvider} to wrap.
 	 * @param startOffset the starting offset in the ByteProvider.
 	 * @param length the number of bytes from the {@link ByteProvider} to allow to be read by this InputStream.

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/ByteProviderWrapper.java

@@ -33,7 +33,6 @@ public class ByteProviderWrapper implements ByteProvider {
 	/**
 	 * Creates a wrapper around a {@link ByteProvider} that contains the same bytes as the specified
 	 * provider, but with a new {@link FSRL} identity.
-	 * <p>
 	 * 
 	 * @param provider {@link ByteProvider} to wrap
 	 * @param fsrl {@link FSRL} identity for the instance

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/FileByteProvider.java

@@ -135,7 +135,6 @@ public class FileByteProvider implements MutableByteProvider {
 	 * Read bytes at the specified index into the given byte array.
 	 * <p>
 	 * See {@link InputStream#read(byte[], int, int)}.
-	 * <p>
 	 * 
 	 * @param index file offset to start reading
 	 * @param buffer byte array that will receive the bytes

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/GhidraRandomAccessFile.java

@@ -55,7 +55,6 @@ public class GhidraRandomAccessFile implements AutoCloseable {
 	 * <p>
 	 * This implementation relies on java.net.RandomAccessFile,
 	 * but adds buffering to limit the amount.
-	 * <p>
 	 *
 	 * <a id="mode"></a><p> The <code>mode</code> argument specifies the access mode
 	 * in which the file is to be opened.  The permitted values and their

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/InputStreamByteProvider.java

@@ -24,7 +24,6 @@ import java.io.*;
  * <p>
  * In other words, this {@link ByteProvider} can only be used to read data at ever increasing 
  * offsets.
- * <p>
  */
 public class InputStreamByteProvider implements ByteProvider {
 	private InputStream inputStream;

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/LEB128Info.java

@@ -27,7 +27,7 @@ public class LEB128Info {
 	/**
 	 * Reads an unsigned LEB128 value from the BinaryReader and returns a {@link LEB128Info} instance
 	 * that contains the value along with size and position metadata.
-	 * <p>
+	 * 
 	 * @param reader {@link BinaryReader} to read bytes from
 	 * @return a {@link LEB128Info} instance with the read LEB128 value with metadata
 	 * @throws IOException if an I/O error occurs or value is outside the range of a java
@@ -40,7 +40,7 @@ public class LEB128Info {
 	/**
 	 * Reads an signed LEB128 value from the BinaryReader and returns a {@link LEB128Info} instance
 	 * that contains the value along with size and position metadata.
-	 * <p>
+	 * 
 	 * @param reader {@link BinaryReader} to read bytes from
 	 * @return a {@link LEB128Info} instance with the read LEB128 value with metadata
 	 * @throws IOException if an I/O error occurs or value is outside the range of a java
@@ -125,7 +125,7 @@ public class LEB128Info {
 	/**
 	 * Reads a LEB128 value from the BinaryReader and returns a {@link LEB128Info} instance
 	 * that contains the value along with size and position metadata.
-	 * <p>
+	 * 
 	 * @param reader {@link BinaryReader} to read bytes from
 	 * @param isSigned true if the value is signed
 	 * @return a {@link LEB128Info} instance with the read LEB128 value with metadata

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/MemoryByteProvider.java

@@ -104,7 +104,6 @@ public class MemoryByteProvider implements ByteProvider {
 	 * Constructs a new {@link MemoryByteProvider} for a specific {@link AddressSpace}.  Bytes 
 	 * will be provided relative to the minimum address (typically 0) in the space, and ranges 
 	 * to the highest address in the same address space currently found in the memory map.
-	 * <p>
 	 * 
 	 * 
 	 * @param memory the {@link Memory}

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/RandomAccessByteProvider.java

@@ -30,7 +30,7 @@ import ghidra.util.Msg;
  * {@link ArrayIndexOutOfBoundsException}s.
  * <p>
  * See {@link SynchronizedByteProvider} as a solution.
- * <p>
+ * 
  * @deprecated See {@link FileByteProvider} as replacement ByteProvider.
  */
 @Deprecated(since = "10.1", forRemoval = true)

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/RangeMappedByteProvider.java

@@ -177,7 +177,6 @@ public class RangeMappedByteProvider implements ByteProvider {
 	 * Read bytes at the specified index into the given byte array.
 	 * <p>
 	 * See {@link InputStream#read(byte[], int, int)}.
-	 * <p>
 	 * 
 	 * @param index file offset to start reading
 	 * @param buffer byte array that will receive the bytes

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/UnlimitedByteProviderWrapper.java

@@ -30,7 +30,6 @@ public class UnlimitedByteProviderWrapper extends ByteProviderWrapper {
 	/**
 	 * Creates a wrapper around a {@link ByteProvider} that contains the same bytes as the specified
 	 * provider.
-	 * <p>
 	 * 
 	 * @param provider {@link ByteProvider} to wrap
 	 * @throws IOException if error
@@ -42,7 +41,6 @@ public class UnlimitedByteProviderWrapper extends ByteProviderWrapper {
 	/**
 	 * Creates a wrapper around a {@link ByteProvider} that contains the same bytes as the specified
 	 * provider, but with a new {@link FSRL} identity.
-	 * <p>
 	 * 
 	 * @param provider {@link ByteProvider} to wrap
 	 * @param fsrl {@link FSRL} identity for the instance

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/coff/archive/CoffArchiveHeader.java

@@ -41,7 +41,7 @@ public final class CoffArchiveHeader implements StructConverter {
 	/**
 	 * Returns true if the data contained in the {@link ByteProvider provider} contains
 	 * a COFF Archive file.
-	 * <p>
+	 * 
 	 * @param provider
 	 * @return
 	 * @throws IOException
@@ -56,7 +56,7 @@ public final class CoffArchiveHeader implements StructConverter {
 	 * <p>
 	 * Returns a {@link CoffArchiveHeader} that has a list of the 
 	 * {@link CoffArchiveMemberHeader members} in the archive.
-	 * <p>
+	 * 
 	 * @param provider
 	 * @param monitor
 	 * @return

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/coff/archive/CoffArchiveMemberHeader.java

@@ -20,7 +20,6 @@ import java.io.IOException;
 import ghidra.app.util.bin.*;
 import ghidra.program.model.data.*;
 import ghidra.util.Msg;
-import ghidra.util.StringUtilities;
 import ghidra.util.exception.DuplicateNameException;
 
 public class CoffArchiveMemberHeader implements StructConverter {
@@ -58,7 +57,7 @@ public class CoffArchiveMemberHeader implements StructConverter {
 	 * <p>
 	 * The archive member's name is fixed up using the specified {@link LongNamesMember longNames}
 	 * object.
-	 * <p>
+	 * 
 	 * @param reader stream from which to read the COFF archive member header from
 	 * @param longNames optional, string table with long file names (only present in some 
 	 * COFF ar formats)

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DIEAggregate.java

@@ -70,7 +70,7 @@ public class DIEAggregate {
 	 * <p>
 	 * DW_AT_abstract_origin and DW_AT_specification attributes are followed to find the previous
 	 * {@link DebugInfoEntry} instances.
-	 * <p>
+	 * 
 	 * @param die starting DIE record
 	 * @return new {@link DIEAggregate} made up of the starting DIE and all DIEs that it points
 	 * to via abstract_origin and spec attributes.
@@ -131,7 +131,7 @@ public class DIEAggregate {
 	 * <p>
 	 * Mainly useful early in the {@link DWARFCompilationUnit}'s bootstrapping process
 	 * when it needs to read values from DIEs.
-	 * <p>
+	 * 
 	 * @param die {@link DebugInfoEntry}
 	 * @return {@link DIEAggregate} containing a single DIE
 	 */
@@ -318,7 +318,6 @@ public class DIEAggregate {
 	 * <p>
 	 * Attributes are searched for in each fragment in this aggregate, starting with the
 	 * 'head' fragment, progressing toward the 'decl' fragment.
-	 * <p>
 	 *
 	 * @param attribute See {@link DWARFAttribute}
 	 * @param clazz must be derived from {@link DWARFAttributeValue}
@@ -337,7 +336,6 @@ public class DIEAggregate {
 	 * <p>
 	 * Attributes are searched for in each fragment in this aggregate, starting with the
 	 * 'head' fragment, progressing toward the 'decl' fragment.
-	 * <p>
 	 *
 	 * @param attribute See {@link DWARFAttribute}
 	 * @return DWARFAttributeValue, or null if not found
@@ -362,7 +360,7 @@ public class DIEAggregate {
 	/**
 	 * Returns the boolean value of the requested attribute, or -defaultValue- if
 	 * the attribute is missing or not the correct type.
-	 * <p>
+	 * 
 	 * @param attribute {@link DWARFAttribute} id
 	 * @param defaultValue value to return if attribute is not present
 	 * @return boolean value, or the defaultValue if attribute is not present
@@ -375,7 +373,7 @@ public class DIEAggregate {
 	/**
 	 * Returns the string value of the requested attribute, or -defaultValue- if
 	 * the attribute is missing or not the correct type.
-	 * <p>
+	 * 
 	 * @param attribute {@link DWARFAttribute} id
 	 * @param defaultValue value to return if attribute is not present
 	 * @return String value, or the defaultValue if attribute is not present
@@ -391,7 +389,7 @@ public class DIEAggregate {
 	/**
 	 * Returns the string value of the {@link DWARFAttribute#DW_AT_name dw_at_name} attribute,
 	 * or null if it is missing.
-	 * <p>
+	 * 
 	 * @return name of this DIE aggregate, or null if missing
 	 */
 	public String getName() {
@@ -435,7 +433,7 @@ public class DIEAggregate {
 	/**
 	 * Returns the {@link DIEAggregate diea} instance pointed to by the requested attribute,
 	 * or null if the attribute does not exist.
-	 * <p>
+	 * 
 	 * @param attribute {@link DWARFAttribute} id
 	 * @return {@link DIEAggregate}, or the null if attribute is not present
 	 */
@@ -484,7 +482,7 @@ public class DIEAggregate {
 
 	/**
 	 * Return a list of children that are of a specific DWARF type.
-	 * <p>
+	 * 
 	 * @param childTag see {@link DWARFTag DWARFTag DW_TAG_* values}
 	 * @return List of children DIEs that match the specified tag
 	 */
@@ -527,7 +525,7 @@ public class DIEAggregate {
 	/**
 	 * Returns the signed integer value of the requested attribute after resolving
 	 * any DWARF expression opcodes.
-	 * <p>
+	 * 
 	 * @param attribute {@link DWARFAttribute} id
 	 * @param defaultValue value to return if attribute is not present
 	 * @return int value, or the defaultValue if attribute is not present
@@ -560,7 +558,7 @@ public class DIEAggregate {
 	/**
 	 * Returns the unsigned integer value of the requested attribute after resolving
 	 * any DWARF expression opcodes.
-	 * <p>
+	 * 
 	 * @param attribute {@link DWARFAttribute} id
 	 * @param defaultValue value to return if attribute is not present
 	 * @return unsigned long value, or the defaultValue if attribute is not present
@@ -673,7 +671,7 @@ public class DIEAggregate {
 	/**
 	 * Returns true if this DIE has a DW_AT_declaration attribute and
 	 * does NOT have a matching inbound DW_AT_specification reference.
-	 * <p>
+	 * 
 	 * @return boolean true if this DIE has a DW_AT_declaration attribute and
 	 * does NOT have a matching inbound DW_AT_specification reference
 	 */

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DWARFCompilationUnit.java

@@ -34,7 +34,6 @@ import ghidra.util.task.TaskMonitor;
  * The first DIE record must be a DW_TAG_compile_unit.
  * <p>
  * DIE records are identified by their byte offset in the .debug_info section.
- * <p>
  */
 public class DWARFCompilationUnit extends DWARFUnitHeader {
 	/**

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DWARFDataTypeConflictHandler.java

@@ -36,7 +36,6 @@ import ghidra.util.SystemUtilities;
  * <p>
  * When a candidate data type is matched with an existing data type, this
  * conflict handler will specify that the new data type is:
- * <p>
  * <ul>
  * <li>discarded and replaced by the existing data type
  * ({@link ConflictResult#USE_EXISTING})</li>
@@ -45,7 +44,6 @@ import ghidra.util.SystemUtilities;
  * </ul>
  * or the candidate data type was <b>NOT</b> matched with an existing data type,
  * and the new data type is:
- * <p>
  * <ul>
  * <li>kept, but renamed with a .conflictNNNN suffix to make it unique
  * ({@link ConflictResult#RENAME_AND_ADD})</li>
@@ -147,7 +145,7 @@ public class DWARFDataTypeConflictHandler extends DataTypeConflictHandler {
 	 * <p> Each defined component in the candidate partial structure must be present
 	 * in the 'full' structure and must be equiv. <p> The order and sparseness of
 	 * the candidate partial structure is not important, only that all of its
-	 * defined components are present in the full structure. <p>
+	 * defined components are present in the full structure.
 	 */
 	private boolean isStructurePart(Structure full, Structure part, Set<Long> visitedDataTypes) {
 		// Both structures should be equal in length

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DWARFDataTypeImporter.java

@@ -42,7 +42,6 @@ import ghidra.util.exception.InvalidInputException;
  * <p>
  * Create a new instance of this class for each {@link DIEAggregate} datatype that you wish
  * to convert into a DataType.
- * <p>
  */
 public class DWARFDataTypeImporter {
 	private DWARFProgram prog;
@@ -344,7 +343,7 @@ public class DWARFDataTypeImporter {
 
 	/**
 	 * Gets the corresponding Ghidra base type.
-	 * <p>
+	 * 
 	 * @param diea
 	 * @throws IOException
 	 * @throws DWARFExpressionException
@@ -401,7 +400,7 @@ public class DWARFDataTypeImporter {
 
 	/**
 	 * Simple passthru, returns whatever type this "const" modifier applies to.
-	 * <p>
+	 * 
 	 * @param diea
 	 * @throws IOException
 	 * @throws DWARFExpressionException
@@ -424,7 +423,6 @@ public class DWARFDataTypeImporter {
 	 * <p>
 	 * This method takes liberties with the normal{@literal DWARF->Ghidra Impl DataType->Ghidra DB DataType}
 	 * workflow to be able to merge values into previous db enum datatypes.
-	 * <p>
 	 *
 	 * @param diea
 	 * @return

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DWARFDataTypeManager.java

@@ -198,7 +198,7 @@ public class DWARFDataTypeManager {
 	 * Returns a Ghidra {@link DataType} corresponding to the specified {@link DIEAggregate},
 	 * or the specified defaultValue if the DIEA param is null or does not map to an already
 	 * defined datatype (registered with {@link #addDataType(long, DataType, DWARFSourceInfo)}).
-	 * <p>
+	 * 
 	 * @param diea {@link DIEAggregate} that defines a data type
 	 * @param defaultValue Ghidra {@link DataType} to return if the specified DIEA is null
 	 * or not already defined.
@@ -230,7 +230,6 @@ public class DWARFDataTypeManager {
 	 * Returns a Ghidra {@link DataType} corresponding to the specified DIE (based on its
 	 * offset), or the specified defaultValue if the DIE does not map to a defined
 	 * datatype (registered with {@link #addDataType(long, DataType, DWARFSourceInfo)}).
-	 * <p>
 	 *
 	 * @param dieOffset offset of a DIE record that defines a data type
 	 * @param defaultValue Ghidra {@link DataType} to return if the specified DIE not already defined.
@@ -312,7 +311,7 @@ public class DWARFDataTypeManager {
 	/**
 	 * Returns a Ghidra {@link DataType datatype} that corresponds to a type
 	 * that can be used to represent an offset.
-	 * <p>
+	 * 
 	 * @param size
 	 * @return
 	 */
@@ -366,7 +365,7 @@ public class DWARFDataTypeManager {
 	 * <p>
 	 * Any newly created Ghidra data types will be cached and the same instance will be returned
 	 * if the same DWARF named base type is requested again.
-	 * <p>
+	 * 
 	 * @param name
 	 * @param dwarfSize
 	 * @param dwarfEncoding
@@ -672,7 +671,6 @@ public class DWARFDataTypeManager {
 	/**
 	 * Regex to match common fixed-size type names like "int64", "int64_t", etc, by triggering
 	 * off some known size designators in the string.
-	 * <p>
 	 * <pre>
 	 * \D+ &larr; one or more non-digits
 	 *    (8|16|32|64|128) &larr; list of common sizes that we are triggering on

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DWARFLocationList.java

@@ -47,7 +47,7 @@ public class DWARFLocationList {
 
 	/**
 	 * Read a v4 {@link DWARFLocationList} from the debug_loc section.
-	 * <p>
+	 * 
 	 * @param reader stream positioned at the start of a .debug_loc location list 
 	 * @param cu the compUnit that refers to the location list
 	 * @return list of DWARF locations (address range and location expression)

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DWARFName.java

@@ -26,7 +26,6 @@ import ghidra.program.model.symbol.SymbolType;
 /**
  * A immutable hierarchical path based name implementation that can be viewed as either
  * {@link Namespace namespaces} or {@link CategoryPath categorypaths}.
- * <p>
  */
 public class DWARFName {
 
@@ -246,7 +245,6 @@ public class DWARFName {
 	/**
 	 * Returns true if this instance's {@link #getName() name} value is different
 	 * than its {@link #getOriginalName() original} form.
-	 * <p>
 	 *
 	 * @return boolean true if the original name doesn't match the ghidra-ized name
 	 */

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DWARFProgram.java

@@ -71,7 +71,7 @@ public class DWARFProgram implements Closeable {
 	 * If the program is a MachO binary (Mac), it must have a ".dSYM" directory co-located 
 	 * next to the original binary file on the native filesystem (outside of Ghidra).  See the 
 	 * DSymSectionProvider for more info.
-	 * <p>
+	 * 
 	 * @param program {@link Program} to test
 	 * @return boolean true if program probably has DWARF info, false if not
 	 */
@@ -104,7 +104,6 @@ public class DWARFProgram implements Closeable {
 	 * <p>
 	 * This is similar to {@link #isDWARF(Program)}, but is a stronger check that is more
 	 * expensive as it could involve searching for external files.
-	 * <p>
 	 * 
 	 * @param program {@link Program} to test
 	 * @param monitor {@link TaskMonitor} that can be used to cancel
@@ -1336,7 +1335,7 @@ public class DWARFProgram implements Closeable {
 	 * A fixup value that needs to be applied to static addresses of the program.
 	 * <p>
 	 * This value is necessary if the program's built-in base address is overridden at import time.
-	 * <p>
+	 * 
 	 * @return long value to add to static addresses discovered in DWARF to make it agree with
 	 * Ghidra's imported program.
 	 */

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DWARFRegisterMappings.java

@@ -15,11 +15,11 @@
  */
 package ghidra.app.util.bin.format.dwarf;
 
-import ghidra.program.model.lang.Register;
-
 import java.util.Collections;
 import java.util.Map;
 
+import ghidra.program.model.lang.Register;
+
 /**
  * Immutable mapping information between DWARF and Ghidra.
  * <p>
@@ -30,7 +30,7 @@ import java.util.Map;
  * in xml files referenced from the language *.ldefs file in an
  * &lt;external_name tool="DWARF.register.mapping.file" name="register_mapping_filename_here"/&gt; 
  * <p>
- * The format is:<p>
+ * The format is:
  * <pre>
  * &lt;dwarf&gt;
  *   &lt;register_mappings&gt;

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DWARFRegisterMappingsManager.java

@@ -15,13 +15,12 @@
  */
 package ghidra.app.util.bin.format.dwarf;
 
+import java.io.IOException;
+import java.io.InputStream;
 import java.util.*;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
-import java.io.IOException;
-import java.io.InputStream;
-
 import org.jdom.*;
 import org.jdom.input.SAXBuilder;
 
@@ -32,7 +31,6 @@ import ghidra.util.xml.XmlUtilities;
 
 /**
  * Factory class to instantiate and cache {@link DWARFRegisterMappings} objects.
- * <p>
  */
 public class DWARFRegisterMappingsManager {
 	private static final String DWARF_REGISTER_MAPPING_NAME = "DWARF.register.mapping.file";
@@ -54,7 +52,6 @@ public class DWARFRegisterMappingsManager {
 	/**
 	 * Returns a possibly cached {@link DWARFRegisterMappings} object for the
 	 * specified language,
-	 * <p>
 	 * 
 	 * @param lang {@link Language} to get the matching DWARF register mappings
 	 *            for
@@ -79,7 +76,6 @@ public class DWARFRegisterMappingsManager {
 	 * <p>
 	 * Throws {@link IOException} if the lang does not have a mapping or it is
 	 * invalid.
-	 * <p>
 	 * 
 	 * @param lang {@link Language} to read the matching DWARF register mappings
 	 *            for
@@ -118,7 +114,6 @@ public class DWARFRegisterMappingsManager {
 	/**
 	 * Creates a new {@link DWARFRegisterMappings} from the data present in the
 	 * xml element.
-	 * <p>
 	 * 
 	 * @param rootElem JDom XML element containing the &lt;dwarf&gt; root
 	 *            element of the mapping file.

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DWARFUtil.java

@@ -58,7 +58,7 @@ public class DWARFUtil {
 	 * which holds a specific value.
 	 * <p>
 	 * Can be thought of as an enum numeric value to to name lookup.
-	 * <p>
+	 * 
 	 * @param clazz
 	 * @param value
 	 * @return
@@ -110,7 +110,7 @@ public class DWARFUtil {
 	 * <p>
 	 * For example, "_ZN19class1_inline_funcs3fooEv" -&gt;
 	 * [19 chars]'class1_inline_funcs', [3 chars]'foo'
-	 * <p>
+	 * 
 	 * @param s
 	 * @return
 	 */
@@ -143,7 +143,7 @@ public class DWARFUtil {
 
 	/**
 	 * Try to find gnu mangled name nesting info in a DIE's children's linkage strings.
-	 * <p>
+	 * 
 	 * @param die
 	 * @return a list of string of nesting names, ending with what should be the DIE parameter's
 	 * name.
@@ -175,7 +175,7 @@ public class DWARFUtil {
 	 * <p>
 	 * operator&lt;() and operator&lt;&lt;() are handled so their angle brackets
 	 * don't trigger the template start/end angle bracket incorrectly.
-	 * <p>
+	 * 
 	 * @param name symbol name with C++ template portions
 	 * @return base portion of the symbol name without template portion
 	 */
@@ -193,7 +193,7 @@ public class DWARFUtil {
 
 	/**
 	 * Creates a name for anon types based on their position in their parent's childList.
-	 * <p>
+	 * 
 	 * @param diea the die aggregate.
 	 * @return the anonymous name of the die aggregate.
 	 */
@@ -468,7 +468,7 @@ public class DWARFUtil {
 	 * Returns a value specified in a {@link Language} definition via a
 	 * <pre>&lt;external_name tool="<b>name</b>" name="<b>value</b>"/&gt;</pre>
 	 * entry.
-	 * <p>
+	 * 
 	 * @param lang {@link Language} to query
 	 * @param name name of the value
 	 * @return String value

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DataTypeGraphComparator.java

@@ -45,7 +45,6 @@ public class DataTypeGraphComparator {
 		 * This callback can choose to abort traversing the tree of child types if it returns
 		 * false.  (ie. if this was a Pointer DataType, returning false would stop
 		 * the graph comparator from comparing the DataType pointed to by this Pointer)
-		 * <p>
 		 *
 		 * @param dt1 element from the first/left/src DataType graph
 		 * @param dt2 matching element from the second/right/dest DataType graph
@@ -57,7 +56,7 @@ public class DataTypeGraphComparator {
 	/**
 	 * Compares two {@link DataType datatypes} graphs, calling the observer callback
 	 * for each paired DataType that occupy equivalent positions in each graph.
-	 * <p>
+	 * 
 	 * @param preDT - Original (impl) DataType from before submitting to DataTypeManager.
 	 * @param postDT - Result DataType from the DataTypeManager
 	 * @param observer - Callback called for each position in the preDT graph that has a matching

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/DebugInfoEntry.java

@@ -132,7 +132,7 @@ public class DebugInfoEntry {
 
 	/**
 	 * Return a list of children that are of a specific DWARF type.
-	 * <p>
+	 * 
 	 * @param childTag DIE tag used to filter the child DIEs
 	 * @return list of matching child DIE records
 	 */

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/StringTable.java

@@ -54,7 +54,7 @@ public class StringTable {
 
 	/**
 	 * Returns true if the specified offset is a valid offset for this string table.
-	 * <p>
+	 * 
 	 * @param offset location of possible string
 	 * @return boolean true if location is valid
 	 */

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/attribs/DWARFAttribute.java

@@ -229,7 +229,7 @@ public enum DWARFAttribute {
 		 * Reads a {@link DWARFAttribute.AttrDef} instance from the {@link BinaryReader reader}.
 		 * <p>
 		 * Returns a null if its a end-of-list marker.
-		 * <p>
+		 * 
 		 * @param reader {@link BinaryReader} abbr stream
 		 * @return new {@link AttrDef}, or null if end-of-list
 		 * @throws IOException if error reading

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/attribs/DWARFAttributeDef.java

@@ -43,7 +43,7 @@ public class DWARFAttributeDef<E extends Enum<E>> {
 	 * Reads a {@link DWARFAttributeDef} instance from the {@link BinaryReader reader}.
 	 * <p>
 	 * Returns a null if its a end-of-list marker (which is only used by an attributespec list).
-	 * <p>
+	 * 
 	 * @param <E> attribute id enum type
 	 * @param reader {@link BinaryReader}
 	 * @param mapper func that converts an attribute id int into its enum

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/expression/DWARFExpression.java

@@ -191,7 +191,7 @@ public class DWARFExpression {
 	/**
 	 * Finds the index of an {@link DWARFExpressionOperation operation} by its offset
 	 * from the beginning of the expression.
-	 * <p>
+	 * 
 	 * @param offset
 	 * @return -1 if there is no op at the specified offset
 	 */

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/expression/DWARFExpressionEvaluator.java

@@ -34,7 +34,6 @@ import ghidra.program.model.lang.Register;
  * Can only specify offset from register for framebase and stack relative<br>
  * <p>
  * Result can be a numeric value (ie. static address) or a register 'name' or a stack based offset.
- * <p>
  */
 public class DWARFExpressionEvaluator {
 
@@ -121,7 +120,7 @@ public class DWARFExpressionEvaluator {
 	 * <p>
 	 * Note, you should check {@link #isDeref()} to see if the register is just a pointer
 	 * to the object instead of the object itself.
-	 * <p>
+	 * 
 	 * @return
 	 */
 	public Register getTerminalRegister() {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/expression/DWARFExpressionOperation.java

@@ -21,7 +21,6 @@ import java.util.Arrays;
  * An immutable representation of a single {@link DWARFExpression} instruction and its operands.
  * <p>
  * A DWARF expression operation can take 0, 1, or 2 operands.
- * <p>
  */
 class DWARFExpressionOperation {
 	protected final int offset;
@@ -71,7 +70,7 @@ class DWARFExpressionOperation {
 	 * <p>
 	 * Ie. If this opcode was DW_OP_reg12 (0x5c), and the base op code was DW_OP_reg0 (0x50),
 	 * the result value would be 12.
-	 * <p>
+	 * 
 	 * @param baseOpCode Ordinal value of the opcode that this opcode is being compared ot.
 	 * @return numeric difference between this opcode and the base opcode.
 	 */

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/line/DWARFLineContentType.java

@@ -70,7 +70,7 @@ public enum DWARFLineContentType {
 		 * Reads a {@link DWARFLineContentType.Def} instance from the {@link BinaryReader reader}.
 		 * <p>
 		 * Returns a null if its a end-of-list marker.
-		 * <p>
+		 * 
 		 * @param reader {@link BinaryReader} stream
 		 * @return {@link DWARFLineContentType.Def}, or null if stream was at a end-of-list marker
 		 * (which isn't really a thing for line content defs, but is a thing for attribute defs)

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/dwarf/sectionprovider/CompressedSectionProvider.java

@@ -15,13 +15,12 @@
  */
 package ghidra.app.util.bin.format.dwarf.sectionprovider;
 
+import java.io.IOException;
+import java.io.InputStream;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.zip.InflaterInputStream;
 
-import java.io.IOException;
-import java.io.InputStream;
-
 import ghidra.app.util.bin.BinaryReader;
 import ghidra.app.util.bin.ByteProvider;
 import ghidra.formats.gfilesystem.FSUtilities;
@@ -35,7 +34,6 @@ import ghidra.util.task.TaskMonitor;
  * A wrapper around another DWARFSectionProvider, this provider
  * fetches DWARF section data that has been compressed and stored in sections in the underlying 
  * {@link DWARFSectionProvider}.
- * <p>
  */
 public class CompressedSectionProvider implements DWARFSectionProvider {
 	private static final int ZLIB_MAGIC_BE = 0x5a4c4942;	// "ZLIB"

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/elf/ElfCompressedSectionHeader.java

@@ -23,7 +23,6 @@ import ghidra.app.util.bin.BinaryReader;
  * Header at the beginning of an ELF compressed section.
  * <p>
  * See https://docs.oracle.com/cd/E53394_01/html/E54813/section_compression.html
- * <p>
  * <pre>
  * typedef struct {
  *      Elf32_Word      ch_type;

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/elf/info/NoteGnuProperty.java

@@ -15,11 +15,10 @@
  */
 package ghidra.app.util.bin.format.elf.info;
 
+import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
 
-import java.io.IOException;
-
 import ghidra.app.util.bin.BinaryReader;
 import ghidra.app.util.bin.StructConverter;
 import ghidra.framework.options.Options;
@@ -35,7 +34,6 @@ import ghidra.util.NumericUtilities;
  * An ELF note that contains a list of enumerated "properties".
  * <p>
  * Currently known property types are stack_size and no_copy_on_protected (flag).
- * <p>
  * <pre>
  *  array of Elf_Prop {
  *    word pr_type;

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/golang/GoRegisterInfo.java

@@ -30,7 +30,6 @@ import ghidra.util.exception.InvalidInputException;
 /**
  * Immutable information about registers, alignment sizes, etc needed to allocate storage
  * for parameters during a function call.
- * <p>
  */
 public class GoRegisterInfo {
 

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/golang/GolangDWARFFunctionFixup.java

@@ -45,7 +45,6 @@ import ghidra.util.task.TaskMonitor;
  * <p>
  * Go's 'unique' usage of DW_TAG_subroutine_type to define its ptr-to-ptr-to-func is handled in
  * DWARFDataTypeImporter.makeDataTypeForFunctionDefinition().
- * <p>
  */
 @ExtensionPointProperties(priority = DWARFFunctionFixup.PRIORITY_NORMAL_EARLY)
 public class GolangDWARFFunctionFixup implements DWARFFunctionFixup {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/golang/rtti/GoRttiMapper.java

@@ -1026,7 +1026,6 @@ public class GoRttiMapper extends DataTypeMapper implements DataTypeMapperContex
 
 	/**
 	 * Returns a function definition for a method that is attached to a golang type.
-	 * <p>
 	 * 
 	 * @param methodName name of method
 	 * @param methodType golang function def type
@@ -1339,7 +1338,6 @@ public class GoRttiMapper extends DataTypeMapper implements DataTypeMapperContex
 	/**
 	 * Returns the {@link GoName} corresponding to an offset that is relative to the controlling
 	 * GoModuledata's typesOffset.
-	 * <p>
 	 * 
 	 * @param ptrInModule the address of the structure that contains the offset that needs to be
 	 * calculated.  The containing-structure's address is important because it indicates which
@@ -1534,7 +1532,7 @@ public class GoRttiMapper extends DataTypeMapper implements DataTypeMapperContex
 		/**
 		 * Returns true if the specified function should be included in the bootstrap function defs
 		 * that are written to the golang_NNNN.gdt archive.
-		 * <p>
+		 * 
 		 * @return true if function should be included in golang.gdt bootstrap file
 		 */
 		public boolean isBootstrapFunction() {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/golang/rtti/GoSlice.java

@@ -38,7 +38,6 @@ import ghidra.util.exception.CancelledException;
  * <p>
  * Like java's type erasure for generics, a golang slice instance does not have type information 
  * about the elements found in the array blob (nor the size of the blob).
- * <p>
  */
 @StructureMapping(structureName = "runtime.slice")
 public class GoSlice implements StructureMarkup<GoSlice> {
@@ -269,7 +268,7 @@ public class GoSlice implements StructureMarkup<GoSlice> {
 
 	/**
 	 * Treats this slice as a array of unsigned integers, of the specified intSize.
-	 * <p>
+	 * 
 	 * @param intSize size of integer
 	 * @return array of longs, containing the (possibly smaller) integers contained in the slice
 	 * @throws IOException if error reading

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/golang/rtti/GoString.java

@@ -33,7 +33,7 @@ public class GoString implements StructureMarkup<GoString> {
 
 	/**
 	 * Creates a artificial gostring instance that was not read from a memory location.
-	 * <p>
+	 * 
 	 * @param goBinary {@link GoRttiMapper}
 	 * @param stringData location of char array
 	 * @param len length of char array

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/golang/rtti/types/GoBaseType.java

@@ -30,7 +30,6 @@ import ghidra.app.util.bin.format.golang.structmapping.*;
  * <p>
  * Additionally, there can be an {@link GoUncommonType} structure immediately after this type, if
  * the uncommon bit is set in tflag.
- * <p>
  * <pre>
  * struct specialized_type { basetype_struct; (various_fields)* } struct uncommon; 
  * </pre>

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/golang/structmapping/FieldReadFunction.java

@@ -19,7 +19,7 @@ import java.io.IOException;
 
 /**
  * Functional interface to read a structure field's value.
- * <p>
+ * 
  * @see #get(FieldContext)
  * 
  * @param <T> type of structure mapped class that contains this field

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/macho/prelink/NoPreLinkSectionException.java

@@ -17,9 +17,7 @@ package ghidra.app.util.bin.format.macho.prelink;
 
 
 public class NoPreLinkSectionException extends Exception {
-	/**
+
-	 * 
-	 */
 	private static final long serialVersionUID = 1L;
 
 	public NoPreLinkSectionException( String message ) {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/omf/OmfRecord.java

@@ -36,9 +36,6 @@ public abstract class OmfRecord implements StructConverter {
 	protected BinaryReader dataReader;
 	protected long dataEnd;
 
-	/**
-	 * 
-	 */
 	public OmfRecord() {
 		// nothing to do
 	}

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/BoundImportDescriptor.java

@@ -29,7 +29,6 @@ import ghidra.util.exception.DuplicateNameException;
  * A class to represent the 
  * <code>IMAGE_BOUND_IMPORT_DESCRIPTOR</code>
  * data structure defined in <b><code>winnt.h</code></b>.
- * <p>
  * <pre>
  * typedef struct _IMAGE_BOUND_IMPORT_DESCRIPTOR {
  *     DWORD   TimeDateStamp;

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/BoundImportForwarderRef.java

@@ -27,7 +27,6 @@ import ghidra.util.exception.DuplicateNameException;
  * A class to represent the 
  * <code>IMAGE_BOUND_FORWARDER_REF</code>
  * data structure defined in <b><code>winnt.h</code></b>.
- * <p>
  * <pre>
  * typedef struct _IMAGE_BOUND_FORWARDER_REF {
  *     DWORD   TimeDateStamp;

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/DelayImportDescriptor.java

@@ -29,7 +29,6 @@ import ghidra.util.exception.DuplicateNameException;
  * A class to represent the 
  * <code>ImgDelayDescr</code>
  * data structure defined in <b><code>DELAYIMP.H</code></b>.
- * <p>
  * <pre>
  * typedef struct ImgDelayDescr {
  *     DWORD           grAttrs;        // attributes

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/ImageCor20Header.java

@@ -32,6 +32,7 @@ import ghidra.util.exception.DuplicateNameException;
 import ghidra.util.task.TaskMonitor;
 
 /**
+ * <pre>
  * typedef struct IMAGE_COR20_HEADER
  * {
  *     // Header versioning
@@ -61,7 +62,6 @@ import ghidra.util.task.TaskMonitor;
  *
  *};
  *</pre>
- *
  */
 public class ImageCor20Header implements StructConverter, PeMarkupable {
 	private static final String NAME = "IMAGE_COR20_HEADER";

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/MachineName.java

@@ -15,9 +15,6 @@
  */
 package ghidra.app.util.bin.format.pe;
 
-/**
- * 
- */
 class MachineName {
 
 	static String getName(short machine) {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/SecurityCertificate.java

@@ -28,7 +28,6 @@ import ghidra.util.exception.DuplicateNameException;
  * struct as defined in <b><code>winbase.h</code></b>.
  * <p>
  * This structure encapsulates a signature used in verifying executables.
- * <p>
  * <pre>
  * typedef struct _WIN_CERTIFICATE {
  *     DWORD       dwLength;

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/DebugSymbolSelector.java

@@ -19,9 +19,6 @@ import java.io.IOException;
 
 import ghidra.app.util.bin.BinaryReader;
 
-/**
- * 
- */
 public class DebugSymbolSelector {
 
 	public static DebugSymbol selectSymbol(BinaryReader reader, int ptr) throws IOException {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/S_BLOCK32.java

@@ -15,9 +15,6 @@
  */
 package ghidra.app.util.bin.format.pe.debug;
 
-/**
- * 
- */
 class S_BLOCK32 extends DebugSymbol {
 
 	S_BLOCK32(short length, short type) {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/S_COMPILE.java

@@ -15,9 +15,6 @@
  */
 package ghidra.app.util.bin.format.pe.debug;
 
-/**
- * 
- */
 class S_COMPILE extends DebugSymbol {
 
 	S_COMPILE(short length, short type) {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/S_CONSTANT32.java

@@ -19,11 +19,7 @@ import java.io.IOException;
 
 import ghidra.app.util.bin.BinaryReader;
 import ghidra.util.Conv;
-import ghidra.util.Msg;
 
-/**
- * 
- */
 class S_CONSTANT32 extends DebugSymbol {
 
 	S_CONSTANT32(short length, short type, BinaryReader reader, int ptr) throws IOException {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/S_END.java

@@ -18,9 +18,6 @@ package ghidra.app.util.bin.format.pe.debug;
 import ghidra.app.util.bin.BinaryReader;
 import ghidra.util.Msg;
 
-/**
- * 
- */
 class S_END extends DebugSymbol {
 
 	S_END(short length, short type, BinaryReader reader, int ptr) {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/S_GDATA32_NEW.java

@@ -21,9 +21,6 @@ import ghidra.app.util.bin.BinaryReader;
 import ghidra.util.Conv;
 import ghidra.util.Msg;
 
-/**
- * 
- */
 class S_GDATA32_NEW extends DebugSymbol {
 
 	S_GDATA32_NEW(short length, short type, BinaryReader reader, int ptr) throws IOException {

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/S_LABEL32.java

@@ -21,9 +21,6 @@ import ghidra.app.util.bin.BinaryReader;
 import ghidra.util.Conv;
 import ghidra.util.Msg;
 
-/**
- * 
- */
 class S_LABEL32 extends DebugSymbol {
     private byte flags;
 	

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/S_LDATA32_NEW.java

@@ -20,9 +20,6 @@ import java.io.IOException;
 import ghidra.app.util.bin.BinaryReader;
 import ghidra.util.Conv;
 
-/**
- * 
- */
 class S_LDATA32_NEW extends DebugSymbol{
     private int reserved;
 	private byte [] padding;

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/S_OBJNAME.java

@@ -20,9 +20,6 @@ import java.io.IOException;
 import ghidra.app.util.bin.BinaryReader;
 import ghidra.util.Conv;
 
-/**
- * 
- */
 class S_OBJNAME extends DebugSymbol {
     private int signature;
 	private byte nameLen;

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/S_UDT32.java

@@ -20,9 +20,6 @@ import java.io.IOException;
 import ghidra.app.util.bin.BinaryReader;
 import ghidra.util.Conv;
 
-/**
- * 
- */
 class S_UDT32 extends DebugSymbol {
     private int checksum;
 	private byte typeLen;

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/S_UDT32_NEW.java

@@ -20,9 +20,6 @@ import java.io.IOException;
 import ghidra.app.util.bin.BinaryReader;
 import ghidra.util.Conv;
 
-/**
- * 
- */
 class S_UDT32_NEW extends DebugSymbol {
     private int symType;
 

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/pe/debug/UnknownSymbol.java

@@ -21,9 +21,6 @@ import ghidra.app.util.bin.BinaryReader;
 import ghidra.util.Conv;
 import ghidra.util.Msg;
 
-/**
- * 
-*/
 class UnknownSymbol extends DebugSymbol{
     private byte [] unknown;
 

Ghidra/Features/Base/src/main/java/ghidra/app/util/cparser/CPP/DefineTable.java

@@ -23,9 +23,6 @@ import ghidra.app.util.cparser.CPP.PreProcessor.PPToken;
 import ghidra.program.model.data.*;
 import ghidra.util.Msg;
 
-/**
- * 
- */
 public class DefineTable {
 	private static final String VARARG_ELLIPSIS = "...";
 
@@ -44,9 +41,6 @@ public class DefineTable {
 
 	private final static String VALUE = "value";
 
-	/**
-	 * 
-	 */
 	public DefineTable() {
 		super();
 		// TODO Auto-generated constructor stub