Thanks for the review. Fixed that empty #ifdef block, missed one line code there to free glamor screen. From: eugeni.dodonov at gmail.com [mailto:eugeni.dodonov at gmail.com] On Behalf Of Eugeni Dodonov Sent: Friday, November 11, 2011 8:59 PM To: Zhigang Gong Cc: intel-gfx at lists.freedesktop.org Subject: Re: [PATCH 2/3] glamor: turn on glamor. On Fri, Nov 11, 2011 at 06:31, Zhigang Gong <zhigang.gong at linux.intel.com> wrote: @@ -1109,7 +1127,8 @@ static void I830FreeScreen(int scrnIndex, int flags) { ScrnInfoPtr scrn = xf86Screens[scrnIndex]; intel_screen_private *intel = intel_get_screen_private(scrn); - +#ifdef GLAMOR +#endif Empty #ifdef block? -- Eugeni Dodonov <http://eugeni.dodonov.net/> -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20111114/eb7d38d2/attachment.html>
