[Spice-devel] [spice-html5] display: Dump bitmap metadata in log_draw()
Jeremy White
jwhite at codeweavers.com
Tue Jul 17 14:00:33 UTC 2018
>>> diff --git a/display.js b/display.js
>>> index 7719b23..c55b686 100644
>>> --- a/display.js
>>> +++ b/display.js
>>> @@ -769,6 +769,12 @@ SpiceDisplayConn.prototype.log_draw = function(prefix, draw)
>>> str += "; src_bitmap type " + draw.data.src_bitmap.descriptor.type + ", flags " + draw.data.src_bitmap.descriptor.flags;
>>> if (draw.data.src_bitmap.surface_id !== undefined)
>>> str += "; src_bitmap surface_id " + draw.data.src_bitmap.surface_id;
>>> + if (draw.data.src_bitmap.bitmap)
>>> + str += "; BITMAP format " + draw.data.src_bitmap.bitmap.format +
>>> + "; flags " + draw.data.src_bitmap.bitmap.format +
>>
>> That looks like a repeat of the format, rather than the flags.
>
> Yup, this duplicates the width/heigh info, however this also has the
> stride which is not present in the other places. Showing this is the
> main goal of the patch, as I did not go as far as fixing the actual bug
> (too unfamiliar with js :-/)
Ah, I think you misunderstood. I wasn't concerned about the extra info,
it's that I noticed that you supply a label of 'flags', but then
actually pass in the format, which looks to be a copy/paste bug.
Cheers,
Jeremy
More information about the Spice-devel
mailing list