[cairo] [PATCH 14/41] drm/intel: intel_scaled_glyph_fini(): renamed local priv variable for better consistency

Uli Schlachter psychon at znc.in
Sat Dec 12 13:17:22 PST 2015


Am 12.12.2015 um 18:03 schrieb Enrico Weigelt, metux IT consult:
> On 12.12.2015 10:53, Uli Schlachter wrote:
> 
>> I can see that, yeah. However, this still removes the if (glyph != NULL) check
>> that was there before (should be priv != NULL after this patch, no?).
> 
> That check was also missing in the other backends I used as templates
> (eg. GL)
> 
> Should we add it there, too ?

I'd guess that if other backends don't have it, it isn't necessary. I don't
know. My complaint was only about the patch doing not exactly what its commit
message says.

-- 
A learning experience is one of those things that say,
'You know that thing you just did? Don't do that.'
                     -- Douglas Adams


More information about the cairo mailing list