You are viewing a plain text version of this content. The canonical link for it is here.

Posted to proton@qpid.apache.org by uromahn <ul...@ulrichromahn.net> on 2014/09/18 10:17:16 UTC

Any idea why MessengerFactory and MessengerFactoryImpl have disappeared

Hi Everybody!

when comparing the latest trunk in SVN (and the git mirror) to the 0.7
release branch, I noticed that the MessengerFactory interface and
corresponding implementation MessengerFactoryImpl have been completely
removed from the source.
I suspect that this must have happened in error in a commit from Rafael H.
Schloming on May 1st 2014.

As part of a good API, it is good practice to provide service factory
methods so developer can abstract themselves from actual implementations. By
the way, IMHO it wasn't a good idea to put MessengerFactoryImpl in the
"impl" package to begin with. It would be better located in the top
"messenger" package since a Factory implementation is part of a public API.

Just being curious,
-Uli



--
View this message in context: http://qpid.2158936.n2.nabble.com/Any-idea-why-MessengerFactory-and-MessengerFactoryImpl-have-disappeared-tp7613797.html
Sent from the Apache Qpid Proton mailing list archive at Nabble.com.