The current nvi port still contains a lot of bugs related to wide character support that is enabled by the unicode-useflag. (For example, using the '#' command crashes nvi even in all-ASCII files.)
I noticed Sven Verdoolaege's Git repository (see: http://repo.or.cz/w/nvi.git) already contains fixes for a lot of these issues; I've collected the relevant changes into a big patch file that I'll attach.
Would a developer please give this a quick look at this and then update the ebuild to apply this patch?
Steps to Reproduce:
Created attachment 345646 [details, diff]
Patch to fix several --enable-widechar bugs
(In reply to Maks Verver from comment #1)
> Created attachment 345646 [details, diff] [details, diff]
> Patch to fix several --enable-widechar bugs
Thanks for taking the time and doing this. However, this is a major patch and I wonder how much have you actually tested it. Do you use NVI regularly? for how long have you used it with this patch?
I'd probably release a masked version of NVI with this patch an see if it doesn't break anything else.
I've been using the patched version for half a year without problems, and although I do use vi quite a lot, I cannot guarantee my typical usage patterns provide adequate test coverage.
Since the current version in portage has known problems, I would say it makes more sense to add the patch and deal with whatever problems arise than to keep a known-buggy version around. The changes in the patch seem sensible enough, so I wouldn't expect too many complications.
Felix, you're the maintainer of this package. Have you reviewed this patch?
I could not reproduce the report. Please provide the requested information.
felix, could you reproduce it?