On Tue, 18 Apr 2000, Bill Trost wrote: > My modest proposal, to add a second "dimension" to the message-handling > code by adding a pluggable list of strategies that each message type > uses, seems headed to a death of neglect. Brandon asked a good critical > question, and Dave Noha gave an answer and a positive response. How > do other developers feel? Should I give more details on what the code > would look like, or how it should get there? Do other core developers, > like Oskar and Ian, have any comments?
I don't think it is worth ripping up the code. I think you will find that you end up with a bunch of "strategies" that are used only once in the current code, and only a couple that appear a lot (and those will mostly already be handled by methods in the message class (sendBack(), sendReply()). If you can show some good examples of where this will make things easier and better then they are now, and list exactly what "strategies" you see repeated again and again, maybe I will consider it, but as it stands I don't think the message classes are sufficiently fucked to justify any sort of radical rework. There are a couple of things that could be cleaned up (Theo did some good things this weekend), especially a bunch of shared code between QueryRestarted, TimedOut, and InsertReply which should be pretty easy to solve by subclassing them together. > I think making this happen, or deciding it isn't, is important, because, > either way, I would like to start working on a UDP/multicast solution, > and would really like to do it *within* the strategy framework if that's > going to happen. __What UDP/multicast solution?___ I don't see that happening within the _Freenet_ framework. > _______________________________________________ > Freenet-dev mailing list > Freenet-dev at lists.sourceforge.net > http://lists.sourceforge.net/mailman/listinfo/freenet-dev -- Oskar Sandberg md98-osa at nada.kth.se #!/bin/perl -sp0777i<X+d*lMLa^*lN%0]dsXx++lMlN/dsM0<j]dsj $/=unpack('H*',$_);$_=`echo 16dio\U$k"SK$/SM$n\EsN0p[lN*1 lK[d2%Sa2/d0$^Ixp"|dc`;s/\W//g;$_=pack('H*',/((..)*)$/) _______________________________________________ Freenet-dev mailing list Freenet-dev at lists.sourceforge.net http://lists.sourceforge.net/mailman/listinfo/freenet-dev
