Re: MC and file highlighting

[Jindrich Novy]

On Wed, 2005-08-17 at 17:49 +0400, X-Stranger wrote:
 Hi all!
 
 I think it will be much better, if MC will be able to hightlight files on 
 panels depending of it's extensions. I wrote a small patch (see attached 
 file), that realises this feature. What do you think about to include it in 
 MC source code?
 
 Regards, X-Stranger
 
 P.S. Patch realises file highlighting feature and a dialog to setup file 
 extensions lists and save it into MC's ini-file.
 ___
 Mc-devel mailing list
 http://mail.gnome.org/mailman/listinfo/mc-devel

This is a very nice feature! It's what I missed since I was using the
old good DOS Navigator very long ago. The patch seems sane, the only bad
thing I found was the duplicated comment at line 543 in the patch that
should be removed and one C++ comment at line 219 that should be changed
to C one.

The other thing that hit my eyes was the Solaris hack at line 428. Is
that really needed? I feel that strsep() is sufficient here and the
sunos_get_token() call might be a source of problems in the future
regarding the portability of mc what brings a neglectable benefit (if
any).

Jindrich

-- 
Jindrich Novy [EMAIL PROTECTED], http://people.redhat.com/jnovy/
(o_   _o)
//\  The worst evil in the world is refusal to think. //\
V_/_ _\_V


___
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel

What about the next release ?

[Pavel Tsekov]

Hello,

I was wondering what would the next release include ? Is it going to be
based on HEAD or on the 4.6.1 branch + fixes and some new features ? Would
we like to have 4.6.2 or rather 4.7 ? I prefer that we have 4.6.2 soon
which is going to be based on 4.6.1 + fixes. If we choose to go this way
we have to start porting fixes / enhancements from HEAD to 4.6.2

Any thoughts ?

Thanks!
___
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel

Roland keep the Changelog entries clean

[Pavel Tsekov]

Hello,

This kind of comments has no place in the changelog:

[...]
 Whoever wrote that should be blamed for it, but the code has been there
 since the beginning.
[...]

Please, don't do it.
___
Mc-devel mailing list
http://mail.gnome.org/mailman/listinfo/mc-devel