Address review comments. Made BusinessCalendar and Calendar construct…

…ors package so that they can be converted to interfaces in the future.
deephaven · Dec 15, 2023 · a007d3f · a007d3f
1 parent 787b646
commit a007d3f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/engine/time/src/main/java/io/deephaven/time/calendar/BusinessCalendar.java b/engine/time/src/main/java/io/deephaven/time/calendar/BusinessCalendar.java
@@ -147,7 +147,7 @@ private YearData getYearData(final int year) {
      *        is different from the schedule for a standard business day or weekend.
      * @throws RequirementFailure if any argument is null.
      */
-    public BusinessCalendar(final String name, final String description, final ZoneId timeZone,
+    BusinessCalendar(final String name, final String description, final ZoneId timeZone,
             final LocalDate firstValidDate, final LocalDate lastValidDate,
             final CalendarDay<LocalTime> standardBusinessDay, final Set<DayOfWeek> weekendDays,
             final Map<LocalDate, CalendarDay<Instant>> holidays) {

diff --git a/engine/time/src/main/java/io/deephaven/time/calendar/Calendar.java b/engine/time/src/main/java/io/deephaven/time/calendar/Calendar.java
@@ -41,7 +41,7 @@ public class Calendar {
      * @param timeZone calendar time zone.
      * @throws RequirementFailure if any parameter is {@code null}
      */
-    public Calendar(final String name, final String description, final ZoneId timeZone) {
+    Calendar(final String name, final String description, final ZoneId timeZone) {
         this.name = Require.neqNull(name, "name");
         this.description = Require.neqNull(description, "description");
         this.timeZone = Require.neqNull(timeZone, "timeZone");