hide 'unused variable' compiler warning

r-lib / later

Schedule an R function or formula to run after a specified period of time.

https://r-lib.github.io/later

Other

137 stars 27 forks source link

hide 'unused variable' compiler warning #176

Closed MichaelChirico closed 1 year ago

MichaelChirico commented 1 year ago

Copied the approach here:

https://github.com/r-lib/later/blob/bc088f896acbd09f0fde1a78e7f056ec18aadea8/src/later_posix.cpp#L54

OTOH, if write() fails, presumably that'll be captured in this value & we should cascade that? I don't know write() well enough but I'd guess if (!ret) ... would be better.

wch commented 1 year ago

I'm not sure why the Github Action workflow isn't running, but I'll merge and let it run on main.