[ViewVC] Diff of: cvs/deliantra/server/common/map.C

Comparing deliantra/server/common/map.C (file contents):
Revision 1.33 by root, Sun Oct 8 16:51:50 2006 UTC vs.
Revision 1.35 by root, Sun Dec 10 01:16:27 2006 UTC

           }                     /* for this space */
       }                         /* for this j */
 }
+maptile::maptile ()
+{
+  in_memory = MAP_SWAPPED;
+  /* The maps used to pick up default x and y values from the
+   * map archetype.  Mimic that behaviour.
+   */
+  MAP_WIDTH (this) = 16;
+  MAP_HEIGHT (this) = 16;
+  MAP_RESET_TIMEOUT (this) = 0;
+  MAP_TIMEOUT (this) = 300;
+  MAP_ENTER_X (this) = 0;
+  MAP_ENTER_Y (this) = 0;
+  /*set part to -1 indicating conversion to weather map not yet done */
+  MAP_WORLDPARTX (this) = -1;
+  MAP_WORLDPARTY (this) = -1;
+}
 /*
  * Allocates, initialises, and returns a pointer to a maptile.
  * Modified to no longer take a path option which was not being
  * used anyways.  MSW 2001-07-01
  */
 maptile *
 get_linked_map (void)
 {
-  maptile *map = new maptile;
+  maptile *mp, *map = new maptile;
-  maptile *mp;
-  for (mp = first_map; mp != NULL && mp->next != NULL; mp = mp->next);
+  for (mp = first_map; mp && mp->next; mp = mp->next);
   if (mp == NULL)
     first_map = map;
   else
     mp->next = map;
-  map->in_memory = MAP_SWAPPED;
-  /* The maps used to pick up default x and y values from the
-   * map archetype.  Mimic that behaviour.
-   */
-  MAP_WIDTH (map) = 16;
-  MAP_HEIGHT (map) = 16;
-  MAP_RESET_TIMEOUT (map) = 0;
-  MAP_TIMEOUT (map) = 300;
-  MAP_ENTER_X (map) = 0;
-  MAP_ENTER_Y (map) = 0;
-  /*set part to -1 indicating conversion to weather map not yet done */
-  MAP_WORLDPARTX (map) = -1;
-  MAP_WORLDPARTY (map) = -1;
   return map;
 }
 /*
  * Allocates the arrays contained in a maptile.
  * This basically allocates the dynamic array of spaces for the
  * map.
  */
 void
-allocate_map (maptile *m)
+maptile::allocate ()
 {
-  m->in_memory = MAP_IN_MEMORY;
+  in_memory = MAP_IN_MEMORY;
   /* Log this condition and free the storage.  We could I suppose
    * realloc, but if the caller is presuming the data will be intact,
    * that is their poor assumption.
    */
-  if (m->spaces)
+  if (spaces)
     {
-      LOG (llevError, "allocate_map called with already allocated map (%s)\n", m->path);
+      LOG (llevError, "allocate_map called with already allocated map (%s)\n", path);
-      free (m->spaces);
+      free (spaces);
     }
-  m->spaces = (MapSpace *) calloc (1, MAP_WIDTH (m) * MAP_HEIGHT (m) * sizeof (MapSpace));
+  spaces = (MapSpace *)
+    calloc (1, width * height * sizeof (MapSpace));
-  if (m->spaces == NULL)
+  if (!spaces)
     fatal (OUT_OF_MEMORY);
 }
 /* Create and returns a map of the specific size.  Used
  * in random map code and the editor.
   maptile *m = get_linked_map ();
   m->width = sizex;
   m->height = sizey;
   m->in_memory = MAP_SWAPPED;
-  allocate_map (m);
+  m->allocate ();
   return m;
 }
 /* Takes a string from a map definition and outputs a pointer to the array of shopitems
  * corresponding to that string. Memory is allocated for this, it must be freed
 {
   char buf[HUGE_BUF], msgbuf[HUGE_BUF], maplorebuf[HUGE_BUF], *key = NULL, *value, *end;
   int msgpos = 0;
   int maplorepos = 0;
-  while (fgets (buf, HUGE_BUF - 1, fp) != NULL)
+  while (fgets (buf, HUGE_BUF, fp) != NULL)
     {
       buf[HUGE_BUF - 1] = 0;
       key = buf;
       while (isspace (*key))
         key++;
        * that use the parameter.
        */
       if (!strcmp (key, "msg"))
         {
-          while (fgets (buf, HUGE_BUF - 1, fp) != NULL)
+          while (fgets (buf, HUGE_BUF, fp) != NULL)
             {
               if (!strcmp (buf, "endmsg\n"))
                 break;
               else
                 {
           if (msgpos != 0)
             m->msg = strdup_local (msgbuf);
         }
       else if (!strcmp (key, "maplore"))
         {
-          while (fgets (buf, HUGE_BUF - 1, fp) != NULL)
+          while (fgets (buf, HUGE_BUF, fp) != NULL)
             {
               if (!strcmp (buf, "endmaplore\n"))
                 break;
               else
                 {
       LOG (llevError, "Error loading map header for %s, flags=%d\n", filename, flags);
       delete_map (m);
       return NULL;
     }
-  allocate_map (m);
+  m->allocate ();
   m->in_memory = MAP_LOADING;
   load_objects (m, thawer, flags & (MAP_BLOCK | MAP_STYLE));
   m->in_memory = MAP_IN_MEMORY;
       LOG (llevError, "Error loading map header for %s (%s)\n", m->path, m->tmpname);
       delete_map (m);
       m = load_original_map (m->path, 0);
       return NULL;
     }
-  allocate_map (m);
+  m->allocate ();
   m->in_memory = MAP_LOADING;
   load_objects (m, thawer, 0);
   m->in_memory = MAP_IN_MEMORY;
       LOG (llevError, "Error loading map header for overlay %s (%s)\n", m->path, pathname);
       delete_map (m);
       m = load_original_map (m->path, 0);
       return NULL;
     }
-  /*allocate_map(m); */
+  /*m->allocate ();*/
   m->in_memory = MAP_LOADING;
   load_objects (m, thawer, MAP_OVERLAY);
   m->in_memory = MAP_IN_MEMORY;

Diff Legend

-–
+Removed lines
-+
+Added lines
-<
+Changed lines
->
+Changed lines

Comparing deliantra/server/common/map.C (file contents): Revision 1.33 by root, Sun Oct 8 16:51:50 2006 UTC vs. Revision 1.35 by root, Sun Dec 10 01:16:27 2006 UTC

Diff Legend

Comparing deliantra/server/common/map.C (file contents):
Revision 1.33 by root, Sun Oct 8 16:51:50 2006 UTC vs.
Revision 1.35 by root, Sun Dec 10 01:16:27 2006 UTC