code without proper comments along header was considered to be a garbage and not worth a single cent
I think it is actually becoming more and more preferred that one writes code in a way that does not require comments. You often will hear things like "good code explains itself." I'm not too big a fan of this, but that is the direction the industry seems to be going in.
That being said, bitcoin does still fail here at times--especially when you encounter single lines where boost functions are passed to other boost functions to other boost functions in a way that quickly taxes your senses of what is really happening. The jazzy naming conventions also are unappealing to me. (Ex: CConnman. Just call it ConnectionManager ffs.) For the longest time everyone seemed to be perfectly okay with AppInit2 as a function name. Now we have AppInitMain, which isn't at all intuitively different from the name AppInit. Overall, the code base does seem to be improving. Two steps forward, one step back.