Skip to content

Commit

Permalink
JDO-823: Fix more sonarcloud issues of type Code Smells - Track uses …
Browse files Browse the repository at this point in the history
…of "TODO" tags
  • Loading branch information
mboapache committed Feb 16, 2023
1 parent 38c7968 commit fc91403
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 8 deletions.
4 changes: 2 additions & 2 deletions api/src/main/java/javax/jdo/annotations/Element.java
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,7 @@
*/
@SuppressWarnings("rawtypes")
Class<? extends AttributeConverter> converter() default
UseDefault.class; // TODO Current JDK doesn't allow "default null"
UseDefault.class; // Current JDK doesn't allow "default null"

/**
* Whether we should disable any converter that was specified as default for this type on the PMF.
Expand All @@ -174,7 +174,7 @@ Class<? extends AttributeConverter> converter() default
*
* @return Whether PMF attribute conversion is to be disabled.
*/
boolean useDefaultConversion() default false; // TODO Current JDK doesn't allow "default null"
boolean useDefaultConversion() default false;

/**
* Vendor extensions.
Expand Down
4 changes: 2 additions & 2 deletions api/src/main/java/javax/jdo/annotations/Key.java
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,7 @@
*/
@SuppressWarnings("rawtypes")
Class<? extends AttributeConverter> converter() default
UseDefault.class; // TODO Current JDK doesn't allow "default null"
UseDefault.class; // Current JDK doesn't allow "default null"

/**
* Whether we should disable any converter that was specified as default for this type on the PMF.
Expand All @@ -170,7 +170,7 @@ Class<? extends AttributeConverter> converter() default
*
* @return Whether PMF attribute conversion is to be disabled.
*/
boolean useDefaultConversion() default false; // TODO Current JDK doesn't allow "default null"
boolean useDefaultConversion() default false;

/**
* Vendor extensions.
Expand Down
4 changes: 2 additions & 2 deletions api/src/main/java/javax/jdo/annotations/Persistent.java
Original file line number Diff line number Diff line change
Expand Up @@ -262,7 +262,7 @@
*/
@SuppressWarnings("rawtypes")
Class<? extends AttributeConverter> converter() default
UseDefault.class; // TODO Current JDK doesn't allow "default null"
UseDefault.class; // Current JDK doesn't allow "default null"

/**
* Whether we should disable any converter that was specified as default for this type on the PMF.
Expand All @@ -272,7 +272,7 @@ Class<? extends AttributeConverter> converter() default
*
* @return Whether PMF attribute conversion is to be disabled.
*/
boolean useDefaultConversion() default false; // TODO Current JDK doesn't allow "default null"
boolean useDefaultConversion() default false;

/**
* Vendor extensions for this member.
Expand Down
4 changes: 2 additions & 2 deletions api/src/main/java/javax/jdo/annotations/Value.java
Original file line number Diff line number Diff line change
Expand Up @@ -161,7 +161,7 @@
*/
@SuppressWarnings("rawtypes")
Class<? extends AttributeConverter> converter() default
UseDefault.class; // TODO Current JDK doesn't allow "default null"
UseDefault.class; // Current JDK doesn't allow "default null"

/**
* Whether we should disable any converter that was specified as default for this type on the PMF.
Expand All @@ -171,7 +171,7 @@ Class<? extends AttributeConverter> converter() default
*
* @return Whether PMF attribute conversion is to be disabled.
*/
boolean useDefaultConversion() default false; // TODO Current JDK doesn't allow "default null"
boolean useDefaultConversion() default false;

/**
* Vendor extensions.
Expand Down

0 comments on commit fc91403

Please sign in to comment.