24 Jan
2008
24 Jan
'08
2:35 p.m.
Dan Smith wrote:
JG> A bit nitpicky here, but I think here name_only is an int that's JG> being use as a boolean, while getInstance is an actual boolean. JG> I'm fine with doing things either way, but for the sake of JG> consistency they should probably be the same. I'd vote for making JG> name_only a bool.
Agreed. She didn't add this, but while it's being modified it would be nice to get it changed.
Yea, sorry I forgot to add, "and I won't yell at you for making an 'unnecessary change' since the line is already being modified. :) -- -Jay