Skip to content

Commit 5c234e8

Browse files
committed
not a macro anymore so lowercase
1 parent 251e88d commit 5c234e8

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/frollapply.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#include "data.table.h"
22

3-
static inline void MEMCPY_SEXP(SEXP dest, size_t offset, SEXP src, int count)
3+
static inline void memcpy_sexp(SEXP dest, size_t offset, SEXP src, int count)
44
{
55
switch (TYPEOF(dest)) {
66
case INTSXP: {
@@ -32,7 +32,7 @@ static inline void MEMCPY_SEXP(SEXP dest, size_t offset, SEXP src, int count)
3232
* we don't call memcpyVector from memcpyDT because they are called in tight loop and we don't want to have extra branches inside
3333
*/
3434
SEXP memcpyVector(SEXP dest, SEXP src, SEXP offset, SEXP size) {
35-
MEMCPY_SEXP(dest, INTEGER_RO(offset)[0] - INTEGER_RO(size)[0], src, LENGTH(dest));
35+
memcpy_sexp(dest, INTEGER_RO(offset)[0] - INTEGER_RO(size)[0], src, LENGTH(dest));
3636
return dest;
3737
}
3838
// # nocov start ## does not seem to be reported to codecov most likely due to running in a fork, I manually debugged that it is being called when running froll.Rraw
@@ -41,7 +41,7 @@ SEXP memcpyDT(SEXP dest, SEXP src, SEXP offset, SEXP size) {
4141
const int ncol = LENGTH(dest);
4242
const int nrow = LENGTH(VECTOR_ELT(dest, 0));
4343
for (int i = 0; i < ncol; i++) {
44-
MEMCPY_SEXP(VECTOR_ELT(dest, i), INTEGER_RO(offset)[0] - INTEGER_RO(size)[0], VECTOR_ELT(src, i), nrow);
44+
memcpy_sexp(VECTOR_ELT(dest, i), INTEGER_RO(offset)[0] - INTEGER_RO(size)[0], VECTOR_ELT(src, i), nrow);
4545
}
4646
return dest;
4747
}
@@ -51,9 +51,9 @@ SEXP memcpyVectoradaptive(SEXP dest, SEXP src, SEXP offset, SEXP size) {
5151
const size_t oi = INTEGER_RO(offset)[0];
5252
const int nrow = INTEGER_RO(size)[oi - 1];
5353
const size_t o = oi - nrow; // oi should always be bigger than nrow because we filter out incomplete window using ansMask
54-
SETLENGTH(dest, nrow); // must be before MEMCPY_SEXP because attempt to set index 1/1 in SET_STRING_ELT test 6010.150
54+
SETLENGTH(dest, nrow); // must be before memcpy_sexp because attempt to set index 1/1 in SET_STRING_ELT test 6010.150
5555
if (nrow) { // support k[i]==0
56-
MEMCPY_SEXP(dest, o, src, nrow);
56+
memcpy_sexp(dest, o, src, nrow);
5757
}
5858
return dest;
5959
}
@@ -68,7 +68,7 @@ SEXP memcpyDTadaptive(SEXP dest, SEXP src, SEXP offset, SEXP size) {
6868
SEXP d = VECTOR_ELT(dest, i);
6969
SETLENGTH(d, nrow);
7070
if (nrow) { // support k[i]==0
71-
MEMCPY_SEXP(d, o, VECTOR_ELT(src, i), nrow);
71+
memcpy_sexp(d, o, VECTOR_ELT(src, i), nrow);
7272
}
7373
}
7474
return dest;

0 commit comments

Comments
 (0)