Martin Stjernholm wrote:
>"Stephen R. van den Berg" <[email protected]> wrote:

>>     if((THIS->foo=foo))
>>     { debug_malloc_touch(THIS->foo);

>I suggest replacing this bit with the move_svalue macro:

>    if (foo) {
>      move_svalue (THIS->foo, foo);

>It has some debug stuff to detect if the old svalue is used later by
>mistake, but more importantly it's good for code clarity.

Erm, these are not svalues, but objects, so move_svalue() cannot be used.
Any other suggestions?
-- 
Stephen.

"There are three types of people in this world: those who make things happen,
 those who watch things happen and those who wonder what happened."
  • PIK... Stephen R. van den Berg
    • ... Stephen R. van den Berg
    • ... Marcus Comstedt (ACROSS) (Hail Ilpalazzo!) @ Pike (-) developers forum
      • ... Stephen R. van den Berg
        • ... Marcus Comstedt (ACROSS) (Hail Ilpalazzo!) @ Pike (-) developers forum
    • ... Henrik Grubbstr�m (Lysator) @ Pike (-) developers forum
      • ... Marcus Comstedt (ACROSS) (Hail Ilpalazzo!) @ Pike (-) developers forum
        • ... Henrik Grubbstr�m (Lysator) @ Pike (-) developers forum
    • ... Martin Stjernholm
      • ... Stephen R. van den Berg
        • ... Martin Stjernholm

Reply via email to