None of these changes are necessary, or correct, unfortunately.
GetSpellName is already handled by an upvalue that selects the correct function on all relevant clients (C_Spell is not available on classic)
C_AddOns.GetAddOnEnableState swapped the arguments to have the addon name first, and character second, like the current code
Are you actually seeing any problems?
None of these changes are necessary, or correct, unfortunately.