Skip to content

fix(nvimcom.c): ensure compatibility with R version 4.4.0 and above#561

Merged
jalvesaq merged 4 commits intomainfrom
fix/compilation-error
Apr 27, 2026
Merged

fix(nvimcom.c): ensure compatibility with R version 4.4.0 and above#561
jalvesaq merged 4 commits intomainfrom
fix/compilation-error

Conversation

@PMassicotte
Copy link
Copy Markdown
Collaborator

Add conditional compilation to use Rf_allocLang for R 4.4.0+. Maintain backward compatibility with older R versions using Rf_allocList.

Add conditional compilation to use Rf_allocLang for R 4.4.0+.
Maintain backward compatibility with older R versions using Rf_allocList.
@PMassicotte PMassicotte marked this pull request as ready for review April 27, 2026 11:21
@jalvesaq
Copy link
Copy Markdown
Member

I pushed two commits, but didn't change the required R version...

@PMassicotte
Copy link
Copy Markdown
Collaborator Author

This is all good on my side!

@jalvesaq
Copy link
Copy Markdown
Member

Let's wait for @ClemensKohl's confirmation that this pull request fixes the bug...

@ClemensKohl
Copy link
Copy Markdown

I just tried it out, and it installed without a problem! Thanks a lot for looking into it!

@jalvesaq jalvesaq merged commit 989437b into main Apr 27, 2026
11 checks passed
@jalvesaq jalvesaq deleted the fix/compilation-error branch April 27, 2026 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants