Hi Kay, > [PIMPL] > I tend to disagree here (oh well, there do not seem to be too many > topics for which we seem to have similar opinions, anyway :-),
:) > if you do > not want to show the internals of your implementation, I suggest to > introduce an "interface" and to provide a public factory function only. > Providing a public class having a impl pointer member only, seems to be > useless. Sure, there are other means to hide implementation. All have advantages and disadvantages (e.g. if somebody prefers inheritance over composition, then factories for interfaces are not the way to go). I consider PIMPL one valid solution for certain situations (e.g. extending a complex class hierarchy - with millions of lines of code, grown in decades - without completely rewriting it :). >>I'm not sure about this. As outline above, I don't think we really >>address those in the code. > > OK, if you say so, I believe you ..., so it is all by design :-) *also* by design, but not only. >>I agree that the current state, without complete dependecies, causes >>trouble and wasted time on developer side, in those cases where you >>_missed_ a dependency which you should have handled manually. And yes, >>the less experienced a developer is, the more probable it is that s/he >>will be bit by this. > > Just think of the myriads of open source developers wanting to help us ... They already have an OOo build env with complete dependencies, so what stops them from joining? SCNR :) >>However, in my opinion, the increased average efficiency in the daily >>work is worth this trouble from time to time ... > > Mmmm, several times I had to fix "bugs" which just were build problems, > because of not understood dependencies. Sure, me too. I don't say there are no drawbacks, I'm just saying they might be smaller than if we had complete dependencies (in the current build env). Ciao Frank -- - Frank Schönheit, Software Engineer [EMAIL PROTECTED] - - Sun Microsystems http://www.sun.com/staroffice - - OpenOffice.org Database http://dba.openoffice.org - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
