DevDevin

Very cool project, but... Documentation is spare and comments in the code are almost non-existent. I would not call the design/code very elegant either. I see functions with >200 lines of code. That is a code smell for me. I see try/catch blocks with no exception handling. (and no comments) WTF? Also, I don't see any unit tests, Yikes!

I still think this is a great open source project and a lot can be learned from it. I just wouldn't characterize the code as "elegant". My $.02.

Points well-taken. What I meant by an elegant design was really at the architectural level. I think that it solves the problem with elegance in that regard. If you break it down to the documentation/commenting level then I'd agree that there is much room for improvement. The nice thing, however, is that I think the code reads very cleanly and that is probably why the author decided not to riddle comments throughout. After all, if you read some of the most well-regarded books on the subject many of them say that well-written code becomes self-documenting. The no unit tests part is a good point. I'll have to see whether I can convince someone to put those up as well. They do exist! :) Also, if you see places where there is no exception handling I guarantee you that a bug was logged, a debate ensued, and that an explanation was documented in the tracking system. Sorry that it didn't make it to the production code. Also a function with over 200 lines of code should definitely be subject to scrutiny, but these things are never absolute. I'd be interested to know which function you think needs to be refactored. Last, but not least, this is open source now so feel free to enhance it to your heart's content.

This looks like something written in C back in 1996, just translated to C#.

If I coded something this bad, I'd be fired. Microsoft instead exalts it by making it a "Contribution" to the "Community" - yeah right.

> "Therefore, I can't claim rights to the elegant design, but I can claim rights to some of the quality--hopefully not the lack thereof!"

I wouldn't call this elegant in the slightest. This is a total hack job. If I were you I'd pull this down right away.

It's been a couple years since I went Googling for "distributed event system" and I'm shocked to see this turd still hanging around.

If this is any example of the dreck Microsoft keeps locked away in their "security through obscurity" paradigm, it's no wonder that Windows Vista is such a train wreck.

Good grief - don't call this "open source" call it "please review my total lack of skill."