organicmaps/icu
13
Fork 0
mirror of https://github.com/unicode-org/icu.git synced 2025-04-05 21:45:37 +00:00
Code Issues 1 Projects Releases Wiki Activity Actions 12

ICU-22991 Reduce fStamp to 8 bits

Browse source 
See #3323
This commit is contained in:
Frank Tang 2025-01-06 22:05:52 +00:00 • committed by Frank Yung-Fong Tang
parent 4c3622f218
commit 6091406a3a
3 changed files with 72 additions and 84 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
icu4c/source/i18n/calendar.cpp
View file

@ -156,7 +156,7 @@ U_CFUNC void ucal_dump(UCalendar* cal) {
#endif
/* Max value for stamp allowable before recalculation */
#define STAMP_MAX 10000
#define STAMP_MAX 127
static const char * const gCalTypes[] = {
"gregorian",
@ -700,10 +700,7 @@ fIsTimeSet(false),
fAreFieldsSet(false),
fAreAllFieldsSet(false),
fAreFieldsVirtuallySet(false),
fNextStamp(static_cast<int32_t>(kMinimumUserStamp)),
fTime(0),
fLenient(true),
fZone(nullptr),
fRepeatedWallTime(UCAL_WALLTIME_LAST),
fSkippedWallTime(UCAL_WALLTIME_LAST)
{
@ -726,10 +723,7 @@ fIsTimeSet(false),
fAreFieldsSet(false),
fAreAllFieldsSet(false),
fAreFieldsVirtuallySet(false),
fNextStamp(static_cast<int32_t>(kMinimumUserStamp)),
fTime(0),
fLenient(true),
fZone(nullptr),
fRepeatedWallTime(UCAL_WALLTIME_LAST),
fSkippedWallTime(UCAL_WALLTIME_LAST)
{
@ -759,10 +753,7 @@ fIsTimeSet(false),
fAreFieldsSet(false),
fAreAllFieldsSet(false),
fAreFieldsVirtuallySet(false),
fNextStamp(static_cast<int32_t>(kMinimumUserStamp)),
fTime(0),
fLenient(true),
fZone(nullptr),
fRepeatedWallTime(UCAL_WALLTIME_LAST),
fSkippedWallTime(UCAL_WALLTIME_LAST)
{
@ -792,7 +783,6 @@ Calendar::~Calendar()
Calendar::Calendar(const Calendar &source)
: UObject(source)
{
fZone = nullptr;
*this = source;
}
@ -1163,12 +1153,9 @@ Calendar::setTimeInMillis( double millis, UErrorCode& status ) {
fAreFieldsSet = fAreAllFieldsSet = false;
fIsTimeSet = fAreFieldsVirtuallySet = true;
for (int32_t i=0; i<UCAL_FIELD_COUNT; ++i) {
fFields[i] = 0;
fStamp[i] = kUnset;
}
uprv_memset(fFields, 0, sizeof(fFields));
uprv_memset(fStamp, kUnset, sizeof(fStamp));
fNextStamp = kMinimumUserStamp;
}
// -------------------------------------
@ -1203,7 +1190,7 @@ Calendar::set(UCalendarDateFields field, int32_t value)
computeFields(ec);
}
fFields[field] = value;
/* Ensure that the fNextStamp value doesn't go pass max value for int32_t */
/* Ensure that the fNextStamp value doesn't go pass max value for int8_t */
if (fNextStamp == STAMP_MAX) {
recalculateStamp();
}
@ -1264,10 +1251,9 @@ void Calendar::setRelatedYear(int32_t year)
void
Calendar::clear()
{
for (int32_t i=0; i<UCAL_FIELD_COUNT; ++i) {
fFields[i] = 0; // Must do this; other code depends on it
fStamp[i] = kUnset;
}
uprv_memset(fFields, 0, sizeof(fFields));
uprv_memset(fStamp, kUnset, sizeof(fStamp));
fNextStamp = kMinimumUserStamp;
fIsTimeSet = fAreFieldsSet = fAreAllFieldsSet = fAreFieldsVirtuallySet = false;
// fTime is not 'cleared' - may be used if no fields are set.
}
@ -4228,7 +4214,7 @@ Calendar::recalculateStamp() {
int32_t currentValue;
int32_t j, i;
fNextStamp = 1;
fNextStamp = kInternallySet;
for (j = 0; j < UCAL_FIELD_COUNT; j++) {
currentValue = STAMP_MAX;

97
icu4c/source/i18n/unicode/calendar.h
View file

@ -1882,38 +1882,6 @@ private:
*/
int32_t getActualHelper(UCalendarDateFields field, int32_t startValue, int32_t endValue, UErrorCode &status) const;
private:
/**
* The flag which indicates if the current time is set in the calendar.
*/
UBool fIsTimeSet;
/**
* True if the fields are in sync with the currently set time of this Calendar.
* If false, then the next attempt to get the value of a field will
* force a recomputation of all fields from the current value of the time
* field.
* <P>
* This should really be named areFieldsInSync, but the old name is retained
* for backward compatibility.
*/
UBool fAreFieldsSet;
/**
* True if all of the fields have been set. This is initially false, and set to
* true by computeFields().
*/
UBool fAreAllFieldsSet;
/**
* True if all fields have been virtually set, but have not yet been
* computed. This occurs only in setTimeInMillis(). A calendar set
* to this state will compute all fields from the time if it becomes
* necessary, but otherwise will delay such computation.
*/
UBool fAreFieldsVirtuallySet;
protected:
/**
* Get the current time without recomputing.
@ -1952,9 +1920,9 @@ private:
/**
* Pseudo-time-stamps which specify when each field was set. There
* are two special values, UNSET and INTERNALLY_SET. Values from
* MINIMUM_USER_SET to Integer.MAX_VALUE are legal user set values.
* MINIMUM_USER_SET to STAMP_MAX are legal user set values.
*/
int32_t fStamp[UCAL_FIELD_COUNT];
int8_t fStamp[UCAL_FIELD_COUNT];
protected:
/**
@ -2170,7 +2138,7 @@ private:
/**
* The next available value for fStamp[]
*/
int32_t fNextStamp;// = MINIMUM_USER_STAMP;
int8_t fNextStamp = kMinimumUserStamp;
/**
* Recalculates the time stamp array (fStamp).
@ -2181,30 +2149,60 @@ private:
/**
* The current time set for the calendar.
*/
UDate fTime;
/**
* @see #setLenient
*/
UBool fLenient;
UDate fTime = 0;
/**
* Time zone affects the time calculation done by Calendar. Calendar subclasses use
* the time zone data to produce the local time. Always set; never nullptr.
*/
TimeZone* fZone;
TimeZone* fZone = nullptr;
/**
* The flag which indicates if the current time is set in the calendar.
*/
bool fIsTimeSet:1;
/**
* True if the fields are in sync with the currently set time of this Calendar.
* If false, then the next attempt to get the value of a field will
* force a recomputation of all fields from the current value of the time
* field.
* <P>
* This should really be named areFieldsInSync, but the old name is retained
* for backward compatibility.
*/
bool fAreFieldsSet:1;
/**
* True if all of the fields have been set. This is initially false, and set to
* true by computeFields().
*/
bool fAreAllFieldsSet:1;
/**
* True if all fields have been virtually set, but have not yet been
* computed. This occurs only in setTimeInMillis(). A calendar set
* to this state will compute all fields from the time if it becomes
* necessary, but otherwise will delay such computation.
*/
bool fAreFieldsVirtuallySet:1;
/**
* @see #setLenient
*/
bool fLenient:1;
/**
* Option for repeated wall time
* @see #setRepeatedWallTimeOption
*/
UCalendarWallTimeOption fRepeatedWallTime;
UCalendarWallTimeOption fRepeatedWallTime:3; // Somehow MSVC need 3 bits for UCalendarWallTimeOption
/**
* Option for skipped wall time
* @see #setSkippedWallTimeOption
*/
UCalendarWallTimeOption fSkippedWallTime;
UCalendarWallTimeOption fSkippedWallTime:3; // Somehow MSVC need 3 bits for UCalendarWallTimeOption
/**
* Both firstDayOfWeek and minimalDaysInFirstWeek are locale-dependent. They are
@ -2214,11 +2212,14 @@ private:
* out the week count for a specific date for a given locale. These must be set when
* a Calendar is constructed.
*/
UCalendarDaysOfWeek fFirstDayOfWeek;
uint8_t fMinimalDaysInFirstWeek;
UCalendarDaysOfWeek fWeekendOnset;
UCalendarDaysOfWeek fFirstDayOfWeek:4; // Somehow MSVC need 4 bits for
// UCalendarDaysOfWeek
UCalendarDaysOfWeek fWeekendOnset:4; // Somehow MSVC need 4 bits for
// UCalendarDaysOfWeek
UCalendarDaysOfWeek fWeekendCease:4; // Somehow MSVC need 4 bits for
// UCalendarDaysOfWeek
uint8_t fMinimalDaysInFirstWeek;
int32_t fWeekendOnsetMillis;
UCalendarDaysOfWeek fWeekendCease;
int32_t fWeekendCeaseMillis;
/**

27
icu4j/main/core/src/main/java/com/ibm/icu/util/Calendar.java
View file

@ -13,6 +13,7 @@ import java.io.ObjectOutputStream;
import java.io.Serializable;
import java.text.StringCharacterIterator;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Date;
import java.util.Locale;
import java.util.MissingResourceException;
@ -1355,9 +1356,9 @@ public abstract class Calendar implements Serializable, Cloneable, Comparable<Ca
/**
* Pseudo-time-stamps which specify when each field was set. There
* are two special values, UNSET and INTERNALLY_SET. Values from
* MINIMUM_USER_SET to Integer.MAX_VALUE are legal user set values.
* MINIMUM_USER_SET to STAMP_MAX are legal user set values.
*/
private transient int stamp[];
private transient byte stamp[];
/**
* The currently set time for this calendar, expressed in milliseconds after
@ -1507,10 +1508,10 @@ public abstract class Calendar implements Serializable, Cloneable, Comparable<Ca
* The next available value for <code>stamp[]</code>, an internal array.
* @serial
*/
private transient int nextStamp = MINIMUM_USER_STAMP;
private transient byte nextStamp = MINIMUM_USER_STAMP;
/* Max value for stamp allowable before recalculation */
private static int STAMP_MAX = 10000;
private static byte STAMP_MAX = Byte.MAX_VALUE;
// the internal serial version which says which version was written
// - 0 (default) for version up to JDK 1.1.5
@ -1684,7 +1685,7 @@ public abstract class Calendar implements Serializable, Cloneable, Comparable<Ca
private void recalculateStamp() {
int index;
int currentValue;
byte currentValue;
int j, i;
nextStamp = 1;
@ -1721,7 +1722,7 @@ public abstract class Calendar implements Serializable, Cloneable, Comparable<Ca
throw new IllegalStateException("Invalid fields[]");
}
///CLOVER:ON
stamp = new int[fields.length];
stamp = new byte[fields.length];
int mask = (1 << ERA) |
(1 << YEAR) |
(1 << MONTH) |
@ -2054,9 +2055,9 @@ public abstract class Calendar implements Serializable, Cloneable, Comparable<Ca
areFieldsSet = areAllFieldsSet = false;
isTimeSet = areFieldsVirtuallySet = true;
for (int i=0; i<fields.length; ++i) {
fields[i] = stamp[i] = 0; // UNSET == 0
}
Arrays.fill(fields, 0);
Arrays.fill(stamp, (byte)0);
nextStamp = MINIMUM_USER_STAMP;
}
@ -2455,9 +2456,9 @@ public abstract class Calendar implements Serializable, Cloneable, Comparable<Ca
*/
public final void clear()
{
for (int i=0; i<fields.length; ++i) {
fields[i] = stamp[i] = 0; // UNSET == 0
}
Arrays.fill(fields, 0);
Arrays.fill(stamp, (byte)0);
nextStamp = MINIMUM_USER_STAMP;
isTimeSet = areFieldsSet = areAllFieldsSet = areFieldsVirtuallySet = false;
}
@ -4904,7 +4905,7 @@ public abstract class Calendar implements Serializable, Cloneable, Comparable<Ca
Calendar other = (Calendar) super.clone();
other.fields = new int[fields.length];
other.stamp = new int[fields.length];
other.stamp = new byte[fields.length];
System.arraycopy(this.fields, 0, other.fields, 0, fields.length);
System.arraycopy(this.stamp, 0, other.stamp, 0, fields.length);