Warn or error if Rcpp.h (and R.h) was included
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
Piste de recherche
Start in RcppEigen.h and inspect how the Rcpp and R header guards are detected before Eigen is included. Trace the include order described in the issue, then confirm that including R.h or Rcpp.h before RcppEigen.h produces the intended warning without changing the normal direct-include path.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
As discussed over in Rcpp in https://github.com/RcppCore/Rcpp/pull/1402 we are seeing some side effects from some package including R.h and later including possible also Rcpp.h and then RcppEigen.h leaving some of the masking of R's own error() function as Rf_error() to not be effective so that it later clashes with Eigen function error().
In general, it is always cleanest to just include RcppEigen.h as it does take care of both the others. For RcppArmadillo this once lead to issues to that it (very early on) switch to an actual compile error. We may do that one day but for now it might be a good idea to at least warn from RcppEigen.h if the header guards for either or both of Rcpp and R are seen.
- Langage dominant
- C++
- Étoiles
- 118
- Forks
- 43
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de RcppCore/RcppEigen
-
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
-
RcppEigen Submatrix Views Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 20/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
Toutes les issues de RcppCore/RcppEigen
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
-
Sensor initialization takes very long when `--initial-sim-time` is set to current UNIX timestamp Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
gazebosim/gz-sensors#662 · 1 commentaire ·
-
enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
-
comp-datalake
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
ClickHouse/ClickHouse#121222 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
LadybirdBrowser/ladybird#12123 ·