RE: [PATCH] 2.5.24+ fix needed for non-modular video build

[Christopher Hoover]

> On Mon, Jul 22, 2002 at 01:45:14PM -0700, Christopher Hoover wrote:
> 
>  > -#ifdef MODULE
>  > -#if defined(CONFIG_PROC_FS) && defined(CONFIG_VIDEO_PROC_FS)
>  >  static void videodev_proc_destroy(void)
>  >  {
>  >  	if (video_dev_proc_entry != NULL)
>  > @@ -298,8 +296,6 @@
>  >  	if (video_proc_entry != NULL)
>  >  		remove_proc_entry("video", &proc_root);
>  >  }
>  > -#endif
>  > -#endif
> 
> Why are you removing the inner ifdef too ? This looks like it
> makes sense (to me at least)

It makes sense but it is redundant -- there is a wider-scoped ifdef in
the file that catches that function and others.

-ch

mailto:[e-mail]
mailto:[e-mail]