Re: [Mesa-dev] [PATCH 1/6] translate_generic: use memcpy if possible

2010-08-13 Thread Luca Barbieri
> In this change you've got an int value (copy_size) which has some > special meaning when negative -- can you add comments explaining what > the meaning of a negative size is?  Is there a way to use some more > explicit flag value to indicate this condition? I think it makes sense, since -1 stand

Re: [Mesa-dev] [PATCH 1/6] translate_generic: use memcpy if possible

2010-08-13 Thread Keith Whitwell
Luca, In this change you've got an int value (copy_size) which has some special meaning when negative -- can you add comments explaining what the meaning of a negative size is? Is there a way to use some more explicit flag value to indicate this condition? Keith On Thu, 2010-08-12 at 10:08 -070