netdev - Re: [PATCH v2] net: #ifdef inet_bind_bucket::ib

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20081112.031651.239477846.davem@davemloft.net>
Date:	Wed, 12 Nov 2008 03:16:51 -0800 (PST)
From:	David Miller <davem@...emloft.net>
To:	adobriyan@...il.com
Cc:	dada1@...mosbay.com, netdev@...r.kernel.org
Subject: Re: [PATCH v2] net: #ifdef inet_bind_bucket::ib_net

From: Alexey Dobriyan <adobriyan@...il.com>
Date: Wed, 12 Nov 2008 13:44:39 +0300

> On the other hand, people are interrupted with ctags jumping when suddenly
> whole new file appears, so loss of context is even more.

I can't remember ever reading a patch using ctags, nor actual code for
that matter.  It's not the best facility.

People read straight code.  And inline functions defined somewhere had
damn well better be named well enough to have their basic operation
or side effect be understood.

If you absolutely, positively, must know the implementation details,
you go look it up (using 'git grep' or plain 'grep', not ctags).
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html