Re: [LTP] [PATCH 2/5] lapi/kcmp.h: Replace GPL with SPDX-License-Identifier

[Petr Vorel <pvorel@suse.cz>]

Hi Xu,

> Signed-off-by: Yang Xu <xuyang2018.jy@fujitsu.com>
> ---
>  include/lapi/kcmp.h | 25 ++++++-------------------
>  1 file changed, 6 insertions(+), 19 deletions(-)

> diff --git a/include/lapi/kcmp.h b/include/lapi/kcmp.h
> index 59371fc07..77086191f 100644
> --- a/include/lapi/kcmp.h
> +++ b/include/lapi/kcmp.h
> @@ -1,29 +1,16 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
>  /*
>   * Copyright (c) 2015 Cedric Hnyda <chnyda@suse.com>
> - *
> - * This program is free software; you can redistribute it and/or
> - * modify it under the terms of the GNU General Public License as
> - * published by the Free Software Foundation; either version 2 of
> - * the License, or (at your option) any later version.
> - *
> - * This program is distributed in the hope that it would be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> - * GNU General Public License for more details.
> - *
> - * You should have received a copy of the GNU General Public License
> - * along with this program; if not, write the Free Software Foundation,
> - * Inc.,  51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
>   */

> -#ifndef KCMP_H
> -#define KCMP_H
> +#ifndef LAPI_KCMP_H__
> +#define LAPI_KCMP_H__

>  #include <sys/types.h>
>  #include "config.h"
>  #include "lapi/syscalls.h"

> -#if !defined(HAVE_ENUM_KCMP_TYPE)
> +#ifndef HAVE_ENUM_KCMP_TYPE

>  enum kcmp_type {
>  	KCMP_FILE,
> @@ -42,7 +29,7 @@ enum kcmp_type {

>  #endif

> -#if !defined(HAVE_KCMP)
> +#ifndef HAVE_KCMP

>  int kcmp(int pid1, int pid2, int type, int fd1, int fd2)
$ make check-pidfd_getfd01
../../../../include/lapi/kcmp.h:34:5: warning: Symbol 'kcmp' has no prototype or library ('tst_') prefix. Should it be static?

Could you please mark this as static inline?

Not sure if this should be with the same commit as SPDX.
When doing cleanup it'd be worth to fix it.

Kind regards,
Petr

>  {
> @@ -51,4 +38,4 @@ int kcmp(int pid1, int pid2, int type, int fd1, int fd2)

>  #endif

> -#endif /* KCMP_H */
> +#endif /* LAPI_KCMP_H__ */

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp