Use Section::typeInfo() more

This commit is contained in:
ISSOtm
2026-09-17 13:19:26 -04:00
parent 3b250d5386
commit 0b9f8ab523
3 changed files with 7 additions and 8 deletions
+2 -2
View File
@@ -263,7 +263,7 @@ void layout_PlaceSection(std::string const &name, bool isOptional) {
"\"%s\" is specified to be a %s section, but it is already a %s section", "\"%s\" is specified to be a %s section, but it is already a %s section",
name.c_str(), name.c_str(),
typeInfo.name.c_str(), typeInfo.name.c_str(),
sectionTypeInfo[section->type].name.c_str() section->typeInfo().name.c_str()
); );
} }
@@ -276,7 +276,7 @@ void layout_PlaceSection(std::string const &name, bool isOptional) {
"The linker script places section \"%s\" in %s bank %" PRIu32 "The linker script places section \"%s\" in %s bank %" PRIu32
", but it was already defined in bank %" PRIu32, ", but it was already defined in bank %" PRIu32,
name.c_str(), name.c_str(),
sectionTypeInfo[section->type].name.c_str(), section->typeInfo().name.c_str(),
bank, bank,
section->bank section->bank
); );
+2 -3
View File
@@ -855,7 +855,6 @@ void sdobj_ReadFile(FileStackNode const &src, FILE *file, std::vector<Symbol> &f
// RAM sections can have a size, but don't get any data (they shouldn't have any) // RAM sections can have a size, but don't get any data (they shouldn't have any)
if (section->type != SECTTYPE_INVALID) { if (section->type != SECTTYPE_INVALID) {
SectionTypeInfo const &typeInfo = sectionTypeInfo[section->type];
// Otherwise, how would the type already be known at this point? // Otherwise, how would the type already be known at this point?
assume(section->isAddressFixed); assume(section->isAddressFixed);
@@ -866,7 +865,7 @@ void sdobj_ReadFile(FileStackNode const &src, FILE *file, std::vector<Symbol> &f
"\"%s\" is implicitly defined as a %s section (being at address $%04" PRIx16 "\"%s\" is implicitly defined as a %s section (being at address $%04" PRIx16
"), but it has data! (Was a bad `__at()` value used?)", "), but it has data! (Was a bad `__at()` value used?)",
section->name.c_str(), section->name.c_str(),
typeInfo.name.c_str(), section->typeInfo().name.c_str(),
section->org section->org
); );
} }
@@ -876,7 +875,7 @@ void sdobj_ReadFile(FileStackNode const &src, FILE *file, std::vector<Symbol> &f
"\"%s\" is implicitly defined as a %s section (being at address $%04" PRIx16 "\"%s\" is implicitly defined as a %s section (being at address $%04" PRIx16
"), but it doesn't have any data! (Was a bad `__at()` value used?)", "), but it doesn't have any data! (Was a bad `__at()` value used?)",
section->name.c_str(), section->name.c_str(),
typeInfo.name.c_str(), section->typeInfo().name.c_str(),
section->org section->org
); );
} }
+3 -3
View File
@@ -115,8 +115,8 @@ static void mergeSections(Section &target, std::unique_ptr<Section> &&other) {
*other, *other,
"Section \"%s\" is defined with type `%s`, but also with type `%s`", "Section \"%s\" is defined with type `%s`, but also with type `%s`",
target.name.c_str(), target.name.c_str(),
sectionTypeInfo[target.type].name.c_str(), target.typeInfo().name.c_str(),
sectionTypeInfo[other->type].name.c_str() other->typeInfo().name.c_str()
); );
} }
@@ -190,7 +190,7 @@ void sect_AddSection(std::unique_ptr<Section> &&section) {
fatal( fatal(
"Section \"%s\" is of type `%s`, which cannot be `UNION`ized", "Section \"%s\" is of type `%s`, which cannot be `UNION`ized",
section->name.c_str(), section->name.c_str(),
sectionTypeInfo[section->type].name.c_str() section->typeInfo().name.c_str()
); );
} else { } else {
sections.add(section->name, std::move(section)); sections.add(section->name, std::move(section));