Double // in codepages list error
Pavel Roskin
proski at gnu.org
Sun Aug 12 06:26:58 UTC 2001
On Sun, 12 Aug 2001, Walery Studennikov wrote:
> On Sat, Aug 11, 2001 at 11:12:19PM -0400, Pavel Roskin wrote:
> >
> > I understand that my criticism really should be directed at Walery, but
> > I'm asking everybody to clean up the code being modified instead of piling
> > one "quick fix" onto another.
>
> The patch is attached.
> Now we use concat_dir_and_file in charsets.c and on error we
> return filename that we failed to open.
It's ugly. It's a nontrivial use of the return value that should
be at least commented in the code (ideally: avoided).
I don't understand what's wrong with moving the error messages to
charset.c from setup.c as I proposed.
Please use unified diff next time.
By the way, I've already fixed charset.c to use concat_dir_and_file().
The only remaining change is moving GUI where it belongs.
--
Regards,
Pavel Roskin
More information about the mc-devel
mailing list