framebuffer: Remove pointless test

The error value for mmap() is MAP_FAILED, which is not nul. But we check for that case in Open() anyway.

framebuffer: Remove pointless test
The error value for mmap() is MAP_FAILED, which is not nul. But we check for that case in Open() anyway.
e86bc7ed · Rémi Denis-Courmont · 484a11ef · e86bc7ed
Commit e86bc7ed authored Feb 11, 2009 by Rémi Denis-Courmont
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 6 deletions

modules/video_output/fb.c modules/video_output/fb.c +3 -6

No files found.
--- a/modules/video_output/fb.c
+++ b/modules/video_output/fb.c
@@ -944,12 +944,9 @@ static int OpenDisplay( vout_thread_t *p_vout )
 *****************************************************************************/
 static void CloseDisplay( vout_thread_t *p_vout )
 {
-    if( p_vout->p_sys->p_video )
-    {
-        /* Clear display */
-        memset( p_vout->p_sys->p_video, 0, p_vout->p_sys->i_page_size );
-        munmap( p_vout->p_sys->p_video, p_vout->p_sys->i_page_size );
-    }
+    /* Clear display */
+    memset( p_vout->p_sys->p_video, 0, p_vout->p_sys->i_page_size );
+    munmap( p_vout->p_sys->p_video, p_vout->p_sys->i_page_size );

    if( p_vout->p_sys->i_fd >= 0 )
    {