ok, my last post on comemnting your code caused a bit of a reaction (which is a good thing — always glad to get other people’s points of view). But, just to show what I have been dealing with, heres a small sample:

(whoops – i broke my blog posting this code — see the extended entry).

and this is just one function of many scattered across various timelines. Now, this defintely goes along with some of the other comments in teh last post about meaningful variable names and all that…but i think a comment here and there in this one wouldnt have hurt.

bc[as]. function stateselected(thestate) {
var sf;
var ds;
var xs;
var ys;
if (not swos) {
trace(“state selected =”+thestate);
curstate = thestate;
as.ss2.sh.statewinners.removeAll();
bs.ss2.sh.biowinners.removeAll();
//More Info”, datalist[i][0]);
bs.ss2.sh.biowinners.addItem(datalist[i][10][1]+”, “+datalist[i][9][0]);
}
}
}
}
}
}