Skip to content

Commit

Permalink
8260475: Deprecate for removal protected access members in DateTimeSt…
Browse files Browse the repository at this point in the history
…ringConverter

Reviewed-by: kcr, aghaisas
  • Loading branch information
nlisker committed Feb 5, 2021
1 parent 887a443 commit d4058a1
Showing 1 changed file with 22 additions and 0 deletions.
Expand Up @@ -43,18 +43,37 @@
public class DateTimeStringConverter extends StringConverter<Date> {

// ------------------------------------------------------ Private properties

/**
* @deprecated This field was exposed erroneously and will be removed in a future version.
*/
@Deprecated(forRemoval = true, since = "17")
protected final Locale locale;

/**
* @deprecated This field was exposed erroneously and will be removed in a future version.
*/
@Deprecated(forRemoval = true, since = "17")
protected final String pattern;

/**
* @deprecated This field was exposed erroneously and will be removed in a future version.
*/
@Deprecated(forRemoval = true, since = "17")
protected final DateFormat dateFormat;

/**
* @deprecated This field was exposed erroneously and will be removed in a future version.
* @since JavaFX 8u40
*/
@Deprecated(forRemoval = true, since = "17")
protected final int dateStyle;

/**
* @deprecated This field was exposed erroneously and will be removed in a future version.
* @since JavaFX 8u40
*/
@Deprecated(forRemoval = true, since = "17")
protected final int timeStyle;


Expand Down Expand Up @@ -199,7 +218,10 @@ public DateTimeStringConverter(DateFormat dateFormat) {
*
* @return a {@code DateFormat} instance for formatting and parsing in this
* {@link StringConverter}
*
* @deprecated This method exposes internal implementation details and will be removed in a future version.
*/
@Deprecated(forRemoval = true, since = "17")
protected DateFormat getDateFormat() {
DateFormat df = null;

Expand Down

1 comment on commit d4058a1

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.