[Okular-devel] Review Request 109119: Txt backend: Drop detectEncoding method and just feed more data to KEncodingProber

[Azat Khuzhin]

> On Feb. 27, 2013, 10:43 p.m., Albert Astals Cid wrote:
> > Code looks, ok, but now the big question, is this really an improvement? I guess the only improvement would be speed-wise and not encoding-detection-wise. Have you opened really big files and noticed it is faster? Otherwise maybe there is no really need to change the existing code if it works as well?

Yes it is only speed-wise improvement, as I already told.
> Description:
> Don't to extra job, when encoding is detected just return encoding.

For big files there is improvement, but this files is quite big (~900 000  lines long), if we oriented on books
Just two times faster for such number of lines.

Maybe there is no need in this patch, but for me it more appropriate way.


- Azat


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109119/#review28231
-----------------------------------------------------------


On Feb. 25, 2013, 8:15 p.m., Azat Khuzhin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109119/
> -----------------------------------------------------------
> 
> (Updated Feb. 25, 2013, 8:15 p.m.)
> 
> 
> Review request for Okular.
> 
> 
> Description
> -------
> 
> Don't to extra job, when encoding is detected just return encoding.
> 
> 
> Diffs
> -----
> 
>   generators/txt/document.h 08babea 
>   generators/txt/document.cpp badb647 
> 
> Diff: http://git.reviewboard.kde.org/r/109119/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Azat Khuzhin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20130228/9bf273fa/attachment-0001.html>