1berry/pawn-compiler
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Disable warnings 250 and 251 when a global variable is used inside a loop condition

Browse Source
This commit is contained in:
Stanislav Gromov 2021-01-30 21:20:25 +07:00
parent ae30cfb20b
commit bff030799b
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
source/compiler/sc2.c
View File

@ -3309,8 +3309,7 @@ SC_FUNC void markusage(symbol *sym,int usage)
sym->lnumber=fline; sym->lnumber=fline;
if ((usage & uREAD)!=0 && (sym->ident==iVARIABLE || sym->ident==iREFERENCE)) if ((usage & uREAD)!=0 && (sym->ident==iVARIABLE || sym->ident==iREFERENCE))
sym->usage &= ~uASSIGNED; sym->usage &= ~uASSIGNED;
if ((usage & (uREAD | uWRITTEN))!=0 && (sym->vclass==sLOCAL || sym->vclass==sSTATIC) if ((usage & (uREAD | uWRITTEN))!=0 && (sym->ident==iVARIABLE || sym->ident==iREFERENCE))
&& (sym->ident==iVARIABLE || sym->ident==iREFERENCE))
markloopvariable(sym,usage); markloopvariable(sym,usage);
/* check if (global) reference must be added to the symbol */ /* check if (global) reference must be added to the symbol */
if ((usage & (uREAD | uWRITTEN))!=0) { if ((usage & (uREAD | uWRITTEN))!=0) {
@ -3421,11 +3420,20 @@ SC_FUNC void restoreassignments(int fromlevel,symstate *assignments)
static void markloopvariable(symbol *sym,int usage) static void markloopvariable(symbol *sym,int usage)
{ {
if (sc_status!=statWRITE)
return;
while (sym->parent!=NULL) while (sym->parent!=NULL)
sym=sym->parent; sym=sym->parent;
/* check if the variable used inside a loop condition */ /* check if the variable used inside a loop condition */
if (pc_loopcond) { if (pc_loopcond) {
if ((usage & uWRITTEN)!=0) { if (sym->vclass==sGLOBAL) {
/* stop counting variables that were used in loop condition, otherwise
* warnings 250 and 251 may be inaccurate (global variables can be
* modified from another function(s) called from the loop body, and
* currently there's no reasonable way to track this) */
pc_loopcond=FALSE;
pc_numloopvars=0;
} else if ((usage & uWRITTEN)!=0) {
/* the symbol is being modified inside a loop condition before being read; /* the symbol is being modified inside a loop condition before being read;
* set the uNOLOOPVAR flag, so later we'll know we shouldn't mark the symbol * set the uNOLOOPVAR flag, so later we'll know we shouldn't mark the symbol
* with the uLOOPVAR flag */ * with the uLOOPVAR flag */