cosmetic: both `if (FALSE) expr` and for-loops return NULL
since NULL in tag() will be dropped, there is no need to check is.null(icon)
Merge pull request #467 from yihui/cosmetic

Some cosmetic changes regarding NULL