Comment 5 for bug 514295

Revision history for this message
Данило Шеган (danilo) wrote :

As far as the patch goes: since we are removing 3 chars, we should probably check for those 3 chars instead of checking for only two. And as far as the code structure goes, we should do a single if check and store the unpacked value in a variable.