I was wondering if it is recommended to eliminate all redundant opengl calls.
For instance, should I be doing something like (wrapped in a function):
You should probably avoid it. Here (specifically 22.040, you'll have to scroll down the page since there's no anchor for it), it is noted that redundant calls are generally bad practice even though some implementations attempt to minimize the impact.
Tools like gDEBugger can be used to help track down where you might be making redundant calls, too, if you're in a position where you're not sure how many you may be making in your code base.