Why not do it like:

switch($last) {
case 'g':
$val *= 1024;
case 'm':
$val *= 1024;
case 'k':
$val *= 1024;
}

is much shorter and cleaner looking.

Derick

I've seen your commit. I didn't do it like that because I think that the break's missing could confuse someone.
Anyway, I'll follow your suggestion and change the code..


Nuno

Reply via email to