nice comment, still recommend to hide under "if'