Don't highlight "builtins" that appear as attribute names about python-syntax HOT 6 CLOSED

I added this fix into this file to test for myself, and it seems to mess up the formatting of float literals, (e.g., with 100.5, the "100" part will be Normal and the ".5" part will be Number). Any thoughts on how to fix this? I'm not that great with regex.

from python-syntax.

Actually managed to fix it by changing it to
syn match pythonBtInIsolator "\(\.\)\@<!\<\k\+\D\>" contains=pythonBuiltinFunc,pythonStatement
(added \D to the keyword search)

from python-syntax.

@jevogel Can you create a pull request?

However it seems containedin=pythonBtInIsolator is redundant and can be removed. And the regexp probably also can be simplified.

from python-syntax.

So I've tried a simplified version of the patch. It works but it breaks highlighting for numbers even with the fix from @jevogel. For example complex numbers, hexadecimal numbers and error checking for numbers don't work. Also I don't really like to use \@<! which can be really slow. So I think it's a bad idea to apply the patch.

from python-syntax.

Can you detail in which aspects complex, hexadecimals and number error checking are broken? They seem to work fine for me with @jevogel's patch. Performance can be improved by only doing 1-character lookbehind, "\(\.\)\@1<!\<\K\+\>". (\K = \k but excluding digits)

from python-syntax.

@anntzer try my fork https://github.com/purpleP/python-syntax.git

from python-syntax.