github/android-dev-ext
1
1
Fork 0
mirror of https://github.com/adelphes/android-dev-ext.git synced 2025-12-23 01:48:18 +00:00
Code Issues 41 Packages Projects Releases Wiki Activity

add more info when methods/ctrs cannot be matched

Browse Source
This commit is contained in:
Dave Holoway
2020-06-06 13:33:06 +01:00
parent 432cd24f3c
commit 13068cbcea
1 changed files with 13 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
langserver/java/body-parser3.js
View File

@@ -1573,13 +1573,22 @@ function methodCallExpression(tokens, instance, call_arguments, typemap) {
}); });
if (!types[0] && !methods[0]) { if (!types[0] && !methods[0]) {
const callargtypes = call_arguments.map(a => a.variables[0] ? a.variables[0].type.fullyDottedTypeName : '<unknown-type>').join(', ');
if (instance.methods[0]) { if (instance.methods[0]) {
addproblem(tokens, ParseProblem.Error(tokens.current, `No compatible method found. Tried to match:\n${instance.methods.map(m => m.label).join('\n')}`)) const methodlist = instance.methods.map(m => m.label).join('\n- ');
addproblem(tokens, ParseProblem.Error(tokens.current,
`No compatible method found. Tried to match:\n- ${methodlist}\nagainst call argument types: (${callargtypes})`))
// fake a result with AnyMethod
methods.push(new AnyMethod(instance.source));
} else if (instance.types[0]) { } else if (instance.types[0]) {
const ctrlist = instance.types[0].constructors.map(c => c.label).join('\n- ');
const match_message = instance.types[0].constructors.length const match_message = instance.types[0].constructors.length
? `Tried to match:\n${instance.types[0].constructors.map(c => c.label).join('\n')}` ? `Tried to match:\n- ${ctrlist}\nagainst call argument types: (${callargtypes})`
: 'The type has no accessible constructors'; : 'The type has no accessible constructors';
addproblem(tokens, ParseProblem.Error(tokens.current, `No compatible constructor found for type '${instance.types[0].fullyDottedTypeName}'. ${match_message}`)) addproblem(tokens, ParseProblem.Error(tokens.current,
`No compatible constructor found for type '${instance.types[0].fullyDottedTypeName}'. ${match_message}`));
// fake a result with AnyType
types.push(new AnyType(instance.source));
} }
} }
@@ -1975,7 +1984,7 @@ function resolveTypeOrPackage(ident, scoped_type, imports, typemap) {
/** /**
* AnyType is a special type that's used to fill in types that are missing. * AnyType is a special type that's used to fill in types that are missing.
* To prevent cascading errors, AnyType should be fully assign/case/type-compatible * To prevent cascading errors, AnyType should be fully assign/cast/type-compatible
* with any other type * with any other type
*/ */
class AnyType extends JavaType { class AnyType extends JavaType {