neovim icon indicating copy to clipboard operation
neovim copied to clipboard

Published 20 hours ago • verified publisher icon neovim

docs

Open justinmk opened this issue 2 years ago • 3 comments

todo

  • [x] fix :trust doc. ref
  • [ ] nvim requires rpc responses in order. ref
  • [ ] NVIM_APPNAME
    • cgroups blurb
    • UIs normally should NOT set this

justinmk avatar Feb 22 '23 15:02 justinmk

I hate that github doesn't allow pointing to code that wasn't touched by the PR.

Anyway, there's a line in dev_style.txt that we concluded was kind of silly and should be removed: "Do not use filenames that already exist in /usr/include, such as db.h." I forgot to remove it since it was so small, but you could do it in this PR perhaps.

dundargoc avatar Mar 02 '23 09:03 dundargoc

"Do not use filenames that already exist in /usr/include, such as db.h."

Why is it silly? That idea was recently applied in e.g. https://github.com/neovim/neovim/pull/22442/commits/b68c020a9c7f51b905bf57c4051ab5198efc66b0

justinmk avatar Mar 02 '23 12:03 justinmk

It is mostly relevant for system headers we actually use (because uncrustify otherways gets into a fit, like in the referenced commit), but it is good style anyway to avoid libc/POSIX header names I guess.

bfredl avatar Mar 04 '23 21:03 bfredl