On Tue, Oct 16, 2012 at 2:14 PM, Nuno Lopes <> wrote:

> Hi,
>  + case IS_BOOL:
>> + case IS_NULL:
>>   case IS_RESOURCE:
>>   case IS_ARRAY:
>>   default:
> it doesn't make sense to have those cases and the default. Please remove
> all those useless cases.

I see it as self-documentation to include them as it indicates without
needing to think that it's intentional that they are treated as default. If
the overall consensus is that they should be removed, that's fine (and I'll
remove them). But I consider this more readable and easier to comprehend
than without...



Reply via email to