Skip to content

Commit 7ac8988

Browse files
committed
vpc: Rollback MASM compilation as it doesn't work well with github ci builds
1 parent 2141aa3 commit 7ac8988

File tree

3 files changed

+5
-53
lines changed

3 files changed

+5
-53
lines changed

utils/vpc/projectgenerator_win32_2010.cpp

Lines changed: 5 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -37,14 +37,13 @@ enum TypeKeyNames_e {
3737
TKN_COMPILE,
3838
TKN_RESOURCECOMPILE,
3939
TKN_CUSTOMBUILD,
40-
TKN_MASM,
4140
TKN_NONE,
4241
TKN_MAX_COUNT,
4342
};
4443

45-
static const char *s_TypeKeyNames[] = {
46-
"Library", "ClInclude", "ClCompile", "ResourceCompile",
47-
"CustomBuild", "MASM", "None"};
44+
static const char *s_TypeKeyNames[] = {"Library", "ClInclude",
45+
"ClCompile", "ResourceCompile",
46+
"CustomBuild", "None"};
4847

4948
const char *CProjectGenerator_Win32_2010::GetKeyNameForFile(
5049
CProjectFile *pFile) {
@@ -60,8 +59,6 @@ const char *CProjectGenerator_Win32_2010::GetKeyNameForFile(
6059
pKeyName = s_TypeKeyNames[TKN_COMPILE];
6160
} else if (IsHFileExtension(pExtension)) {
6261
pKeyName = s_TypeKeyNames[TKN_INCLUDE];
63-
} else if (IsMasmFileExtension(pExtension)) {
64-
pKeyName = s_TypeKeyNames[TKN_MASM];
6562
} else if (!V_stricmp(pExtension, "lib")) {
6663
pKeyName = s_TypeKeyNames[TKN_LIBRARY];
6764
} else if (!V_stricmp(pExtension, "rc")) {
@@ -72,30 +69,6 @@ const char *CProjectGenerator_Win32_2010::GetKeyNameForFile(
7269
return pKeyName;
7370
}
7471

75-
enum FileTypeNames_e {
76-
FTN_DOCUMENT = 0,
77-
FTN_NONE,
78-
FTN_MAX_COUNT,
79-
};
80-
81-
static const char *s_FileTypeNames[] = {"Document", ""};
82-
83-
const char *CProjectGenerator_Win32_2010::GetFileTypeForFile(
84-
CProjectFile *pFile) {
85-
static_assert(V_ARRAYSIZE(s_FileTypeNames) == FTN_MAX_COUNT);
86-
87-
const char *pExtension = V_GetFileExtension(pFile->m_Name.Get());
88-
89-
const char *pKeyName = s_FileTypeNames[FTN_NONE];
90-
if (pExtension) {
91-
if (IsMasmFileExtension(pExtension)) {
92-
pKeyName = s_FileTypeNames[FTN_DOCUMENT];
93-
}
94-
}
95-
96-
return pKeyName;
97-
}
98-
9972
bool CProjectGenerator_Win32_2010::WritePropertyGroupTool(
10073
CProjectTool *pProjectTool, CProjectConfiguration *pConfiguration) {
10174
if (!pProjectTool) return true;
@@ -126,27 +99,13 @@ bool CProjectGenerator_Win32_2010::WriteFile(CProjectFile *pFile,
12699
return true;
127100
}
128101

129-
const char *pFileType = GetFileTypeForFile(pFile);
130-
bool bHasFileType = pFileType && *pFileType;
131-
132102
if (!pFile->m_Configs.Count()) {
133-
if (!bHasFileType) {
134-
m_XMLWriter.Write(
135-
CFmtStrMax("<%s Include=\"%s\" />", pKeyName, pFile->m_Name.Get()));
136-
} else {
137-
m_XMLWriter.PushNode(pKeyName,
138-
CFmtStr("Include=\"%s\"", pFile->m_Name.Get()));
139-
m_XMLWriter.Write(CFmtStrMax("<FileType>%s</FileType>", pFileType));
140-
m_XMLWriter.PopNode(true);
141-
}
103+
m_XMLWriter.Write(
104+
CFmtStrMax("<%s Include=\"%s\" />", pKeyName, pFile->m_Name.Get()));
142105
} else {
143106
m_XMLWriter.PushNode(pKeyName,
144107
CFmtStr("Include=\"%s\"", pFile->m_Name.Get()));
145108

146-
if (bHasFileType) {
147-
m_XMLWriter.Write(CFmtStrMax("<FileType>%s</FileType>", pFileType));
148-
}
149-
150109
for (intp i = 0; i < pFile->m_Configs.Count(); i++) {
151110
if (!WriteConfiguration(pFile->m_Configs[i])) return false;
152111
}

utils/vpc/projectgenerator_win32_2010.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ class CProjectGenerator_Win32_2010 : public IVCProjWriter {
4949
const char *pFileTypeName);
5050

5151
const char *GetKeyNameForFile(CProjectFile *pFile);
52-
const char *GetFileTypeForFile(CProjectFile *pFile);
5352

5453
CXMLWriter m_XMLWriter;
5554
CXMLWriter m_XMLFilterWriter;

utils/vpc/sys_utils.h

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -137,12 +137,6 @@ inline bool IsHFileExtension(const char *pExtension) {
137137
!V_stricmp(pExtension, "hh") || !V_stricmp(pExtension, "h");
138138
}
139139

140-
inline bool IsMasmFileExtension(const char *pExtension) {
141-
if (!pExtension) return false;
142-
143-
return !V_stricmp(pExtension, "asm") || !V_stricmp(pExtension, "masm");
144-
}
145-
146140
CUtlString Sys_GuidFromFileName(const char *szFileName);
147141

148142
#endif // VPC_SYS_UTILS_H_

0 commit comments

Comments
 (0)