Comments on snippet: 'Inverse Hex Color'

koncept said on 10/5/08

Sun, 05 Oct 2008 19:31:56 GMT

You caught something there... There should be a break after case 3. Corrected.

Raisor said on 10/5/08

Sun, 05 Oct 2008 15:04:47 GMT

Guess my formatting wasn't correct, I was asking: Shouldn't this:

case 3: 
       $color=preg_replace("/(.)(.)(.)/","\1\1\2\2\3\3",$color); 
case 6:

actually be this:

case 3: 
case 6: 
       $color=preg_replace("/(.)(.)(.)/","\1\1\2\2\3\3",$color);

Raisor said on 10/5/08

Sun, 05 Oct 2008 15:02:04 GMT

Shouldn't this: case 3: $color=preg_replace("/(.)(.)(.)/","\\1\\1\\2\\2\\3\\3",$color); case 6: actually be this: case 3: case 6: $color=preg_replace("/(.)(.)(.)/","\\1\\1\\2\\2\\3\\3",$color);