na.rm not defaulted in layer()

[oneilsh]

Not sure if this is considered an issue or not, but I noticed that while na.rm recently defaults to FALSE for some stats (035e6ff, 067e096) the layer() function doesn't have a default, resulting in Error: is.logical(na.rm) is not TRUE if params is unspecified.

Just checking to see if this is the intended behavior/will stay that way going forward. (Same with no default for position?)

p <- ggplot() +
  layer(data = diamonds, 
        stat = "bin",
        geom = "bar",
        mapping = aes(x = price),
        position = "identity")
plot(p)

[hadley]

Fixed in 7d67785

I don't see how position could default to anything other than NULL though. It would be inconsistent with geom and stat