Much better, though you shouldn't need the "if (button) then ... end" wrapper around the contents of your AuraButton_Update hook, since the default UI should never be calling update functions for buttons that don't exist. Leaving it won't really affect performance, it just adds an extra level of indentation for no particular reason. Smaller amounts are usually better when it comes to code.