Hi Atul, good points, as always! My initial intent was to make createPacket() a static function but then I thought it would be nice to define an abstract PacketFactory in Protocols::Generics (more...)

Hi Peter I am wondering if we need a separate class PacketFactory? Another design could be to make createPacket a static function of Packet class and use it ( this would make (more...)

Hi Peter Yes I agree with you... What I meant was there should be only one place where the switch happens ( and the create packet should return a Packet* but (more...)

Hello everybody, Here is yet another patch related to BitTorrent packets. I’ve implemented BitTorrent’s PacketFactory with extractPacket() and createPacket() methods. According to the design of PacketFactory, extractPacket() looks into the raw (more...)