POV-Ray: Newsgroups: povray.programming: objects.h

POV-Ray : Newsgroups : povray.programming : objects.h - Copy_Flag		Server Time 23 Dec 2024 14:48:38 EST (-0500)

From: Massimo Valentini
Subject: objects.h - Copy_Flag
Date: 13 Sep 2002 07:26:47
Message: <3d81cb77@news.povray.org>

Again, looking in the code I spotted this MACRO that seems to me
wrong:

-> File: objects.h Line: 55-56
 *   Copy_Flag   (Object1, Object2, Flag) : Set the Flag in Object1 to the
 *                                          value of the Flag in Object2.

-> File: objects.h Line: 108-111
#define Copy_Flag(Object1, Object2, Flag) \
  { (Object1)->Flags = (((Object1)->Flags) & (!Flag)) | \
                       (((Object2)->Flags) &  (Flag)); }




I think ~ is correct instead of !

#define Copy_Flag(Object1, Object2, Flag) \
  { (Object1)->Flags = (((Object1)->Flags) & (~Flag)) | \
                       (((Object2)->Flags) &  (Flag)); }

HTH Massimo

Post a reply to this message

From: Thorsten Froehlich
Subject: Re: objects.h - Copy_Flag
Date: 16 Sep 2002 07:51:40
Message: <3d85c5cc@news.povray.org>

In article <3d81cb77@news.povray.org> , "Massimo Valentini" 
<six### [at] inwindit> wrote:

> Again, looking in the code I spotted this MACRO that seems to me
> wrong:
>
> -> File: objects.h Line: 55-56
>  *   Copy_Flag   (Object1, Object2, Flag) : Set the Flag in Object1 to the
>  *                                          value of the Flag in Object2.
>
> -> File: objects.h Line: 108-111
> #define Copy_Flag(Object1, Object2, Flag) \
>   { (Object1)->Flags = (((Object1)->Flags) & (!Flag)) | \
>                        (((Object2)->Flags) &  (Flag)); }
>
> I think ~ is correct instead of !

I agree.  However, s it is only used in Copy_Poly (the function that copies
declared poly objects), I guess nobody noticed yet in years...

    Thorsten

____________________________________________________
Thorsten Froehlich, Duisburg, Germany
e-mail: tho### [at] trfde

Visit POV-Ray on the web: http://mac.povray.org

Post a reply to this message