Patch to export init_stat_ex_from_stat so that it can be used in VFS modules rather than duplicating code
Guenther Deschner
gd at samba.org
Thu Jun 9 09:58:04 MDT 2011
Hi RIchard,
On 06/09/2011 01:24 AM, Richard Sharpe wrote:
> OK,
>
> Here is an updated version:
>
> Signed-off-by: Richard Sharpe<realrichardsharpe at gmail.com>
>
> From: Richard Sharpe<realrichardsharpe at gmail.com>
> Date: Fri, 3 Jun 2011 08:27:13 -0700
> Subject: [PATCH] Export init_stat_ex_from_stat for use in VFS modules.
>
> ---
> source3/include/includes.h | 1 +
> source3/include/proto.h | 11 +++++++++++
> source3/lib/system.c | 12 ++++++------
> 3 files changed, 18 insertions(+), 6 deletions(-)
>
> diff --git a/source3/include/includes.h b/source3/include/includes.h
> index e6b70c7..55a8e85 100644
> --- a/source3/include/includes.h
> +++ b/source3/include/includes.h
> @@ -72,6 +72,7 @@
> #include "system/locale.h"
> #include "system/time.h"
> #include "system/wait.h"
> +#include "system/filesys.h"
^^^ please, please don't do that.
It took us a long time to trim down the list of global includes (and
make includes.h more and more identical between s3 and s4) and you would
just re-add a whole bunch of globally included header files just in
order to have one additional prototype again globally available. This is
how we did things in the past but we should not continue to do it this way.
A better way of resolving this would be to split out the prototypes of
the init_stat_ex_from_stat() call to a tiny separate header file for
now, and include it only where really needed.
Does that make sense ?
Thanks,
Guenther
--
Günther Deschner GPG-ID: 8EE11688
Red Hat gdeschner at redhat.com
Samba Team gd at samba.org
More information about the samba-technical
mailing list