On Tue, 07 Nov 2017 14:18:04 +0000 Andrew Williams <a...@andywilliams.me> said:
> Hi, > > Indeed on later reading I see it was lacking in detail so I followed up > later. > > It’s unclear if, following rasters last email, he is still in agreement > with the _count() change but it seems like an improvement so I intend to go > forward with it as discussed. i'm fine with it. i was just explaining why i disagreed in detail about it being misleading. :) > Thanks, > Andy > On Tue, 7 Nov 2017 at 13:57, Mike Blumenkrantz < > michael.blumenkra...@gmail.com> wrote: > > > In this case I would agree with raster; the original mail could have > > provided a little more detail because it is non-obvious to the reader why > > this could be misleading for users. > > > > efl_ref_count() is in line with the rest of our naming (e.g., > > eina_array_count(), eina_list_count(), eina_hash_population(), etc) and > > seems like a clear improvement. Nice work. > > > > On Tue, Nov 7, 2017 at 7:41 AM Carsten Haitzler <ras...@rasterman.com> > > wrote: > > > > > On Mon, 06 Nov 2017 09:53:32 +0000 Andrew Williams <a...@andywilliams.me > > > > > > said: > > > > > > > Can you please explain why? This is a method that returns a count of > > > > references but has no mention of count in the name. > > > > > > Well your original email gave no details as to why it's misleading. I > > > disagree > > > because i see something like: > > > > > > Eo *efl_ref(Eo *obj); > > > int efl_ref_get(Eo *obj); > > > void efl_unref(Eo *obj); > > > ... > > > > > > i.e. the context of the api, and to me it screams of "ref is reference or > > > references" and since there is something to ref and unref already ref_get > > > returning an int obviously is returning a reference count (to me). since > > > i'm > > > familiar with reference counting techniques in general (nothing to do > > with > > > the > > > code implementation or api here), to me it's obvious that this returns > > the > > > reference count (an integer) and it's short form for that. > > > > > > if i saw this api in glib or gtk or libjpeg or libpng or zlib or any > > other > > > library i'd think the same with zero documentation and just api > > signatures > > > like > > > the above. > > > > > > i've learned api's by just reading .h files (like xlib) and seeing this > > and > > > taking a guess as to what it means. an int generally to me indicates it's > > > returning a number as opposed to something opaque in higher level libs > > like > > > efl/gtk etc.. it has meaning. libc calls return ints and are different as > > > they > > > tend to mean an error or success, but look at glib or gtk or others and > > > they > > > dont do this and efl doesn't do the libc thing either as a whole. > > > > > > so given the context as a whole and as a sub-part of the api like > > above... > > > i > > > disagree that its misleading. to me its pretty obvious. maybe because > > i've > > > spent a lot of time with various api's and so on i tend to fill in the > > > blanks > > > quickly seeing this kind of information. i don't know. > > > > > > but if you think it's misleading can you give more information? why is it > > > misleading? what is it leading you to think it is and why is that? what > > is > > > the > > > difference in view point or experience and how conclusions are being > > > attained? > > > > > > > We have ref_add and wref_add that are talking about actual references > > but > > > > ref_get returns a count? > > > > This is misleading. > > > > > > > > Andy > > > > > > > > On Mon, 6 Nov 2017 at 09:49 Carsten Haitzler <ras...@rasterman.com> > > > wrote: > > > > > > > > > On Mon, 06 Nov 2017 09:46:41 +0000 Andrew Williams < > > > a...@andywilliams.me> > > > > > said: > > > > > > > > > > i'd have to disagree on it being misleading... :/ > > > > > > > > > > > Hi, > > > > > > > > > > > > Whilst writing the docs we have realised that efl_ref_get is a > > > slightly > > > > > > misleading method name as it returns the reference count. Whilst we > > > are > > > > > > breaking APIs for our first interfaces release would people mind me > > > > > > changing efl_ref_get to efl_refcount_get (as per this eo patch, and > > > all > > > > > the > > > > > > efl ramifications). > > > > > > > > > > > > I'd prefer not to leave the old method in place but could do so if > > > you > > > > > > think there is a big problem making this breaking change? > > > > > > > > > > > > Thanks :) > > > > > > Andy > > > > > > > > > > > > > > > > > > -- > > > > > > http://andywilliams.me > > > > > > http://ajwillia.ms > > > > > > > > > > > > > > > -- > > > > > ------------- Codito, ergo sum - "I code, therefore I am" > > > -------------- > > > > > Carsten Haitzler - ras...@rasterman.com > > > > > > > > > > -- > > > > http://andywilliams.me > > > > http://ajwillia.ms > > > > > > > > > ------------------------------------------------------------------------------ > > > > Check out the vibrant tech community on one of the world's most > > > > engaging tech sites, Slashdot.org! http://sdm.link/slashdot > > > > _______________________________________________ > > > > 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" -------------- > > > Carsten Haitzler - ras...@rasterman.com > > > > > > > > > > > > > > ------------------------------------------------------------------------------ > > > Check out the vibrant tech community on one of the world's most > > > engaging tech sites, Slashdot.org! http://sdm.link/slashdot > > > _______________________________________________ > > > enlightenment-devel mailing list > > > enlightenment-devel@lists.sourceforge.net > > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > > > > > ------------------------------------------------------------------------------ > > Check out the vibrant tech community on one of the world's most > > engaging tech sites, Slashdot.org! http://sdm.link/slashdot > > _______________________________________________ > > enlightenment-devel mailing list > > enlightenment-devel@lists.sourceforge.net > > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel > > > -- > http://andywilliams.me > http://ajwillia.ms > ------------------------------------------------------------------------------ > Check out the vibrant tech community on one of the world's most > engaging tech sites, Slashdot.org! http://sdm.link/slashdot > _______________________________________________ > 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" -------------- Carsten Haitzler - ras...@rasterman.com ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link/slashdot _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
