"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.
  • 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