Fork me on GitHub
#editors
<
2016-02-25
>
juhoteperi13:02:27

@dominicm: I'll do PR soon with fixes and improvements

dominicm13:02:42

@juhoteperi: I figured something like that would be possible. I think npm's model works best. Thanks! I'm using it in a project today, so I'll get a few things too I hope. My python-fu isn't as good as it used to be. I need to come up with safer key lookup systems. But it's just so many if/else's that I didn't want to worry about.

dominicm13:02:59

Majority of it works quite nicely though, I'm rather happy with it.

juhoteperi13:02:29

@dominicm: I'll fix key lookups and add support for additional data for candidates (arglist, type etc.)

dominicm13:02:15

@juhoteperi: Awesome, thank you! I hadn't even started thinking about arglist/type stuff. I look forward to seeing that.

dominicm15:02:36

@juhoteperi: I should apologize for some of my bad code in there, I was getting really confused about things that were breaking, and then magically fixed when I changed to doing weird things (like that r variable..). I'm not sure at all what did fix those things in the end. 😛

dominicm15:02:33

This is what I call "I have a train coming in 5 minutes, do whatever gets the code to run" debugging.

juhoteperi15:02:11

@dominicm: I think source shouldn't set min_pattern_length as it looks like that will override user auto_completion_start_length (or could be bug in deoplete)

juhoteperi15:02:43

but I think other sources don't set it?

juhoteperi15:02:54

fixed in the PR simple_smile

dominicm15:02:14

@juhoteperi: No, most don't.. ah, that was it, to do with the regex value whilst I was debugging. Awesome, thank you.

dominicm15:02:10

I've loaded your patches into local, I'll give them a whizz, then accept simple_smile

juhoteperi15:02:57

Looks like fireplace could just embed this deoplete source and it would get used when deoplete is available

juhoteperi15:02:13

clang_complete does that, and it should also work with regular Vim

dominicm16:02:45

It's a possibility. I have no idea if that aligns with the pope's view of fireplace though. We can bring it to his attention & see what he thinks though.

dominicm16:02:12

@juhoteperi: No idea if you're using your changes, but they enable the docstring coming up whilst you're typing. That's awesome, I didn't think we'd be able to achieve feature parity with the omnicompletion (I hadn't checked out deoplete's features yet). One minor annoyance; keywords put 0 into the scratch buffer. But I'll take a look at the best solution for that a little later.

dominicm16:02:21

This is super cool.

juhoteperi16:02:22

I have docstring preview disabled (I don't like the preview window - not sure if Neovim has some alternatives)

juhoteperi16:02:28

What is scratch buffer?

dominicm16:02:12

I don't like it much either. But I'm glad that we have it, just for the sake of matching. scratch buffer = where docstring preview shows.

juhoteperi16:02:31

Aha. Fixed it, val.get("doc") returned None. Using empty string as default works.

dominicm16:02:38

Oh, an empty string works? Neat 😛 I just merged your PR. Sorry 😛

juhoteperi16:02:55

I had already pushed the fix so it was merged

dominicm16:02:21

Oh, well timed!