Extern/Beef
1
0
Fork 0
mirror of https://github.com/beefytech/Beef.git synced 2025-06-19 00:20:25 +02:00
Code Activity

Properly lock around VerifyTypeName

Browse source
This commit is contained in:
Brian Fiete 2022-05-13 12:06:08 -07:00
parent fb1f538d15
commit a8a02870dc
2 changed files with 25 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

9
IDE/src/ui/BuildPropertiesDialog.bf
View file

@ -282,8 +282,13 @@ namespace IDE.ui
typeName.RemoveFromStart(1);
}
if (!gApp.mBfResolveCompiler.VerifyTypeName(scope String(typeName), -1))
isValid = false;
if (gApp.mBfResolveCompiler?.IsPerformingBackgroundOperation() == false)
{
gApp.mBfResolveSystem.Lock(0);
if (!gApp.mBfResolveCompiler.VerifyTypeName(scope String(typeName), -1))
isValid = false;
gApp.mBfResolveSystem.Unlock();
}
}
}
subItem.mTextColor = isValid ? 0xFFFFFFFF : 0xFFFF8080;

29
IDE/src/ui/TypeWildcardEditWidget.bf
View file

@ -51,12 +51,15 @@ namespace IDE.ui
editText.RemoveToEnd(semiPos);
}
bool isValid;
if (editText.StartsWith("@"))
isValid = true;
else
bool isValid = true;
if (!editText.StartsWith("@"))
{
isValid = gApp.mBfResolveCompiler.VerifyTypeName(editText, cursorPos);
if (gApp.mBfResolveCompiler?.IsPerformingBackgroundOperation() == false)
{
gApp.mBfResolveSystem.Lock(0);
isValid = gApp.mBfResolveCompiler.VerifyTypeName(editText, cursorPos);
gApp.mBfResolveSystem.Unlock();
}
}
for (int ofs < editText.Length)
@ -102,15 +105,19 @@ namespace IDE.ui
typeName.RemoveFromStart(1);
startOfs++;
}
bool isValid = gApp.mBfResolveCompiler.VerifyTypeName(scope String(typeName), -1);
if (!isValid)
if (gApp.mBfResolveCompiler?.IsPerformingBackgroundOperation() == false)
{
for (int ofs < typeName.Length)
gApp.mBfResolveSystem.Lock(0);
bool isValid = gApp.mBfResolveCompiler.VerifyTypeName(scope String(typeName), -1);
if (!isValid)
{
text[@typeName.Pos + startOfs + ofs].mDisplayTypeId = 1;
for (int ofs < typeName.Length)
{
text[@typeName.Pos + startOfs + ofs].mDisplayTypeId = 1;
}
}
}
gApp.mBfResolveSystem.Unlock();
}
}
}