From: Patrick McHardy <kaber@trash.net>
To: Jan Engelhardt <jengelh@medozas.de>
Cc: David Miller <davem@davemloft.net>, netfilter-devel@vger.kernel.org
Subject: Re: RFC: net/netfilter reorganization
Date: Sun, 05 Oct 2008 18:21:24 +0200 [thread overview]
Message-ID: <48E8E984.8090807@trash.net> (raw)
In-Reply-To: <alpine.LNX.1.10.0810051212060.21891@fbirervta.pbzchgretzou.qr>
Jan Engelhardt wrote:
> On Sunday 2008-10-05 12:11, Patrick McHardy wrote:
>>> One thing I'm looking forward to is the elimination of all of
>>> the xt_* prefixes, and in general better TAB completion
>>> friendliness when I try to open netfilter source files under
>>> emacs or grep things from the command line with tab completed
>>> paths.
>> I'm not sure how emacs handles completion, would using different
>> prefixes for matches and targets help? Otherwise I guess we can
>> just get rid of the prefixes entirely.
>
> Allow me to speak generally for humans.. we are lazy to type, so
> usually it's just the first three or four characters (or less,
> of course) we want to type before hitting TAB the first time.
> By eliminating prefixes, you already start typing on the "real"
> name, which means your chances that TAB completion returns just
> a single file is much higher.
Thats true. Not using any prefixes requires keeping the
upper letter naming convention for targets though (which
I don't really mind).
>> The main problem with these renames is that it module aliases
>> don't work for "rmmod", which is a pretty common thing with
>> these modules. I guess we'd need to fix up module-init-tools
>> first.
>
> When was the last time you used tab completion on .ko files?
I don't use tab-completion above file-level at all, its takes
way to long to initialize all the more fancy stuff.
> I think this is sufficient:
>
> obj-$(config_foo) += nfct_ftp.o
> nfct_ftp-objs := ftp.c
>
> That way, Mr Developer can use ft<TAB> to get ftp.c, but the
> final module that we will be using with modprobe/rmmod still
> has the prefix (and we should really have one!)
A lot of people are unloading modules in their firewall scripts
and that will break.
next prev parent reply other threads:[~2008-10-05 16:21 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-05 12:34 RFC: net/netfilter reorganization Patrick McHardy
2008-10-05 13:47 ` Jan Engelhardt
2008-10-05 14:02 ` Patrick McHardy
2008-10-05 14:35 ` Jan Engelhardt
2008-10-05 14:48 ` Patrick McHardy
2008-10-05 16:02 ` David Miller
2008-10-05 16:11 ` Patrick McHardy
2008-10-05 16:15 ` Jan Engelhardt
2008-10-05 16:21 ` Patrick McHardy [this message]
2008-10-05 16:25 ` Jan Engelhardt
2008-10-05 16:32 ` Patrick McHardy
2008-10-05 19:06 ` Jozsef Kadlecsik
2008-10-05 20:28 ` David Miller
2008-10-05 20:33 ` Jan Engelhardt
2008-10-05 20:48 ` Jan Engelhardt
2008-10-05 21:42 ` Jozsef Kadlecsik
2008-10-05 22:00 ` Patrick McHardy
2008-10-05 23:16 ` Jan Engelhardt
2008-10-06 10:07 ` Patrick McHardy
2008-10-07 1:08 ` Jan Engelhardt
2008-10-07 11:34 ` Roman Zippel
2008-10-07 15:30 ` Jan Engelhardt
2008-10-07 17:09 ` Roman Zippel
2008-10-07 17:44 ` Jan Engelhardt
2008-10-13 18:52 ` Roman Zippel
2008-10-17 14:53 ` Jan Engelhardt
2008-10-06 7:23 ` Jozsef Kadlecsik
2008-10-06 10:09 ` Patrick McHardy
2008-10-05 16:17 ` David Miller
2008-10-05 16:22 ` Patrick McHardy
2008-10-06 16:17 ` Jan Engelhardt
2008-10-05 21:51 ` Jozsef Kadlecsik
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=48E8E984.8090807@trash.net \
--to=kaber@trash.net \
--cc=davem@davemloft.net \
--cc=jengelh@medozas.de \
--cc=netfilter-devel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.