[LTP] [PATCH v3 1/3] Create separate .c file for include/tst_net.h

Cyril Hrubis chrubis@suse.cz
Mon Mar 16 15:18:42 CET 2020


Hi!
> >> +int tst_bit_count(uint32_t i);
> >> +int tst_mask2prefix(struct in_addr mask);
> >> +int tst_ipv4_mask_to_int(const char *prefix);
> >> +int tst_safe_atoi(const char *s, int *ret_i);
> >> +int tst_get_prefix(const char *ip_str, int is_ipv6);
> >> +void tst_get_in_addr(const char *ip_str, struct in_addr *ip);
> >> +void tst_get_in6_addr(const char *ip_str, struct in6_addr *ip6);
> > 
> > Do we need all of these exported in the public API?
> > 
> > It looks to me like most of these shouldn't be exported at all.
> 
> I guess that's a question for Petr. I'll be happy to drop the functions
> meant to be private from the .h file and make them static in the .c file.

Let's do that, there is no point in exporting unused functions.

-- 
Cyril Hrubis
chrubis@suse.cz


More information about the ltp mailing list