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

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

 



May I ask - if there's no way to get rid of these words without breaking backward compatibility and breaking backward compatibility is not an option (or is it?) then what's the point of going through all these hoops and jumps in the first place?

Cheers,
=S.


On 12/01/2022 10.12, Johannes Berg 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