bslathi19/cc65
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Function declaration in functions cannot have storage classes other than 'extern'.

Browse Source
This commit is contained in:
acqn
2020-08-06 17:22:19 +08:00
committed by Oliver Schmidt
parent 43efc256f1
commit d68cd90e47
1 changed files with 16 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
src/cc65/locals.c
View File

@@ -426,13 +426,25 @@ static void ParseOneDecl (const DeclSpec* Spec)
/* Read the declaration */ /* Read the declaration */
ParseDecl (Spec, &Decl, DM_NEED_IDENT); ParseDecl (Spec, &Decl, DM_NEED_IDENT);
/* Set the correct storage class for functions */ /* Check if there are any non-extern storage classes set for function
** declarations. The only valid storage class for function declarations
** inside functions is 'extern'.
*/
if ((Decl.StorageClass & SC_FUNC) == SC_FUNC) { if ((Decl.StorageClass & SC_FUNC) == SC_FUNC) {
/* Function prototypes are always external */
if ((Decl.StorageClass & SC_EXTERN) == 0) { /* Check if there are explicitly specified non-external storage classes */
Warning ("Function must be extern"); if ((Spec->Flags & DS_DEF_STORAGE) != DS_DEF_STORAGE &&
(Decl.StorageClass & SC_EXTERN) == 0 &&
(Decl.StorageClass & SC_STORAGEMASK) != 0) {
Error ("Illegal storage class on function");
} }
/* The default storage class could be wrong. Just use 'extern' in all
** cases.
*/
Decl.StorageClass &= ~SC_STORAGEMASK;
Decl.StorageClass |= SC_EXTERN; Decl.StorageClass |= SC_EXTERN;
} }
/* If we don't have a name, this was flagged as an error earlier. /* If we don't have a name, this was flagged as an error earlier.