Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix error on empty scope #107

Merged
merged 1 commit into from
Oct 8, 2015
Merged

Conversation

mattn
Copy link
Contributor

@mattn mattn commented Oct 8, 2015

When start completion at below's location, it occor errors.

@wsdjeg
Copy link
Contributor

wsdjeg commented Oct 8, 2015

can you open a new issue like VundleVim/Vundle.vim#608

@artur-shaik
Copy link
Owner

@mattn you mean on Test.| completion call?

@mattn
Copy link
Contributor Author

mattn commented Oct 8, 2015

@artur-shaik leading with this.

@wsdjeg
Copy link
Contributor

wsdjeg commented Oct 8, 2015

here the plugin now can complete like this.|

@mattn
Copy link
Contributor Author

mattn commented Oct 8, 2015

@mattn
Copy link
Contributor Author

mattn commented Oct 8, 2015

@artur-shaik I'm not sure this change is best way. so plesae try to fix your self.

@wsdjeg
Copy link
Contributor

wsdjeg commented Oct 8, 2015

but if like this wikk miss uncompile file

@mattn
Copy link
Contributor Author

mattn commented Oct 8, 2015

No, this is not uncomplete.

1 complete v

this.v;

2 add private String foo = at before

private String foo = this.v;

This is strict code.

artur-shaik added a commit that referenced this pull request Oct 8, 2015
@artur-shaik artur-shaik merged commit db05680 into artur-shaik:master Oct 8, 2015
@wsdjeg
Copy link
Contributor

wsdjeg commented Oct 8, 2015

i mean the file which has not been compile!
for example:
Foo.java

public class Foo{
public String a;
}

Foo2.java

public class Foo2 extends Foo{
//case one
|
//case two
this.|
}

@mattn mattn deleted the fix-error2 branch October 8, 2015 09:47
@mattn
Copy link
Contributor Author

mattn commented Oct 8, 2015

Sorry I don't still understand what you mean. Probably it's an another issue. So please file new issue.

@wsdjeg
Copy link
Contributor

wsdjeg commented Oct 8, 2015

i have opened thank you all the same

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants