From:   Yangtao Li <frank.li@vivo.com>
To:     Jaegeuk Kim <jaegeuk@kernel.org>, Chao Yu <chao@kernel.org>
Cc:     Yangtao Li <frank.li@vivo.com>,
        linux-f2fs-devel@lists.sourceforge.net,
        linux-kernel@vger.kernel.org
Subject: [RESEND] f2fs: convert is_extension_exist() to return bool type
Date:   Sun, 19 Mar 2023 15:58:22 +0800
Message-Id: <20230319075822.79337-1-frank.li@vivo.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?iysuEVQDl22kmE+HiZ4HZesLuzceWfzZ9vm5kevLLqiLqn8GlKKjLf5ASKZt?=
 =?us-ascii?Q?Lahyao/IvQAEnOtceS9O71IR8SvUg2ipOSWyoQzoA3xBaT4ahCIs2SBtdjcf?=
 =?us-ascii?Q?P+83CYoB2YwfnutPBhYkuo46SC+K7dsFdCNjY7OgSZqEwgiJYZxqbQsNBW32?=
 =?us-ascii?Q?TvdVzqGo6A65wSAQLwFx5v75IEyCpPY0ISQxdl89Dy6q62Jz3WfummlGw+uo?=
 =?us-ascii?Q?nLwgoARYLaad3GuLquFNCP0H2iHsVACjFjjTqXGPjnJ2Tl0qevtwg/XW7ynS?=
 =?us-ascii?Q?MfOagRETZcCOdCA49qddYBYWU3LowpJ9ug/eECT+IQPcl0D46eVac0tTKN6R?=
 =?us-ascii?Q?USNDkS6Ftwv1XQ3cMbtSJX7Ie8m6X00kqQn6RyNCrOWN7Rczd4/u2gy1vu3J?=
 =?us-ascii?Q?mBYmWyxnPY0i5fmz2kSyiYDbeXmetUw3L3iHQi79clCY6BVHNXLTD/+i1T29?=
 =?us-ascii?Q?6QaaDEi2gLfpUms+G2xpRd+1K/uQp1F/rQfErsZeCnFjdrbM5bbQikW8Mc7J?=
 =?us-ascii?Q?ATPLw8qWlBpBxhduHOtPpVBswdMlPgjFk4uqCa/eN96giemLi2020bUj2dNf?=
 =?us-ascii?Q?z8uo781NM9ej91OtaOXHlyabqK8j/7Cb6Pvwqg+S/n4cUW7a1R8CjmPBgVyr?=
 =?us-ascii?Q?3s/CwJWdreAMMPoQZY36Ass32+TR/pXzNlZdVeDcpvBwUORDKP24+L6LZ5q3?=
 =?us-ascii?Q?cmNV9fy1YMgNr0xlKF5/YMONEPA3KXDcSWbJKBOmTKOgSkc1rgXrv/dYsedj?=
 =?us-ascii?Q?6AvKi/ZPfhyqqgQOiY4BzfigOx+WHFbL4Fjh8ACfBDoSg/P/L4xsTxNXDnXG?=
 =?us-ascii?Q?OdVC39hPbwJjYRBMaH+pMwoau0XlsBjo/Rof/2kkls7CJ84wZHOeC2Hu9ffz?=
 =?us-ascii?Q?F6E40SZim24kjAAK5RuU46f7z2cvPcmPwtj/3GqRNYQ1JJL9+e7Qle3pKrSn?=
 =?us-ascii?Q?6goYmt7f/ZEiICu2ifE5bHZd/nDgLvfONBnGDag3aagYe3tvC5wK/HxfiVu2?=
 =?us-ascii?Q?AzcLgRTmJtQ/+N60G9Gp9Fh/SY2yJwFjZ9y42bUGzDAwAvT8VnWicpZ3KG14?=
 =?us-ascii?Q?H97X9/BjZ4cJBQNcRDlzTBP8h9EktRJ+8bFcb1k4WLTFp6dR99nWcdh2OXeR?=
 =?us-ascii?Q?f8UTDnwrzaT9OOvT0DcUkQ/45nn0r0lw+DLpLlfBVrXsbu5ZlEX0W/3giTnw?=
 =?us-ascii?Q?5j72Lsjf+mix0sUgql+OwNPB5fQ5xZ856AK8Ml+9TSZLaiFN3WDAVaA+et7k?=
 =?us-ascii?Q?IZpNAFEpnj3vk9MT3CTy2fWlkir5CCxFAktAeAB85Y+ceFB9nVOxCvei7x+q?=
 =?us-ascii?Q?lisYyTNTpV41vP1BfW2+343rD3oBWGzZ410qZ8TU1gHSp6QP1pNRqck+M2vN?=
 =?us-ascii?Q?U/o+gZu0WE9jp36s2Ey4g9hdVUGcB0OKH7Y7T1Woeg+XW8dUgIEdPCFTe2D3?=
 =?us-ascii?Q?c2mGm16fjbKMglX4V/8uBSlPtDqkfEKLsG7uZuigNVw3OU7xQTqxRNRchFCp?=
 =?us-ascii?Q?j2iEFVB7ucOYaP3GQgUdOicD6d3Q3a4o0W2vPdPuqukJflx32fu7HpXRAEEB?=
 =?us-ascii?Q?DmFPGU47jDil35O5seUgHYBSfekjo6tY7YzZXu4m?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8c3c7489-edc3-4170-4e87-08db284fbc7d
X-MS-Exchange-CrossTenant-AuthSource: SEZPR06MB5269.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Mar 2023 07:58:33.7300
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: hdynMuH3XRbo7vGkmckWsVPFbqvsCXGk197iBmAfZM+Rks6c4uzAJFGboDuIlgKhi2sjYiArBh50qEYJubz7Pw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TY0PR06MB5626
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

is_extension_exist() only return two values, 0 or 1.
So there is no need to use int type.

Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
Resend this patch because there is no similar patch to merge in this time.
 fs/f2fs/namei.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c
index 11fc4c8036a9..ad597b417fea 100644
--- a/fs/f2fs/namei.c
+++ b/fs/f2fs/namei.c
@@ -22,7 +22,7 @@
 #include "acl.h"
 #include <trace/events/f2fs.h>
 
-static inline int is_extension_exist(const unsigned char *s, const char *sub,
+static inline bool is_extension_exist(const unsigned char *s, const char *sub,
 						bool tmp_ext)
 {
 	size_t slen = strlen(s);
@@ -30,19 +30,19 @@ static inline int is_extension_exist(const unsigned char *s, const char *sub,
 	int i;
 
 	if (sublen == 1 && *sub == '*')
-		return 1;
+		return true;
 
 	/*
 	 * filename format of multimedia file should be defined as:
 	 * "filename + '.' + extension + (optional: '.' + temp extension)".
 	 */
 	if (slen < sublen + 2)
-		return 0;
+		return false;
 
 	if (!tmp_ext) {
 		/* file has no temp extension */
 		if (s[slen - sublen - 1] != '.')
-			return 0;
+			return false;
 		return !strncasecmp(s + slen - sublen, sub, sublen);
 	}
 
@@ -50,10 +50,10 @@ static inline int is_extension_exist(const unsigned char *s, const char *sub,
 		if (s[i] != '.')
 			continue;
 		if (!strncasecmp(s + i + 1, sub, sublen))
-			return 1;
+			return true;
 	}
 
-	return 0;
+	return false;
 }
 
 int f2fs_update_extension_list(struct f2fs_sb_info *sbi, const char *name,
-- 
2.35.1