text icon indicating copy to clipboard operation
text copied to clipboard

Move Word and WordR function to view.go

Open zoli opened this issue 10 years ago • 3 comments

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/7949392-move-word-and-wordr-function-to-view-go?utm_campaign=plugin&utm_content=tracker%2F7240968&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F7240968&utm_medium=issues&utm_source=github).

zoli avatar Jan 22 '15 05:01 zoli

IMO it'd still be useful to have these functions in buffer for non-lime users. We could create a new WordEx(seps string, offset int) Region function so that view can specify the separators to consider.

quarnster avatar Jan 22 '15 08:01 quarnster

Totally agree, Also I think we should do the same thing for Classify method and create sth like ClassifyEx method for buffer. IMO Word and other similar functions should use Classify method with appropriate classes, but currently Classify is using regex for detecting all classes and its slow so until moving Word function logic to Classify for CLASS_WORD_START and CLASS_WORD_END we shouldn't do that.

zoli avatar Jan 22 '15 11:01 zoli

All excellent points :)

quarnster avatar Jan 22 '15 13:01 quarnster