Index: trunk/psModules/src/pmAstrometry.c
===================================================================
--- trunk/psModules/src/pmAstrometry.c	(revision 4779)
+++ trunk/psModules/src/pmAstrometry.c	(revision 4992)
@@ -8,6 +8,6 @@
 *  @author GLG, MHPCC
 *
-*  @version $Revision: 1.3 $ $Name: not supported by cvs2svn $
-*  @date $Date: 2005-08-16 02:29:29 $
+*  @version $Revision: 1.4 $ $Name: not supported by cvs2svn $
+*  @date $Date: 2005-09-11 22:25:39 $
 *
 *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -106,5 +106,5 @@
 }
 
-
+// XXX: Verify these default values for row0, col0, rowBins, colBins
 pmReadout *pmReadoutAlloc(pmCell *cell)
 {
@@ -707,4 +707,5 @@
 */
 
+// XXX: How should we handle errors?  What if psMetadataLookup() is NULL?
 psMetadataItem *pmReadoutGetConcept(pmReadout *readout, const char *concept)
 {
@@ -794,8 +795,8 @@
 }
 
-psRegion *pmCellGetTrimSec(pmCell *cell) // CELL.TRIMSEC
+psRegion pmCellGetTrimSec(pmCell *cell) // CELL.TRIMSEC
 {
     psMetadataItem *tmp = pmCellGetConcept(cell, "CELL.TRIMSEC");
-    return((psRegion *) tmp->data.V);
+    return((psRegion) *((psRegion *) (tmp->data.V)));
 }
 
@@ -824,14 +825,15 @@
 }
 
-psPixelCoord *pmCellGetBin(pmCell *cell) // CELL.BIN
+
+psPixelCoord pmCellGetBin(pmCell *cell) // CELL.BIN
 {
     psMetadataItem *tmp = pmCellGetConcept(cell, "CELL.BIN");
-    return((psPixelCoord *) tmp->data.V);
-}
-
-psPixelCoord *pmCellGetParity(pmCell *cell) // CELL.PARITY
+    return((psPixelCoord)  *((psPixelCoord *) (tmp->data.V)));
+}
+
+psPixelCoord pmCellGetParity(pmCell *cell) // CELL.PARITY
 {
     psMetadataItem *tmp = pmCellGetConcept(cell, "CELL.PARITY");
-    return((psPixelCoord *) tmp->data.V);
+    return((psPixelCoord)  *((psPixelCoord *) (tmp->data.V)));
 }