Re: [PATCH 1/2] wpa_supplicant: src: Remove the word "whitelist"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks Johannes for the feedback, I think this would solve the problem.

On Wed, Jan 12, 2022 at 12:12 AM Johannes Berg
<johannes@xxxxxxxxxxxxxxxx> wrote:
>
> On Tue, 2022-01-11 at 16:06 -0800, Arowa Suliman wrote:
> > First of all, thanks for the review, I submitted a v2 patch list for
> > the comments changes. However, for the enum values how can we get
> > acknowledgement from existing users of the interface? should I submit
> > another patch list for these changes and keep it under review to get
> > other users feedback?
>
> This won't address your "code base must be free of these words" needs,
> but the right way to rename enums would seem to be to offer a #define
> for the old name, e.g.
>
> -       QCA_WLAN_VENDOR_ATTR_ROAMING_PARAM_WHITE_LIST_SSID_NUM_NETWORKS = 3,
> +       QCA_WLAN_VENDOR_ATTR_ROAMING_PARAM_ALLOW_LIST_SSID_NUM_NETWORKS = 3,
>
> ...
>
> +/* old names for API compatibility */
> +#define QCA_WLAN_VENDOR_ATTR_ROAMING_PARAM_WHITE_LIST_SSID_NUM_NETWORKS QCA_WLAN_VENDOR_ATTR_ROAMING_PARAM_ALLOW_LIST_SSID_NUM_NETWORKS
> ...
>
> johannes

_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux