[Kde-pim] charsets in kdepim/kmail/locale.c

[Carles Pina i Estany]

Hello,

On Jul/26/2008, Ingo Klöcker wrote:
> On Saturday 26 July 2008, Carles Pina i Estany wrote:

> > I was checking something in Kmail code when I've found this file:
> > kdepim/kmail/locale.c
> >
> > I haven't checked where this file is used, but I guess that, if there
> > is this file it's used from somewhere :-)
> 
> AFAICS, it is not used. In fact, it doesn't seem to be compiled. (It's
> commented out in CMakeLists.txt.)

Yes I see (rgrep)

> > *: in the patch I left out Norwegian, Swedish and some other country
> > because I know that they have some special characters and I don't
> > know if they need -1 and not -15 to use them. Same for some other
> > countries, but should be better than before.
> >
> > Thank you for your good work,
> 
> Thank you for your patch. If locale.c should be used then we'll make 
> sure that your patch is applied.

ok fantastic. Maybe is a good idea or remove the file or apply the
patch (just in case some day somebody uses and forgets about the
patch) (or add a comment at beginning of the file...)

Thanks,

-- 
Carles Pina i Estany		GPG id: 0x17756391
	http://pinux.info
_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/