Don't send NULL buffer data hint in chrome

[TomH, 2011-07-01 15:06:10]

On 2011/07/01 15:01:45, bsalomon wrote:

LGTM.

To echo a recent review, though, do we have an expectation about #ifdef vs #if?

[bsalomon, 2011-07-01 15:09:32]

In Ganesh we always define all configurable macros. We only use #ifdef to
set default values when a custom value isn't provided and to protect
against multiple inclusion of headers.

On Fri, Jul 1, 2011 at 11:06 AM, <tomhudson@google.com> wrote:

> On 2011/07/01 15:01:45, bsalomon wrote:
>
> LGTM.
>
> To echo a recent review, though, do we have an expectation about #ifdef
> vs #if?
>
>
>
>
http://codereview.appspot.com/**4657067/<http://codereview.appspot.com/4657067/>
>

[reed1, 2011-07-01 15:13:39]

LGTM

is this a candidate for runtime flag in GLInterfaces, rather than compile-time?

[bsalomon, 2011-07-01 15:21:33]

On Fri, Jul 1, 2011 at 11:13 AM, <reed@google.com> wrote:

> LGTM
>
> is this a candidate for runtime flag in GLInterfaces, rather than
> compile-time?
>
> Yup


>
>
http://codereview.appspot.com/**4657067/<http://codereview.appspot.com/4657067/>
>

[bsalomon, 2011-07-01 15:22:17]

Closed with r1778.