tridge the destroyer returns!

prompted by the interpret_security() dead code that Jean-Francois
pointed out I added a make target "finddead" that finds potentially
dead (ie. unused) code. It spat out 304 function names ...

I went through these are deleted many of them, making others static
(finddead also reports functions that are used only in the local
file).

in doing this I have almost certainly deleted some useful code. I may
have even prevented compilation with some compile options. I
apologise. I decided it was better to get rid of this code now and add
back the one or two functions that are needed than to keep all this
baggage.

So, if I have done a bit too much "destroying" then let me know. Keep
the swearing to a minimum :)

One bit I didn't do is the ubibt code. Chris, can you look at that?
Heaps of unused functions there. Can they be made static?
(This used to be commit 2204475c87f3024ea8fd1fbd7385b2def617a46f)

1 files changed, 0 insertions, 11 deletions

diff --git a/source3/lib/bitmap.c b/source3/lib/bitmap.c
index ce677f7846..9ccdbb420b 100644
--- a/source3/lib/bitmap.c
+++ b/source3/lib/bitmap.c
@@ -51,17 +51,6 @@ struct bitmap *bitmap_allocate(int n)
 }
 
 /****************************************************************************
-free a bitmap structure
-****************************************************************************/
-void bitmap_free(struct bitmap *bm)
-{
-	free(bm->b);
-	bm->b = NULL;
-	free(bm);
-}
-
-
-/****************************************************************************
 set a bit in a bitmap
 ****************************************************************************/
 BOOL bitmap_set(struct bitmap *bm, unsigned i)