Modifying a bit weighting of completion list SmartMatch

[elsassph]

Fixes issue #1161

Note: collateral damage is that abbreviation matching is now only enabled for all-uppercase words.

[Meychi]

@Neverbirth, @SlavaRa, @wise0704, comments?

[Neverbirth]

I think I completely missed this PR until now... not a part of code I really know. I'd need to try out how the completion feel with the abbreviation change, by looking at the changes it seems to me the issue is already fixed without that behaviour change?

[wise0704]

Since the method is public, I think you need to add a method with the old signature too...

Something like:

public static int SmartMatch(string label, string word, int len)
{
    return SmartMatch(label, word, len, word == word.ToUpperInvariant());
}

[elsassph]

It needs unit tests as well, because the weighting was changed a bit randomly.

[Meychi]

@elsassph: Will you add tests here or should we merge as such?

[elsassph]

We shouldn't merge without tests I believe. I'll try to do it quickly.

[Meychi]

@elsassph, any progress? :)

[elsassph]

Well... I haven't been able to run tests at all so far :/

[Meychi]

Any progress?