[vim/vim] Fix c++ float and integer literal syntax highlighting (#8939)

28 views
Skip to first unread message

Dominique Pellé

unread,
Oct 2, 2021, 3:18:24 PM10/2/21
to vim/vim, Subscribed

This PR fixes syntax highlighting of C++ floating point and integer literals, which were not properly highlighted as illustrated in the screenshot below:

cpp-wrong-syntax-highlight


You can view, comment on, or merge this pull request online at:

  https://github.com/vim/vim/pull/8939

Commit Summary

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Dominique Pellé

unread,
Oct 2, 2021, 3:25:25 PM10/2/21
to vim/vim, Subscribed

I should have indicated: I emailed the maintainer of vim/runtime/syntax/cpp.vim (Ken Shan).

codecov[bot]

unread,
Oct 2, 2021, 3:26:06 PM10/2/21
to vim/vim, Subscribed

Codecov Report

Merging #8939 (f343267) into master (d85931e) will decrease coverage by 87.62%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@

##           master    #8939       +/-   ##

===========================================

- Coverage   90.08%    2.45%   -87.63%     

===========================================

  Files         151      149        -2     

  Lines      168894   165905     -2989     

===========================================

- Hits       152145     4077   -148068     

- Misses      16749   161828   +145079     
Flag Coverage Δ
huge-clang-none ?
huge-gcc-none ?
huge-gcc-testgui ?
huge-gcc-unittests 2.45% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/float.c 0.00% <0.00%> (-99.22%) ⬇️
src/gui_gtk_f.c 0.00% <0.00%> (-97.43%) ⬇️
src/crypt_zip.c 0.00% <0.00%> (-97.06%) ⬇️
src/cmdhist.c 0.00% <0.00%> (-97.00%) ⬇️
src/sha256.c 0.00% <0.00%> (-96.94%) ⬇️
src/evalbuffer.c 0.00% <0.00%> (-96.88%) ⬇️
src/textprop.c 0.00% <0.00%> (-96.84%) ⬇️
src/match.c 0.00% <0.00%> (-96.76%) ⬇️
src/libvterm/src/rect.h 0.00% <0.00%> (-96.56%) ⬇️
src/evalfunc.c 0.00% <0.00%> (-96.51%) ⬇️
... and 137 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d85931e...f343267. Read the comment docs.

Bram Moolenaar

unread,
Oct 3, 2021, 3:02:16 PM10/3/21
to vim/vim, Subscribed

Ken Shan is no longer the maintainer, see https://github.com/vim-jp/vim-cpp

Bram Moolenaar

unread,
Nov 15, 2021, 2:41:25 PM11/15/21
to vim/vim, Subscribed

Did this get picked up by vim-jp?

K.Takata

unread,
Nov 15, 2021, 7:31:32 PM11/15/21
to vim/vim, Subscribed

Not yet.
@dpelle Could you create a PR at vim-jp/vim-cpp?

Dominique Pellé

unread,
Jan 20, 2023, 4:36:41 PM1/20/23
to vim/vim, Subscribed

Closed #8939.


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/8939/issue_event/8325351697@github.com>

Dominique Pellé

unread,
Jan 20, 2023, 4:36:42 PM1/20/23
to vim/vim, Subscribed

Long overdue, but I created PR vim-jp/vim-cpp#63

Closing this one now.


Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/8939/c1398980168@github.com>

Reply all
Reply to author
Forward
0 new messages