netlib_xarp: remove redundant cast

removes redundant type casts in ARP-related network library functions.
The casts from req.arp_dev (already a char array) to (FAR char *)
are unnecessary and can be safely removed.

Signed-off-by: zhanghongyu <zhanghongyu@xiaomi.com>
This commit is contained in:
zhanghongyu 2024-12-23 13:42:20 +08:00 committed by Xiang Xiao
parent bb9229de06
commit d48b45000d
4 changed files with 4 additions and 7 deletions

View file

@ -76,8 +76,7 @@ int netlib_del_arpmapping(FAR const struct sockaddr_in *inaddr,
memset(&req.arp_ha, 0, sizeof(struct sockaddr_in));
if (ifname != NULL)
{
strlcpy((FAR char *)&req.arp_dev, ifname,
sizeof(req.arp_dev));
strlcpy(req.arp_dev, ifname, sizeof(req.arp_dev));
}
else
{

View file

@ -77,8 +77,7 @@ int netlib_get_arpmapping(FAR const struct sockaddr_in *inaddr,
memcpy(&req.arp_pa, inaddr, sizeof(struct sockaddr_in));
if (ifname != NULL)
{
strlcpy((FAR char *)&req.arp_dev, ifname,
sizeof(req.arp_dev));
strlcpy(req.arp_dev, ifname, sizeof(req.arp_dev));
}
else
{

View file

@ -81,8 +81,7 @@ int netlib_set_arpmapping(FAR const struct sockaddr_in *inaddr,
req.arp_flags = ATF_PERM;
if (ifname != NULL)
{
strlcpy((FAR char *)&req.arp_dev, ifname,
sizeof(req.arp_dev));
strlcpy(req.arp_dev, ifname, sizeof(req.arp_dev));
}
else
{

View file

@ -1310,7 +1310,7 @@ int cmd_arp(FAR struct nsh_vtbl_s *vtbl, int argc, FAR char **argv)
FAR uint8_t *ptr;
if (ifname != NULL &&
strcmp(ifname, (FAR char *)arptab[i].arp_dev) != 0)
strcmp(ifname, arptab[i].arp_dev) != 0)
{
continue;
}