KCodecs - Quick Review

Mario Fux KDE ML kde-ml at unormal.org
Fri Mar 7 16:52:46 UTC 2014


Am Dienstag, 04. März 2014, 16.58:11 schrieb John Layt:
> Hi,

Morning

Just a short notice. It looks as if the description on projects.kde.org for 
KCodecs is wrong:


Thx
Mario

> I know nothing about text codecs, but I've had a *very* quick look at
> KCodecs:
> 
> * Original code by Lars dated 1999!
> * One method marked as deprecated to be removed for KDE4
> * "###FIXME KDE4: the name of the encodings should mostly be uppercase"
> * Code generated by script generate_string_table.pl located in
> kdesdk/scripts * Algorithms marked as copyright by RSA Data Security and
> others, but no mention what the original licence was or real link to
> original source * Encoding probers and lookup tables marked as copyright
> Mozilla 1998, X11 license
> * kentities.c is documented as generated by gperf from either
> kentities.gperf and/or khtmlentities.gperf but neither are in kcodecs,
> instead they are in khtml as is another copy of kentities.c.
> * Public API using boolean parms
> 
> This suggests it could do with some attention:
> * Check still valid to remove deprecated code?
> * Check if encoding names should be made uppercase?
> * The generate_string_table.pl script should probably be moved into
> KCodecs, unless it has more general use?
> * The RSA and other algorithms may need checking for licensing issues, or
> at least improve the license documentation?
> * The probers may need to be checked they are still up to date with the
> original Mozilla code and look-up tables?
> * kentities.c needs investigation and I suspect moving all the files from
> khtml to kcodecs, with khtml then using kcodecs?  Or at least docs added
> that this is where it comes from and should be kept in sync.
> 
> I wonder how much of this functionality is now done in Qt5?  Would it
> benefit from a functional review by someone who knows what they're doing,
> like Thiago or David?
> 
> Cheers!
> 
> John.




More information about the Kde-frameworks-devel mailing list