[vim/vim] runtime: Recognise ".cppm", etc. files as C++ 20 Modules (PR #12261)

47 views
Skip to first unread message

Ben Jackson

unread,
Apr 14, 2023, 7:18:33 AM4/14/23
to vim/vim, Subscribed

clang's c++ 20 modules uses the "cppm" and similiar extensions: https://clang.llvm.org/docs/StandardCPlusPlusModules.html#file-name-requirement

Recognise .cppm, .ccm, .cxxm and .c++m as a C++ file (C++ 20 Module)


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

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

Commit Summary

  • 0848af0 runtime: Recognise ".cppm", etc. files as C++ 20 Modules

File Changes

(1 file)

Patch Links:


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/12261@github.com>

codecov[bot]

unread,
Apr 14, 2023, 7:29:49 AM4/14/23
to vim/vim, Subscribed

Codecov Report

Merging #12261 (0848af0) into master (b7f6f93) will decrease coverage by 0.82%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12261      +/-   ##
==========================================
- Coverage   81.94%   81.13%   -0.82%     
==========================================
  Files         164      154      -10     
  Lines      194102   183642   -10460     
  Branches    43834    41405    -2429     
==========================================
- Hits       159066   149003   -10063     
+ Misses      22209    21691     -518     
- Partials    12827    12948     +121     
Flag Coverage Δ
huge-clang-none 82.67% <ø> (+0.01%) ⬆️
huge-gcc-none ?
huge-gcc-testgui ?
huge-gcc-unittests 0.29% <ø> (ø)
linux 81.13% <ø> (-1.25%) ⬇️
mingw-x64-HUGE ?
mingw-x86-HUGE ?
windows ?

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

see 148 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/12261/c1508365810@github.com>

Bram Moolenaar

unread,
Apr 15, 2023, 11:48:40 AM4/15/23
to vim/vim, Subscribed

Please add entries in src/testdir/test_filetype.vim


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/12261/c1509875684@github.com>

Ben Jackson

unread,
Apr 15, 2023, 11:49:24 AM4/15/23
to vim/vim, Subscribed

Yes sure thing will do!


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/12261/c1509875825@github.com>

Ben Jackson

unread,
Apr 15, 2023, 11:52:05 AM4/15/23
to vim/vim, Push

@puremourning pushed 1 commit.

  • 8ce80d0 runtime: Recognise ".cppm", etc. files as C++ 20 Modules


View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <vim/vim/pull/12261/push/13311235668@github.com>

Ben Jackson

unread,
Apr 15, 2023, 11:52:23 AM4/15/23
to vim/vim, Subscribed

Please add entries in src/testdir/test_filetype.vim

That's done.


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/12261/c1509877064@github.com>

Bram Moolenaar

unread,
Apr 15, 2023, 1:18:01 PM4/15/23
to vim/vim, Subscribed

Closed #12261 via 732d69e.


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/12261/issue_event/9014735649@github.com>

Reply all
Reply to author
Forward
0 new messages