Hi there,
I have reviewed the code and there is very little what I can say about =
it.
In fact, network programming is complitely unknow area for me.
So here are other comments.
1) bool should be replaced with Bool.
2) Files does not contain any documentation. They should have provided =
more
explanation for major classes/methods using Doxygen.
Regards,
mamutas
-----Original Message-----
From: xen...@li...
[mailto:xen...@li...] On Behalf Of
red...@pr...
Sent: Saturday, June 21, 2003 6:19 PM
To: xen...@li...
Subject: [Xenocide-programming] Network Library Peer Review
Well it is time US programmers to start using the list for code related=20
issues. That doesnt means you dont have to read the forums, cause the =
list
is=20
only for low level programming, not design issues. All that goes in the
forums.
Now I am sending the latest CVS (the same as in the CVS) headers for
review...=20
I want you to review it, every comment on interface style, posible =
problems,
posible uses, etc. Do it here... After the review process is done if we =
mess
it up, it is the whole team responsability. In short, lets kill this =
thing
in=20
the first shot.
For example if you want to write simple test apps, even though you =
cannot
make them work because the code is not there, feel free to do it cause =
we
need to=20
flesh out its inner working according at the use it will have. In short =
do:=20
Interface Usability Test something not everybody does.
Greetings
Red Knight
---
Incoming mail is certified Virus Free.
Checked by AVG anti-virus system (http://www.grisoft.com).
Version: 6.0.489 / Virus Database: 288 - Release Date: 6/10/2003
=20
=20
---
Outgoing mail is certified Virus Free.
Checked by AVG anti-virus system (http://www.grisoft.com).
Version: 6.0.489 / Virus Database: 288 - Release Date: 6/10/2003
=20
|
