On Tue, 1 Jan 2013 22:06:02 +0900 Carsten Haitzler (The Rasterman) <[email protected]> wrote:
> On Tue, 1 Jan 2013 12:26:01 +0000 Michael Blumenkrantz > <[email protected]> said: > > > On Tue, 1 Jan 2013 19:47:43 +0900 > > Carsten Haitzler (The Rasterman) <[email protected]> wrote: > > > > > On Tue, 1 Jan 2013 08:32:39 +0000 Michael Blumenkrantz > > > <[email protected]> said: > > > > > > > On Tue, 1 Jan 2013 00:12:10 -0800 > > > > "Enlightenment SVN" <[email protected]> wrote: > > > > > > > > > Log: > > > > > misplaced ERR-- - maybe DBG? > > > > > > > > > > > > > > > > > > > > Author: raster > > > > > Date: 2013-01-01 00:12:09 -0800 (Tue, 01 Jan 2013) > > > > > New Revision: 81945 > > > > > Trac: http://trac.enlightenment.org/e/changeset/81945 > > > > > > > > > > Modified: > > > > > trunk/efl/src/lib/ecore_con/ecore_con.c > > > > > > > > > > Modified: trunk/efl/src/lib/ecore_con/ecore_con.c > > > > > =================================================================== > > > > > --- trunk/efl/src/lib/ecore_con/ecore_con.c 2012-12-31 23:45:24 > > > > > UTC (rev 81944) +++ trunk/efl/src/lib/ecore_con/ecore_con.c > > > > > 2013-01-01 08:12:09 UTC (rev 81945) @@ -1205,7 +1205,8 @@ > > > > > > > > > > e->client = cl; > > > > > e->error = strdup(error); > > > > > - ERR("%s", error); > > > > > +// givving errors like this is not a good thing - MAYBET his belongs > > > > > in > > > > > debug... but not err. +// ERR("%s", error); > > > > > cl->event_count = eina_list_append(cl->event_count, e); > > > > > cl->host_server->event_count = eina_list_append > > > > > (cl->host_server->event_count, e); ecore_event_add > > > > > (ECORE_CON_EVENT_CLIENT_ERROR, e, > > > > > (Ecore_End_Cb)_ecore_con_event_client_error_free, cl->host_server); > > > > > > > > > > > > > > > > > > wasn't misplaced. > > > > > > your connection dies... you shouldnt get stderr output. the app deals with > > > it. at best it's DBG. > > > > > > example: set EINA_ERROR_ABORT and u are trying to detect actual code > > > errors.. and your app aborts because it loses connction to another process > > > that exited... that's not an app error. or a library error - it's debug or > > > info at best. err is to indicatiin library/app inconsistencies internally > > > that should not happen and are something to go debug. :) > > > > > > > at the time that I wrote the code, it was worth having for debugging > > purposes. since there are fewer bugs now, it's less relevant > > indeed - that was my comment. it is more for debug, not actual err. :) ie > maybe > should be DBG() or INF() maybe? > > probably WRN ------------------------------------------------------------------------------ Master SQL Server Development, Administration, T-SQL, SSAS, SSIS, SSRS and more. Get SQL Server skills now (including 2012) with LearnDevNow - 200+ hours of step-by-step video tutorials by Microsoft MVPs and experts. SALE $99.99 this month only - learn more at: http://p.sf.net/sfu/learnmore_122512 _______________________________________________ enlightenment-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
