Re: [msvc][heditv2] free_help bug

From: George (greerga@CIRCLEMUD.ORG)
Date: 05/24/98


On Sun, 24 May 1998, ;P wrote:

>ok..free_room frees all of the parts of the room, but not the actual room
>pointer itself..so i don't think that the first break should be
>there..that way when we abort editing a room, it'll free the OLC_ROOM as
>well..i believe oasis v1.6b is like this..the same should be done for
>hedit, qedit, and anything else that differentiates between a CLEANUP_ALL
>and a CLEANUP_STRUCTS..

It doesn't free the room, though if you look at the patch it says:

+  /* free(room); - ASAP */

There are some places that may need to be looked at, although I'm not aware
of any at the moment that actually have a problem. (Though I have not
looked for them, it's small.)

--
George Greer, greerga@circlemud.org | Genius may have its limitations, but
http://patches.van.ml.org/          | stupidity is not thus handicapped.
http://www.van.ml.org/CircleMUD/    |                  -- Elbert Hubbard


     +------------------------------------------------------------+
     | Ensure that you have read the CircleMUD Mailing List FAQ:  |
     | http://democracy.queensu.ca/~fletcher/Circle/list-faq.html |
     +------------------------------------------------------------+



This archive was generated by hypermail 2b30 : 12/15/00 PST