We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
d = data.table(a=1, b=1) dropcols = character() set(d, NULL, dropcols, NULL) #Warning message: #In set(d, NULL, dropcols, NULL) : # length(LHS)==0; no columns to delete or assign RHS to.
I don't think we really need a warning here, it could work without the need for if (length(dropcols)) set(...).
if (length(dropcols)) set(...)
The text was updated successfully, but these errors were encountered:
data.table/src/assign.c
Lines 346 to 347 in 6f17d18
That would unfortunately also remove the warning from d[, character() := NULL]. Are we OK to remove the warning from there also? IMO make sense
d[, character() := NULL]
Sorry, something went wrong.
Agree don't need to warn, verbose message would be fine
set for empty cols does not warn anymore, #4086
009859e
jangorecki
Successfully merging a pull request may close this issue.
I don't think we really need a warning here, it could work without the need for
if (length(dropcols)) set(...)
.The text was updated successfully, but these errors were encountered: