From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Marek Lindner Date: Thu, 10 May 2012 14:08:07 +0800 References: <1336561976-16088-1-git-send-email-ordex@autistici.org> <1336561976-16088-8-git-send-email-ordex@autistici.org> <20120509.204111.1931607501484626500.davem@davemloft.net> In-Reply-To: <20120509.204111.1931607501484626500.davem@davemloft.net> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201205101408.07750.lindner_marek@yahoo.de> Subject: Re: [B.A.T.M.A.N.] [PATCH 07/14] batman-adv: split neigh_new function into generic and batman iv specific parts Reply-To: The list for a Better Approach To Mobile Ad-hoc Networking List-Id: The list for a Better Approach To Mobile Ad-hoc Networking List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: b.a.t.m.a.n@lists.open-mesh.org Cc: netdev@vger.kernel.org, David Miller David, > But once it gets exported out of that file, you have to give it > an appropriate name. Probably with a "batman_adv_" prefix or > similar. > > Otherwise, for one thing, if this code is statically built into > the tree it can conflict with symbol names elsewhere and break > the build. thanks for pointing this out. I wasn't aware of the potential conflict created by our naming scheme. Obviously, we will change the proposed patch. We are going to review all other exported function names and send appropriate patches later on. Regards, Marek