Skip to content

Commit

Permalink
fwrite warning if no OpenMP changed to Rprintf so that test.data.tabl…
Browse files Browse the repository at this point in the history
…e() would still pass. #580
  • Loading branch information
mattdowle committed Apr 20, 2016
1 parent 1ade3f5 commit 325097d
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/fwrite.c
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,8 @@ SEXP writefile(SEXP list_of_columns,
error("Column %d's length (%d) is not the same as column 1's length (%d)", i+1, length(VECTOR_ELT(list_of_columns, i)), nrows);
}
#ifndef _OPENMP
warning("Your platform/environment has not detected OpenMP support. fwrite() will still work, but slower in single threaded mode.");
Rprintf("Your platform/environment has not detected OpenMP support. fwrite() will still work, but slower in single threaded mode.\n");
// Rprintf rather than warning() because warning() would cause test.data.table() to error about the unexpected warnings
#endif
const Rboolean verbose = LOGICAL(verboseArg)[0];
const Rboolean quote = LOGICAL(quoteArg)[0];
Expand Down

1 comment on commit 325097d

@MichaelChirico
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just saw the new speed comparisons, phenomenal! great work, thanks!

Please sign in to comment.