It's not necessary if we are about to skip the rest of the if clause anyway because name is NULL. <div class="gmail_quote">On May 28, 2015 9:14 PM, "Emil Velikov" <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 28 May 2015 at 00:57, Joonyoung Shim <<a href="mailto:jy0922.shim@samsung.com">jy0922.shim@samsung.com</a>> wrote: > This is to remove to open the DRM device unnecessarily as call > drmAvailable() when name is NULL or drm_server_info is NULL in > drmOpenWithType function. > Why do you believe that calling drmAvailable() is not necessary ? If that's the case should one just nuke the call all together ? -Emil _______________________________________________ dri-devel mailing list <a href="mailto:dri-devel@lists.freedesktop.org">dri-devel@lists.freedesktop.org</a> <a href="http://lists.freedesktop.org/mailman/listinfo/dri-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/dri-devel</a> </blockquote></div>