Error handling of n_max=NA could be improved

[jfsalzmann]

I recently had a hard time debugging that my code made a wrong vroom::vroom() call with an edge case bug on my side where I supplied n_max = NA.

Instead of some error/exception behaviour from vroom, there was an R error thrown from here, line 33

  if (n_max < 0 || is.infinite(n_max)) {
    n_max <- -1
  }

Error in if (condition) { : missing value where TRUE/FALSE needed

[jennybc]

Reprex:

vroom::vroom(vroom::vroom_example("mtcars.csv"), n_max = NA)
#> Error in if (n_max < 0 || is.infinite(n_max)) {: missing value where TRUE/FALSE needed

^{Created on 2026-01-20 with reprex v2.1.1}