On Sat, 17 Nov 2007 11:11:55 -0600 "Nathan Ingersoll" <[EMAIL PROTECTED]>
babbled:

> Is this a proper fix or hiding a bug in the calling application (or
> another part of the lib)? I would think the application should be
> responsible of knowing when they should unref the connection.

personally i like defensive programming. it's better than every process having
to trap its segv's/aborts and give u a debug trace and suddenly lose all your
work/state. i think a compile or runtime definable "please btich to stdout/err
about these problems" is the best way to go (all of evas and ecore are very
defensively programmed - set environment variables to turn on "bitch mode" and
you can also enable "abort mode" for aborting on caught failures).

> On 11/16/07, Enlightenment CVS <[EMAIL PROTECTED]> wrote:
> > Enlightenment CVS committal
> >
> > Author  : ravenlock
> > Project : e17
> > Module  : libs/e_dbus
> >
> > Dir     : e17/libs/e_dbus/src/lib/dbus
> >
> >
> > Modified Files:
> >         e_dbus.c
> >
> >
> > Log Message:
> > Ive gotten a segfault or two here.
> >
> > ===================================================================
> > RCS file: /cvs/e/e17/libs/e_dbus/src/lib/dbus/e_dbus.c,v
> > retrieving revision 1.2
> > retrieving revision 1.3
> > diff -u -3 -r1.2 -r1.3
> > --- e_dbus.c    4 Nov 2007 07:51:36 -0000       1.2
> > +++ e_dbus.c    17 Nov 2007 03:26:08 -0000      1.3
> > @@ -548,6 +548,7 @@
> >  EAPI void
> >  e_dbus_connection_unref(E_DBus_Connection *conn)
> >  {
> > +  if (!conn) return;
> >    if (--(conn->refcount) == 0) e_dbus_connection_close(conn);
> >  }
> >
> >
> >
> >
> > -------------------------------------------------------------------------
> > This SF.net email is sponsored by: Microsoft
> > Defy all challenges. Microsoft(R) Visual Studio 2005.
> > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> > _______________________________________________
> > enlightenment-cvs mailing list
> > [EMAIL PROTECTED]
> > https://lists.sourceforge.net/lists/listinfo/enlightenment-cvs
> >
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2005.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> _______________________________________________
> enlightenment-devel mailing list
> enlightenment-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> 


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    [EMAIL PROTECTED]


-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to