Since we're not restricted to C89 in the driver, I put it as close to the
point of use as possible -- but yes, I can move it to the top of the
function if you think that's clearer.
On Thu, Jan 23, 2014 at 6:58 AM, Courtney Goeltzenleuchter <
court...@lunarg.com> wrote:
>
>
>
> On Tue, Jan 21, 2014 at 3:34 AM, Chris Forbes <chr...@ijw.co.nz> wrote:
>
>> We're about to need this in another place.
>>
>> Signed-off-by: Chris Forbes <chr...@ijw.co.nz>
>> ---
>> src/mesa/drivers/dri/i965/intel_fbo.c | 7 +++++--
>> 1 file changed, 5 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/mesa/drivers/dri/i965/intel_fbo.c
>> b/src/mesa/drivers/dri/i965/intel_fbo.c
>> index 4cdf54d..e7c5571 100644
>> --- a/src/mesa/drivers/dri/i965/intel_fbo.c
>> +++ b/src/mesa/drivers/dri/i965/intel_fbo.c
>> @@ -433,16 +433,19 @@ intel_renderbuffer_update_wrapper(struct
>> brw_context *brw,
>> intel_miptree_check_level_layer(mt, level, layer);
>> irb->mt_level = level;
>>
>> + int layer_multiplier;
>>
> Shouldn't this declaration be at the top of the function?
>
>
>> switch (mt->msaa_layout) {
>> case INTEL_MSAA_LAYOUT_UMS:
>> case INTEL_MSAA_LAYOUT_CMS:
>> - irb->mt_layer = layer * mt->num_samples;
>> + layer_multiplier = mt->num_samples;
>> break;
>>
>> default:
>> - irb->mt_layer = layer;
>> + layer_multiplier = 1;
>> }
>>
>> + irb->mt_layer = layer_multiplier * layer;
>> +
>> intel_miptree_reference(&irb->mt, mt);
>>
>> intel_renderbuffer_set_draw_offset(irb);
>> --
>> 1.8.5.3
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>>
>
>
>
> --
> Courtney Goeltzenleuchter
> LunarG
>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
