Re: [tyndur-devel] [PATCH] pwd.h & sys/socket.h

[Kevin Wolf <kevin@xxxxxxxxxx>]

On Wed, Aug 12, 2009 at 08:45:15AM +0200, Alexander Hartmut Kluth wrote:
> Paar Sachen hinzugefügt
> 
> + pwd.h: char* pw_gecos in struct passwd
> + sys/socket.h: PF_INET und IPPROTO_IP

Zwei Themengebiete => zwei Patches

Ich kann ihn dieses Mal auch so nehmen, aber für die Zukunft bitte
beachten.

> 
> Signed-off-by: Alexander Hartmut Kluth <alexanderkluth@xxxxxx>
> ---
>  src/modules/include/pwd.h        |    2 ++
>  src/modules/include/sys/socket.h |    4 +++-
>  2 files changed, 5 insertions(+), 1 deletions(-)
> 
> diff --git a/src/modules/include/pwd.h b/src/modules/include/pwd.h
> index b6e441b..a2aea98 100644
> --- a/src/modules/include/pwd.h
> +++ b/src/modules/include/pwd.h
> @@ -43,6 +43,8 @@ struct passwd {
>      char*   pw_dir;
>      /** Login-Shell */
>      char*   pw_shell;
> +    
> +    char*   pw_gecos;

Da fehlt ein /** Echter Name des Benutzer */

>  };
>  
>  
> diff --git a/src/modules/include/sys/socket.h b/src/modules/include/sys/socket.h
> index a8446f6..263d181 100644
> --- a/src/modules/include/sys/socket.h
> +++ b/src/modules/include/sys/socket.h
> @@ -34,6 +34,8 @@
>  typedef int socklen_t;
>  typedef unsigned int sa_family_t;
>  
> +const int IPPROTO_IP = 0;
> +
>  /**
>   *  Beschreibt die Adresse einer Gegenstelle fuer einen Socket.
>   *
> @@ -61,7 +63,7 @@ enum {
>  /// Adressfamilien
>  enum {
>      /// IPv4
> -    AF_INET,
> +    AF_INET, PF_INET
>  };

Ich glaube, das wäre besser ein #define PF_INET AF_INET.