On Thu, 2011-04-28 at 18:11 +0200, Milan Crha wrote: 
> Yes, when I was changing server side data-views for book and cal, then
> if turned out that the D-Bus API is exactly the same except of the
> DBUS_PROXY_NAME and book/cal strings in a file.
> 
> Thus having type param for both factories too, though for book unused,
> may clean the code very nicely, forcing us to exactly one implementation
> of base EGDBusFactory, EGDBusView, EDataFactory, EDataView and subclass
> these to EGDBusBookFactory, EGDBusCalFactory, ...  and changing only
> really minimum on them. Basically the effort which started Travis
> Reitter in his eds branches.

Excellent.  I'm sold then.  Sounds like the right thing to do.

This may also be of benefit to Srini and his email-factory branch.  I
can imagine extending the enum to include E_CLIENT_TYPE_MAIL_STORE or
something similar.

_______________________________________________
evolution-hackers mailing list
evolution-hackers@gnome.org
To change your list options or unsubscribe, visit ...
http://mail.gnome.org/mailman/listinfo/evolution-hackers

Reply via email to