Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Feature/no sugar tonight in my coffee#585

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
eddelbuettel merged 7 commits intoRcppCore:masterfromdcdillon:feature/no-sugar-tonight-in-my-coffee
Nov 13, 2016
Merged

Feature/no sugar tonight in my coffee#585

eddelbuettel merged 7 commits intoRcppCore:masterfromdcdillon:feature/no-sugar-tonight-in-my-coffee
Nov 13, 2016

Conversation

@dcdillon
Copy link
Contributor

Fixes#584 by the method detailed in the issue report.

coatless, thirdwing, and kevinushey reacted with thumbs up emoji
@thirdwing
Copy link
Member

I think we can merge and it will be better if@dcdillon can add some change log.

coatless and eddelbuettel reacted with thumbs up emoji

@dcdillon
Copy link
ContributorAuthor

Modifiedinst/NEWS.Rd. Hope that's what you meant.

@eddelbuettel
Copy link
Member

Pretty much every repo has a fileChangeLog. Thought you'd noticed ;-)

@dcdillon
Copy link
ContributorAuthor

Must be blind. I looked! Fixing.

@dcdillon
Copy link
ContributorAuthor

Would it be reasonable to add a 'no_sugar' plugin so this can be turned on easily on a per-source file basis?

@eddelbuettel
Copy link
Member

Maybe. I don't think I'd have use for it. Why would you turn sugar off, and why can't a#define before#include <Rcpp.h> do it?

@dcdillon
Copy link
ContributorAuthor

Just looks cleaner than arbitrarily sticking#define in your.cpp files. Not a huge issue, just a thought.

I turn sugar off to get better handling of matrix operations without causing ambiguities in operator calls.

@eddelbuetteleddelbuettel merged commit127f14d intoRcppCore:masterNov 13, 2016
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dcdillon@thirdwing@eddelbuettel

[8]ページ先頭

©2009-2025 Movatter.jp