search

siesta-project / siesta-vim

vim syntax highlighting scheme for siesta
3 stars 2 forks source link

Boolean also accepts y/n #3

Closed zerothi closed 7 years ago

zerothi commented 8 years ago

Perhaps this should be added for completeness sake.

Would changing:

syn match siestaBoolean "\(\s\|^\)[tf]\($\|\s\)"

to

syn match siestaBoolean "\(\s\|^\)[tfyn]\($\|\s\)"

suffice?

arthurfl commented 8 years ago

It would! However n is already recognized as a unit (force), with

syn match siestaUnits "\(\s\|^\)[jkgmsnced]\($\|\s\)"

So not sure how to handle that yet.. ideally, keywords expecting a boolean as argument should be differentiated so that there is some level of information on what will (can) be the argument, but that would add some level of complexity

zerothi commented 8 years ago

Ok, no worries, I think very few are using y/n anyway :)

2016-04-15 16:34 GMT+02:00 arthurfl notifications@github.com:

It would! However n is already recognized as a unit (force), with

syn match siestaUnits "(\s|^)jkgmsnced"

So not sure how to handle that yet.. ideally, keywords expecting a boolean as argument should be differentiated so that there is some level of information on what will (can) be the argument, but that would add some level of complexity

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/siesta-project/siesta-vim/issues/3#issuecomment-210484989

Kind regards Nick