Would it make sense to have 'href' matching in one place: so in the selector or inside the function?
Now the positive matching ('a[href*="action=edit"][href*="&redlink"]') is in the selector, and negative matching (href.indexOf( 'veaction' ) === -1) in the function.