Hi Peter, I've fixed nearly all ("fixable") warnings in the Protocols::BitTorrent::{Bencoding,Torrents,Trackers} packages and I've found some things which I'd like to discuss. Protocols/BitTorrent/Trackers/TrackerRequestUrlCreator.cpp:110: warning: switch missing default case I haven't noticed that since (more...)

Hi Petr, Thanks for the comments! TransferTest.cpp:75/80 I think you can get rid of the local variable packetProcessors since it’s used only as a paramter of the Transfer ctor. In general yes, but how (more...)

Hi Peter, I've browsed the new code in your dev branch and here are some remarks/ideas. TransferTest.cpp:75/80 I think you can get rid of the local variable packetProcessors since it's used only as (more...)