Hi Andrew, > > One more code-org-type item in the way of broker being done... The > > cpp/src/qpid/broker/Broker.cpp file's only (I believe) > diffs needed to > > go to Windows are that constructor (for the Poller creation), run(), > > and shutdown() methods - if I could split those methods out to > > platform-specific files and leave the rest in > broker/Broker.cpp, that > > would get Windows going and let the bulk of the Broker > implementation > > (quite a bit of important functionality) stay as-is now. In the > > future, there may be some management stuff that may be better split > > off, but for now this can get things going. What do you think about > > splitting off the implementation of these few small methods? > > I may be missing something, but there isn't anything platform specific > about them. The Poller creation, run and shutdown are platform > independent.
Right, but the Windows code has a different Poller, and needs to include a different Poller.h file. It's mainly the need to include a different file that is the difference here. > What is it that you think is different for windows about the Broker > class? Only the need to include a different file. -Steve
