-
-
Notifications
You must be signed in to change notification settings - Fork 208
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/win debug dll (closes #1035) #1037
Changes from 3 commits
26befe8
cd8d2dd
4b86c1b
3268980
9da65da
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||
---|---|---|---|---|---|---|
|
@@ -9,6 +9,8 @@ | |||||
\itemize{ | ||||||
\item Safer \code{Rcpp_list*}, \code{Rcpp_lang*} and | ||||||
\code{Function.operator()} (Romain in \ghpr{1014}, \ghit{1015}). | ||||||
\item A number of \code{#nocov} markers were added (Dirk in | ||||||
\ghprP1036}). | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hmpf. I could swear I can a check which should spotted that. Will fix! |
||||||
} | ||||||
\item Changes in Rcpp Attributes: | ||||||
\itemize{ | ||||||
|
@@ -17,18 +19,27 @@ | |||||
\ghit{1017}). | ||||||
\item Invisible return object are now supported via new option (Kun Ren | ||||||
in \ghpr{1025} fixing \ghit{1024}). | ||||||
\item Unavailable packages referred to in \code{LinkingTo} are now | ||||||
reported (Dirk in \ghpr{1027} fixing \ghit{1026}). | ||||||
\item The \code{sourceCpp} function can now create a debug DLL on | ||||||
Windows (Dirk in \ghpr{1037} fixing \ghit{ghit}). | ||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also, there's There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Bah. Something must be wrong with the water. Or the coffee. Or both. Changed to |
||||||
} | ||||||
\item Changes in Rcpp Documentation: | ||||||
\itemize{ | ||||||
\item The \code{.github/} directory now has more explicit guidance on | ||||||
contributinh, issues, and pull requests (Dirk). | ||||||
\item The Rcpp Attributes vignette describe the new invisible return | ||||||
objection option (Kun Ren in \ghpr{1025}). | ||||||
\item Vignettes are now included as pre-made pdf files (Dirk in \ghpr{1029}) | ||||||
\item The Rcpp FAQ has new entry on the recommended | ||||||
\code{importFrom} directive (Dirk in \ghpr{1031} fixing \ghit{1030}). | ||||||
} | ||||||
\item Changes in Rcpp Deployment: | ||||||
\itemize{ | ||||||
\item Added unit test to check if C++ version remains remains aligned | ||||||
with the package number (Dirk in \ghpr{1022} fixing \ghit{1021}). | ||||||
\item The unit test system was switched to tinytest (Dirk in | ||||||
\ghpr{1028}, \ghpr{1032}, \ghpr{1033}). | ||||||
} | ||||||
} | ||||||
} | ||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be outside the if()? As it stands now, this will only trigger if
verbose = TRUE
is passed (it normally isn't.)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch. I think you are correct. Will fix.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New variant:
Better...