On Fri, 2018-04-20 at 00:32 +0200, Bernhard Gabler wrote: > looking at the patch, I see more occurences of the outdated "IE" in a > constant name and function name: > > case NL80211_REGDOM_SET_BY_COUNTRY_IE: > or > reg_process_hint_country_ie() Right. I searched for further strings that would show up to the user, but didn't see any, but the code still refers to it in many places. > Would it make sense to likewise fix these (and possibly more) > occurrences too, to match the spec from 2016? > For the sake of readability and consistency of code ... I dunno. Country IE isn't the only place where "IE" is used, it's even used in struct names (struct ieee80211_tim_ie, and many others), so I would think that people who actually work on the code will have to be aware of the previous wording of "Information Element" (and the abbreviation IE) anyway. Changing *everything* here to be just _element instead of _ie or then probably the shorter _elem would IMHO be too much code churn. johannes
