Now if you wanted to be uber-efficient, you'd code it this way:
body {
background: #e7d693;
}
a:visited,
a:link,
a:hover, /* not necessary since hover is undefined by default */
a:active {
color: green;
}
The less code you write, the less characters are being transfered to the user agent. That way, the browser does more work rendering and reading CSS (slightly more) and your server sends less data.
Just a random post, really. Just felt like improving your immense amount of {color: green}. :p