[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: koffice/lib/kospell
From:       Montel Laurent <montell () club-internet ! fr>
Date:       2003-02-02 18:04:16
[Download RAW message or body]

Le Sunday 02 February 2003 18:40, David Faure a écrit :
> On Sunday 02 February 2003 18:06, Laurent Montel wrote:
> > -    aspell_config_replace(config, "lang",
> > ksconfig->dictionary().isEmpty() ? "de":
> > ksconfig->dictionary().latin1()); +    aspell_config_replace(config,
> > "lang", ksconfig->dictionary().isEmpty() ? "fr":
> > ksconfig->dictionary().latin1());
>
> Any hardcoded language name looks bad here, whether it's de or fr :)
> IMHO it should just abort if no language is specified.

Yes it's right.
But for the moment kword/kpresenter/kspread can't configure new aspell api
=> so I add this entry to test my code.
When it will perfect I remove it.


Regards.
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic