Changeset - r28343:242bc14cc675
[Not reviewed]
master
0 16 0
Peter Nelson - 11 months ago 2023-12-28 21:34:08
peter1138@openttd.org
Codechange: Simplify ConvertDateToYMD by returning YearMonthDay instead of outputting to a pointer. (#11637)
16 files changed with 31 insertions and 51 deletions:
0 comments (0 inline, 0 general)
src/build_vehicle_gui.cpp
Show inline comments
 
@@ -920,8 +920,7 @@ void TestedEngineDetails::FillDefaultCap
 
int DrawVehiclePurchaseInfo(int left, int right, int y, EngineID engine_number, TestedEngineDetails &te)
 
{
 
	const Engine *e = Engine::Get(engine_number);
 
	TimerGameCalendar::YearMonthDay ymd;
 
	TimerGameCalendar::ConvertDateToYMD(e->intro_date, &ymd);
 
	TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(e->intro_date);
 
	bool refittable = IsArticulatedVehicleRefittable(engine_number);
 
	bool articulated_cargo = false;
 

	
src/cheat_gui.cpp
Show inline comments
 
@@ -104,8 +104,7 @@ static int32_t ClickChangeDateCheat(int3
 
	auto new_year = Clamp(TimerGameCalendar::Year(new_value), CalendarTime::MIN_YEAR, CalendarTime::MAX_YEAR);
 
	if (new_year == TimerGameCalendar::year) return TimerGameCalendar::year.base();
 

	
 
	TimerGameCalendar::YearMonthDay ymd;
 
	TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date, &ymd);
 
	TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date);
 
	TimerGameCalendar::Date new_date = TimerGameCalendar::ConvertYMDToDate(new_year, ymd.month, ymd.day);
 

	
 
	/* Shift cached dates before we change the date. */
src/console_cmds.cpp
Show inline comments
 
@@ -1442,8 +1442,7 @@ DEF_CONSOLE_CMD(ConGetDate)
 
		return true;
 
	}
 

	
 
	TimerGameCalendar::YearMonthDay ymd;
 
	TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date, &ymd);
 
	TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date);
 
	IConsolePrint(CC_DEFAULT, "Date: {:04d}-{:02d}-{:02d}", ymd.year, ymd.month + 1, ymd.day);
 
	return true;
 
}
src/crashlog.cpp
Show inline comments
 
@@ -57,8 +57,7 @@ static void SurveyRecentNews(nlohmann::j
 

	
 
	int i = 0;
 
	for (NewsItem *news = _latest_news; i < 32 && news != nullptr; news = news->prev, i++) {
 
		TimerGameCalendar::YearMonthDay ymd;
 
		TimerGameCalendar::ConvertDateToYMD(news->date, &ymd);
 
		TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(news->date);
 
		json.push_back(fmt::format("({}-{:02}-{:02}) StringID: {}, Type: {}, Ref1: {}, {}, Ref2: {}, {}",
 
		               ymd.year, ymd.month + 1, ymd.day, news->string_id, news->type,
 
		               news->reftype1, news->ref1, news->reftype2, news->ref2));
src/date_gui.cpp
Show inline comments
 
@@ -51,7 +51,7 @@ struct SetDateWindow : Window {
 
		this->InitNested(window_number);
 

	
 
		if (initial_date == 0) initial_date = TimerGameCalendar::date;
 
		TimerGameCalendar::ConvertDateToYMD(initial_date, &this->date);
 
		this->date = TimerGameCalendar::ConvertDateToYMD(initial_date);
 
		this->date.year = Clamp(this->date.year, min_year, max_year);
 
	}
 

	
src/engine.cpp
Show inline comments
 
@@ -663,8 +663,7 @@ void SetYearEngineAgingStops()
 
		if (e->type == VEH_TRAIN && e->u.rail.railveh_type == RAILVEH_WAGON) continue;
 

	
 
		/* Base year ending date on half the model life */
 
		TimerGameCalendar::YearMonthDay ymd;
 
		TimerGameCalendar::ConvertDateToYMD(ei->base_intro + (ei->lifelength.base() * CalendarTime::DAYS_IN_LEAP_YEAR) / 2, &ymd);
 
		TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(ei->base_intro + (ei->lifelength.base() * CalendarTime::DAYS_IN_LEAP_YEAR) / 2);
 

	
 
		_year_engine_aging_stops = std::max(_year_engine_aging_stops, ymd.year);
 
	}
src/landscape.cpp
Show inline comments
 
@@ -609,8 +609,7 @@ byte GetSnowLine()
 
{
 
	if (_snow_line == nullptr) return _settings_game.game_creation.snow_line_height;
 

	
 
	TimerGameCalendar::YearMonthDay ymd;
 
	TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date, &ymd);
 
	TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date);
 
	return _snow_line->table[ymd.month][ymd.day];
 
}
 

	
src/network/network_gui.cpp
Show inline comments
 
@@ -381,8 +381,7 @@ protected:
 
			if (const NWidgetBase *nwid = this->GetWidget<NWidgetBase>(WID_NG_DATE); nwid->current_x != 0) {
 
				/* current date */
 
				Rect date = nwid->GetCurrentRect();
 
				TimerGameCalendar::YearMonthDay ymd;
 
				TimerGameCalendar::ConvertDateToYMD(cur_item->info.game_date, &ymd);
 
				TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(cur_item->info.game_date);
 
				SetDParam(0, ymd.year);
 
				DrawString(date.left, date.right, y + text_y_offset, STR_JUST_INT, TC_BLACK, SA_HOR_CENTER);
 
			}
 
@@ -390,9 +389,8 @@ protected:
 
			if (const NWidgetBase *nwid = this->GetWidget<NWidgetBase>(WID_NG_YEARS); nwid->current_x != 0) {
 
				/* number of years the game is running */
 
				Rect years = nwid->GetCurrentRect();
 
				TimerGameCalendar::YearMonthDay ymd_cur, ymd_start;
 
				TimerGameCalendar::ConvertDateToYMD(cur_item->info.game_date, &ymd_cur);
 
				TimerGameCalendar::ConvertDateToYMD(cur_item->info.start_date, &ymd_start);
 
				TimerGameCalendar::YearMonthDay ymd_cur = TimerGameCalendar::ConvertDateToYMD(cur_item->info.game_date);
 
				TimerGameCalendar::YearMonthDay ymd_start = TimerGameCalendar::ConvertDateToYMD(cur_item->info.start_date);
 
				SetDParam(0, ymd_cur.year - ymd_start.year);
 
				DrawString(years.left, years.right, y + text_y_offset, STR_JUST_INT, TC_BLACK, SA_HOR_CENTER);
 
			}
src/newgrf.cpp
Show inline comments
 
@@ -6520,8 +6520,7 @@ bool GetGlobalVariable(byte param, uint3
 
			return true;
 

	
 
		case 0x02: { // detailed date information: month of year (bit 0-7), day of month (bit 8-12), leap year (bit 15), day of year (bit 16-24)
 
			TimerGameCalendar::YearMonthDay ymd;
 
			TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date, &ymd);
 
			TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date);
 
			TimerGameCalendar::Date start_of_year = TimerGameCalendar::ConvertYMDToDate(ymd.year, 0, 1);
 
			*value = ymd.month | (ymd.day - 1) << 8 | (TimerGameCalendar::IsLeapYear(ymd.year) ? 1 << 15 : 0) | (TimerGameCalendar::date - start_of_year).base() << 16;
 
			return true;
src/saveload/oldloader_sl.cpp
Show inline comments
 
@@ -844,8 +844,7 @@ static bool LoadOldIndustry(LoadgameStat
 
			if (i->type > 0x06) i->type++; // Printing Works were added
 
			if (i->type == 0x0A) i->type = 0x12; // Iron Ore Mine has different ID
 

	
 
			TimerGameCalendar::YearMonthDay ymd;
 
			TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date, &ymd);
 
			TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date);
 
			i->last_prod_year = ymd.year;
 

	
 
			i->random_colour = RemapTTOColour(i->random_colour);
src/script/api/script_date.cpp
Show inline comments
 
@@ -29,8 +29,7 @@
 
{
 
	if (date < 0) return DATE_INVALID;
 

	
 
	::TimerGameCalendar::YearMonthDay ymd;
 
	::TimerGameCalendar::ConvertDateToYMD(date, &ymd);
 
	::TimerGameCalendar::YearMonthDay ymd = ::TimerGameCalendar::ConvertDateToYMD(date);
 
	return ymd.year.base();
 
}
 

	
 
@@ -38,8 +37,7 @@
 
{
 
	if (date < 0) return DATE_INVALID;
 

	
 
	::TimerGameCalendar::YearMonthDay ymd;
 
	::TimerGameCalendar::ConvertDateToYMD(date, &ymd);
 
	::TimerGameCalendar::YearMonthDay ymd = ::TimerGameCalendar::ConvertDateToYMD(date);
 
	return ymd.month + 1;
 
}
 

	
 
@@ -47,8 +45,7 @@
 
{
 
	if (date < 0) return DATE_INVALID;
 

	
 
	::TimerGameCalendar::YearMonthDay ymd;
 
	::TimerGameCalendar::ConvertDateToYMD(date, &ymd);
 
	::TimerGameCalendar::YearMonthDay ymd = ::TimerGameCalendar::ConvertDateToYMD(date);
 
	return ymd.day;
 
}
 

	
src/strings.cpp
Show inline comments
 
@@ -483,8 +483,7 @@ static void FormatBytes(StringBuilder &b
 

	
 
static void FormatYmdString(StringBuilder &builder, TimerGameCalendar::Date date, uint case_index)
 
{
 
	TimerGameCalendar::YearMonthDay ymd;
 
	TimerGameCalendar::ConvertDateToYMD(date, &ymd);
 
	TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(date);
 

	
 
	auto tmp_params = MakeParameters(ymd.day + STR_DAY_NUMBER_1ST - 1, STR_MONTH_ABBREV_JAN + ymd.month, ymd.year);
 
	FormatString(builder, GetStringPtr(STR_FORMAT_DATE_LONG), tmp_params, case_index);
 
@@ -492,8 +491,7 @@ static void FormatYmdString(StringBuilde
 

	
 
static void FormatMonthAndYear(StringBuilder &builder, TimerGameCalendar::Date date, uint case_index)
 
{
 
	TimerGameCalendar::YearMonthDay ymd;
 
	TimerGameCalendar::ConvertDateToYMD(date, &ymd);
 
	TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(date);
 

	
 
	auto tmp_params = MakeParameters(STR_MONTH_JAN + ymd.month, ymd.year);
 
	FormatString(builder, GetStringPtr(STR_FORMAT_DATE_SHORT), tmp_params, case_index);
 
@@ -501,8 +499,7 @@ static void FormatMonthAndYear(StringBui
 

	
 
static void FormatTinyOrISODate(StringBuilder &builder, TimerGameCalendar::Date date, StringID str)
 
{
 
	TimerGameCalendar::YearMonthDay ymd;
 
	TimerGameCalendar::ConvertDateToYMD(date, &ymd);
 
	TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(date);
 

	
 
	/* Day and month are zero-padded with ZEROFILL_NUM, hence the two 2s. */
 
	auto tmp_params = MakeParameters(ymd.day, 2, ymd.month + 1, 2, ymd.year);
src/subsidy_gui.cpp
Show inline comments
 
@@ -142,8 +142,7 @@ struct SubsidyListWindow : Window {
 
	{
 
		if (widget != WID_SUL_PANEL) return;
 

	
 
		TimerGameCalendar::YearMonthDay ymd;
 
		TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date, &ymd);
 
		TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date);
 

	
 
		Rect tr = r.Shrink(WidgetDimensions::scaled.framerect);
 

	
src/survey.cpp
Show inline comments
 
@@ -311,8 +311,7 @@ void SurveyTimers(nlohmann::json &survey
 
	survey["ticks"] = TimerGameTick::counter;
 
	survey["seconds"] = std::chrono::duration_cast<std::chrono::seconds>(std::chrono::steady_clock::now() - _switch_mode_time).count();
 

	
 
	TimerGameCalendar::YearMonthDay ymd;
 
	TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date, &ymd);
 
	TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date);
 
	survey["calendar"] = fmt::format("{:04}-{:02}-{:02} ({})", ymd.year, ymd.month + 1, ymd.day, TimerGameCalendar::date_fract);
 
}
 

	
src/timer/timer_game_calendar.cpp
Show inline comments
 
@@ -66,9 +66,9 @@ static const uint16_t _accum_days_for_mo
 
/**
 
 * Converts a Date to a Year, Month & Day.
 
 * @param date the date to convert from
 
 * @param ymd  the year, month and day to write to
 
 * @returns YearMonthDay representation of the Date.
 
 */
 
/* static */ void TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::Date date, TimerGameCalendar::YearMonthDay *ymd)
 
/* static */ TimerGameCalendar::YearMonthDay TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::Date date)
 
{
 
	/* Year determination in multiple steps to account for leap
 
	 * years. First do the large steps, then the smaller ones.
 
@@ -77,7 +77,6 @@ static const uint16_t _accum_days_for_mo
 
	/* There are 97 leap years in 400 years */
 
	TimerGameCalendar::Year yr = 400 * (date.base() / (CalendarTime::DAYS_IN_YEAR * 400 + 97));
 
	int rem = date.base() % (CalendarTime::DAYS_IN_YEAR * 400 + 97);
 
	uint16_t x;
 

	
 
	if (rem >= CalendarTime::DAYS_IN_YEAR * 100 + 25) {
 
		/* There are 25 leap years in the first 100 years after
 
@@ -110,11 +109,13 @@ static const uint16_t _accum_days_for_mo
 
	/* Skip the 29th of February in non-leap years */
 
	if (!TimerGameCalendar::IsLeapYear(yr) && rem >= ACCUM_MAR - 1) rem++;
 

	
 
	ymd->year = yr;
 
	uint16_t x = _month_date_from_year_day[rem];
 

	
 
	x = _month_date_from_year_day[rem];
 
	ymd->month = x >> 5;
 
	ymd->day = x & 0x1F;
 
	YearMonthDay ymd;
 
	ymd.year = yr;
 
	ymd.month = x >> 5;
 
	ymd.day = x & 0x1F;
 
	return ymd;
 
}
 

	
 
/**
 
@@ -153,11 +154,9 @@ static const uint16_t _accum_days_for_mo
 
{
 
	assert(fract < Ticks::DAY_TICKS);
 

	
 
	TimerGameCalendar::YearMonthDay ymd;
 

	
 
	TimerGameCalendar::date = date;
 
	TimerGameCalendar::date_fract = fract;
 
	TimerGameCalendar::ConvertDateToYMD(date, &ymd);
 
	TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(date);
 
	TimerGameCalendar::year = ymd.year;
 
	TimerGameCalendar::month = ymd.month;
 
}
 
@@ -195,8 +194,7 @@ void TimerManager<TimerGameCalendar>::El
 
	/* increase day counter */
 
	TimerGameCalendar::date++;
 

	
 
	TimerGameCalendar::YearMonthDay ymd;
 
	TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date, &ymd);
 
	TimerGameCalendar::YearMonthDay ymd = TimerGameCalendar::ConvertDateToYMD(TimerGameCalendar::date);
 

	
 
	/* check if we entered a new month? */
 
	bool new_month = ymd.month != TimerGameCalendar::month;
src/timer/timer_game_calendar.h
Show inline comments
 
@@ -101,7 +101,7 @@ public:
 
	};
 

	
 
	static bool IsLeapYear(Year yr);
 
	static void ConvertDateToYMD(Date date, YearMonthDay * ymd);
 
	static YearMonthDay ConvertDateToYMD(Date date);
 
	static Date ConvertYMDToDate(Year year, Month month, Day day);
 
	static void SetDate(Date date, DateFract fract);
 

	
0 comments (0 inline, 0 general)