KMimeType::isBufferBinaryData() is broken

Andreas Pakulat apaku at gmx.de
Sat May 5 20:26:37 BST 2007


On 05.05.07 20:22:35, Ralf Habacker wrote:
> Andreas Pakulat schrieb:
> >Hi,
> >
> >just found out that isBufferBinaryData fails to recognize plain/text
> >when there's a linefeed in the first 32 bytes. IMHO the check should
> >check for bytes in the range 00-0A, 0B-0C and 0E-1F, i.e. leave out
> >linefeed and carriage return. Those two ascii control characters are
> >valid in plain/text.
> >
> >The attached patch does this, objections against committing?
> >  
> either the line
> +        char c = p[i];

Ooops, I just added that line for some debugging :) Of course it won't
be in the final commit.

Andreas

-- 
Bank error in your favor.  Collect $200.




More information about the kde-core-devel mailing list