DC cmd issue

Post a reply

Confirmation code
Enter the code exactly as it appears. All letters are case insensitive.
Smilies
:D :) :( :o :shock: :? 8) :lol: :x :P :oops: :cry: :evil: :twisted: :roll: :wink: :!: :?: :idea: :arrow: :| :mrgreen:
BBCode is ON
[img] is ON
[flash] is OFF
[url] is ON
Smilies are ON
Topic review
   

Expand view Topic review: DC cmd issue

by Paco103 » April 11, 2005, 11:16 am

Valid point, and removes un-necessary exception checking and parsing. I think this change can be accomodated in the updated specs, which I will have posted by this afternoon.

DC cmd issue

by Alex » April 11, 2005, 5:21 am

Is there really any reason to have "DC 0" in there? Every other command has the command name on a line by itself, with DC being the sole exception. For simplicity in parsing, can we change this to one of the following:

DC
*

or

DC
0
*

Myself, I prefer the first. I don't see any reason to explicitly state that you're not sending new peers. That they're not there makes it implicitly clear enough.

Top