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
new translations for upcoming release #4765
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4765 +/- ##
=======================================
Coverage 99.47% 99.47%
=======================================
Files 73 73
Lines 14556 14556
=======================================
Hits 14480 14480
Misses 76 76
Continue to review full report at Codecov.
|
About 20 translations we rely on our team for, have sent out an ask. |
@@ -501,7 +501,7 @@ SEXP forder(SEXP DT, SEXP by, SEXP retGrpArg, SEXP sortGroupsArg, SEXP ascArg, S | |||
int keyAlloc = (ncol+n_cplx)*8 + 1; // +1 for NULL to mark end; calloc to initialize with NULLs | |||
key = calloc(keyAlloc, sizeof(uint8_t *)); // needs to be before loop because part II relies on part I, column-by-column. | |||
if (!key) | |||
STOP("Unable to allocate %"PRId64" bytes of working memory", (uint64_t)keyAlloc*sizeof(uint8_t *)); // # nocov | |||
STOP(_("Unable to allocate %"PRIu64" bytes of working memory"), (uint64_t)keyAlloc*sizeof(uint8_t *)); // # nocov |
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.
@mattdowle this fix is correct right?
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.
@MichaelChirico oh yes, good spot!
If it is ready to merge we could have it now for a hot fix needed due to #4769, adding milestone |
yes it's ready |
Closes #4764
translations courtesy @hongyuanjia