[vim/vim] Fix: Use sysconf() instead of getpagesize() (#2741)

16 views
Skip to first unread message

ichizok

unread,
Mar 22, 2018, 6:13:01 AM3/22/18
to vim/vim, Subscribed

#2735 (comment)

This is fix for 'maxmem' on macOS.
getpagesize() has been dropped from POSIX.


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

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

Commit Summary

  • Use sysconf() instead of getpagesize()

File Changes

Patch Links:


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub

Codecov

unread,
Mar 22, 2018, 6:35:16 AM3/22/18
to vim/vim, Subscribed

Codecov Report

Merging #2741 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2741      +/-   ##
==========================================
+ Coverage      75%      75%   +<.01%     
==========================================
  Files          92       92              
  Lines      134149   134149              
==========================================
+ Hits       100614   100617       +3     
+ Misses      33535    33532       -3
Impacted Files Coverage Δ
src/os_unix.c 54.61% <ø> (+0.13%) ⬆️
src/gui_beval.c 62.66% <0%> (-0.43%) ⬇️
src/message.c 74.56% <0%> (-0.25%) ⬇️
src/gui.c 47.73% <0%> (-0.06%) ⬇️
src/gui_gtk_x11.c 47.63% <0%> (-0.05%) ⬇️
src/terminal.c 56.72% <0%> (-0.05%) ⬇️
src/ex_getln.c 73.29% <0%> (+0.03%) ⬆️
src/term.c 60.21% <0%> (+0.05%) ⬆️
src/misc1.c 84.41% <0%> (+0.06%) ⬆️
... and 1 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 b571c63...e11bf03. Read the comment docs.

Bram Moolenaar

unread,
Mar 22, 2018, 4:44:56 PM3/22/18
to vim/vim, Subscribed

Closed #2741 via 62b7f6a.

Reply all
Reply to author
Forward
0 new messages