> Ah, I see why I was confused.  Previously, 'optional' meant whether
> the arguments were _present_.  Now, an optional argument can be
> present but 'null'.
Correct.

> If there's some good way to clarify the new behavior in the comment,
> it might be worthwhile.  For example, the 3rd and 4th sentences could
> read:
Done.

> Ah, so the code here is intentionally _clearing_ any value that may
> have been set, if the other arguments were not given?  That makes
> sense now -- consider clarifying the comments (e.g. "Ignore" ->
> "Clear").
Yes, done.

Submitted.

Steve

Reply via email to