"brian m. carlson" <sandals@xxxxxxxxxxxxxxxxxxxx> writes: > So to make sure I understand this right, we'll collect traces regardless > if it's enabled, but we'll still honor the --quiet flag if the user > doesn't want to see them? If so, I'm definitely in favor of this > change. Hmph, if we know we won't be emitting, why spend cycles to collect traces in the first place? Does it make the code simpler to follow or something? As long as it is done when the user requests a trace to be taken, I do not care about the wasted cycles too much, I think ;-) > I was worried when I read the cover letter that we'd display > them to the user regardless, but from reading the patch and the commit > message, it seems I misunderstood. > > I think the making the verbose flag a parameter simplifies the code > nicely and puts the rendering decision in the right place. OK.
