Fixed groupings for LLADD API docs.

lladd/common.h

@@ -44,6 +44,8 @@ terms specified in this license.
  *
  * A standard header file, adopted from Autobook. 
  *
+ * http://sources.redhat.com/autobook/
+ *
  * The idea behind this file is twofold.  First, we want to keep as
  * much of the #ifdef portability nonsense in here as possible.
  * Second, we allow users to #include headers that in turn #include
@@ -51,10 +53,8 @@ terms specified in this license.
  * the right thing' and build, even though they do not #include the
  * config.h file that all of the LLADD stuff uses.
  *
- * @todo: Need to make sure everyone actually includes this thing, and
- * also includes constants.h
- *
- * @ingroup LLADD_CORE
+ * @todo Need to make sure every .c file actually includes this thing, and
+ * also includes constants.h, and that no .h files include config.h
  *
  * $Id$
  */

lladd/logger/logEntry.h

@@ -55,8 +55,13 @@ BEGIN_C_DECLS
 
    @todo Was getting some memory over-runs from the fact that I didn't
    know the exact length of a raw log entry.  This seems to be fixed
-   now.  (Need to run through electric fence / determine the 'right
-   way' to do this...)
+   now. 
+   
+   @todo Is there a better way to deal with sizeof() and log entries?
+   
+   @ingroup LLADD_CORE
+   
+   $Id$
 */
 
 typedef struct {

lladd/operations/alloc.h

@@ -3,9 +3,27 @@
 #ifndef __ALLOC_H
 #define __ALLOC_H
 
+/**
+   @file 
+
+   @ingroup OPERATIONS
+
+   $Id$
+*/
+
 Operation getAlloc();
 Operation getDealloc();
+
+/** 
+    Allocate a record.  
+
+    @param The transaction responsible for the allocation @param The
+    size of the new record to be allocated.  (Talloc may allocate a
+    blob if the record will not easily fit on a page.)
+*/
 recordid Talloc(int xid, long size);
+
+/** @todo Currently, we just leak store space on dealloc. */
 void Tdealloc(int xid, recordid rid);
 
 #endif

lladd/operations/increment.h

@@ -50,6 +50,8 @@ terms specified in this license.
  *
  * @see decrement.h
  *
+ * @ingroup OPERATIONS
+ *
  * $Id$
  */ 
 

lladd/operations/lladdhash.h

@@ -20,6 +20,8 @@
    result in poor performance when the bucket size is large, and
    transactions are short.
 
+   @ingroup OPERATIONS
+
    $Id$
 
 */

lladd/page.h

@@ -43,7 +43,7 @@ terms specified in this license.
 /**
  * @file
  *
- * interface for dealing with pages
+ * interface for dealing with slotted pages
  *
  * @ingroup LLADD_CORE
  * $Id$

lladd/transactional.h

@@ -45,7 +45,13 @@ terms specified in this license.
  *
  * The minimal subset of LLADD necessary to implement transactional consistency.
  *
- * This module includes the standard API (excluding operations), the logger, the buffer mananger, and recovery code.
+ * This module includes the standard API (excluding operations), the
+ * logger, the buffer mananger, and recovery code.
+ *
+ * In theory, the other .h files that are installed in /usr/include
+ * aren't needed for application developers.
+ *
+ * @todo Move as much of the stuff in lladd/ to src/lladd/ as possible.
  *
  */
 /**

src/lladd/blobManager.h

@@ -7,8 +7,14 @@ BEGIN_C_DECLS
 
 /**    
        @file
-       blobManager - Provides blob handling 
+       blobManager - Provides blob handling.
+ 
        Plan for modularity: Exactly one blob manager per storeFile.
+       Alternatively, we could use the bufferManager's record length
+       field to allow for more than one blob manager per buffer
+       manager.  (Right now, this field is set to a special value for
+       blobs; thie could be extended for other types of records.)
+
        Blob manager interacts with page manger via page manager's
        public api.