Removed the possibility to create a Semaphore object without specifying

a name - that's mostly done to:
1) force better fitting lock names
2) make the API consistent (the empty ReadWriteLock constructor doesn't
   initialize the lock anymore).


git-svn-id: file:///srv/svn/repos/haiku/trunk/current@3366 a95241bf-73f2-0310-859d-f6bbb57e9c96

src/add-ons/kernel/file_systems/bfs/BlockAllocator.cpp

@@ -312,6 +312,7 @@ AllocationGroup::Free(Transaction *transaction, uint16 start, int32 length)
 BlockAllocator::BlockAllocator(Volume *volume)
 	:
 	fVolume(volume),
+	fLock("bfs allocator"),
 	fGroups(NULL),
 	fCheckBitmap(NULL)
 {

src/add-ons/kernel/file_systems/bfs/Lock.h

@@ -29,7 +29,7 @@
 
 class Semaphore {
 	public:
-		Semaphore(const char *name = "bfs sem")
+		Semaphore(const char *name)
 			:
 #ifdef USE_BENAPHORE
 			fSemaphore(create_sem(0, name)),
@@ -151,14 +151,14 @@ class ReadWriteLock {
 	public:
 		ReadWriteLock(const char *name)
 			:
-			fWriteLock()
+			fWriteLock(name)
 		{
 			Initialize(name);
 		}
 
 		ReadWriteLock()
 			:
-			fWriteLock()
+			fWriteLock("bfs r/w w-lock")
 		{
 		}