4 Dec
2007
4 Dec
'07
3:27 p.m.
On Tue, Dec 04, 2007 at 01:04:28PM -0600, Dan McGee wrote:
Yeah, I realized this, but one of two things should be done: 1. Copy the existing switch statement with all the options (as I ended up doing). 2. Make an explicit comment in the code why certain statements were omitted.
Ok, fair enough. As I already said, I'm fine with 1. I thought it might still be worth adding a little comment where pm_vasprintf is called, in cb_log, but I actually wouldn't know what to write exactly, and I'm not sure it would be really useful, so forget it.