Don't issue MissingAbstractMethodImpl warnings unless WholeProgram
Local linting just looks at the files that were changed, so we dont always see the implementation of an abstract method. Disable the warning in that case. We'll still catch it when we do whole program linting later.
Esse commit está contido em:
@@ -663,7 +663,7 @@ void AnalysisResult::analyzeProgram(bool system /* = false */) {
|
||||
for (StringToFunctionScopePtrMap::const_iterator iterMethod =
|
||||
methods.begin(); iterMethod != methods.end(); ++iterMethod) {
|
||||
FunctionScopePtr func = iterMethod->second;
|
||||
if (!func->hasImpl() && needAbstractMethodImpl) {
|
||||
if (Option::WholeProgram && !func->hasImpl() && needAbstractMethodImpl) {
|
||||
FunctionScopePtr tmpFunc =
|
||||
cls->findFunction(ar, func->getName(), true, true);
|
||||
always_assert(!tmpFunc || !tmpFunc->hasImpl());
|
||||
|
||||
Referência em uma Nova Issue
Bloquear um usuário