Re: [E-devel] eina freeq?

2016-11-10 Thread The Rasterman
On Thu, 10 Nov 2016 15:14:11 + Mike Blumenkrantz said: > I see that this has been pushed and is already being used despite some > objections being raised? I guess I probably missed IRC discussions. did you read the responses to the objections? in fact just

Re: [E-devel] eina freeq?

2016-11-10 Thread Mike Blumenkrantz
I see that this has been pushed and is already being used despite some objections being raised? I guess I probably missed IRC discussions. The reasoning for needing this sounds like we should probably just use jemalloc (

Re: [E-devel] eina freeq?

2016-11-04 Thread The Rasterman
On Fri, 4 Nov 2016 10:18:33 -0200 Gustavo Sverzut Barbieri said: > On Thu, Nov 3, 2016 at 9:27 PM, Carsten Haitzler wrote: > > On Thu, 3 Nov 2016 11:24:14 -0200 Gustavo Sverzut Barbieri > > said: > > > >> I guessed mempool and

Re: [E-devel] eina freeq?

2016-11-04 Thread Gustavo Sverzut Barbieri
On Thu, Nov 3, 2016 at 9:27 PM, Carsten Haitzler wrote: > On Thu, 3 Nov 2016 11:24:14 -0200 Gustavo Sverzut Barbieri > > said: > >> I guessed mempool and eina_trash did that > > nah - mempool i don't think has a "purgatory" for pointers. > they are

Re: [E-devel] eina freeq?

2016-11-03 Thread The Rasterman
On Thu, 3 Nov 2016 11:24:14 -0200 Gustavo Sverzut Barbieri said: > I guessed mempool and eina_trash did that nah - mempool i don't think has a "purgatory" for pointers. they are released back into the pool. trash is actually a cache for storing ptrs but it never actually

Re: [E-devel] eina freeq?

2016-11-03 Thread Gustavo Sverzut Barbieri
I guessed mempool and eina_trash did that -- Gustavo Sverzut Barbieri > Em 3 de nov de 2016, às 05:53, Carsten Haitzler (The Rasterman) > escreveu: > > On Thu, 03 Nov 2016 09:35:21 +0200 Daniel Zaoui > said: > >> Well, my Lord, I hate that

Re: [E-devel] eina freeq?

2016-11-03 Thread The Rasterman
On Thu, 03 Nov 2016 09:35:21 +0200 Daniel Zaoui said: > Well, my Lord, I hate that idea. Do you want to make all EFL asynchronous? this isn't async. it's just deferred. we already do this for evas objects with delete_me. we do it for timers/animators and mark them for

Re: [E-devel] eina freeq?

2016-11-03 Thread Daniel Zaoui
Well, my Lord, I hate that idea. Do you want to make all EFL asynchronous? >From my point of view, seems to be like a hack cause some problems (e.g Eo) >are hard to solve. My comments below. On Thu, 03 Nov 2016 16:11:24 +0900 Carsten Haitzler (The Rasterman) wrote: >

[E-devel] eina freeq?

2016-11-03 Thread The Rasterman
here's an idea. it's very very very very simple create an eina_freeq(). instead of calling free() or whatever free function on something immediately, call: fq = eina_freeq_main_get(); eina_freeq_ptr_add(fq, pointer, size, free); or fq = eina_freeq_global_get();

Re: [E-devel] eina promise.... confusing

2016-05-31 Thread Felipe Magno de Almeida
On Tue, May 31, 2016 at 10:41 PM, Carsten Haitzler wrote: > [snip] > i just say - remove the void * for the job. same for timeout too - it's not > portable and the documentation is confusing as it claims to be data where it's > actually value... and value actually is a

Re: [E-devel] eina promise.... confusing

2016-05-31 Thread The Rasterman
On Tue, 31 May 2016 17:01:40 -0700 Cedric BAIL said: > On Tue, May 31, 2016 at 4:45 PM, Carsten Haitzler > wrote: > > On Tue, 31 May 2016 06:15:29 -0700 Cedric BAIL said: > > > >> On May 30, 2016 22:51, "Carsten Haitzler"

Re: [E-devel] eina promise.... confusing

2016-05-31 Thread Cedric BAIL
On Tue, May 31, 2016 at 4:45 PM, Carsten Haitzler wrote: > On Tue, 31 May 2016 06:15:29 -0700 Cedric BAIL said: > >> On May 30, 2016 22:51, "Carsten Haitzler" wrote: >> > >> > the api for promises seems pretty confusing. just look

Re: [E-devel] eina promise.... confusing

2016-05-31 Thread Cedric BAIL
On Tue, May 31, 2016 at 4:45 PM, Carsten Haitzler wrote: > On Tue, 31 May 2016 06:15:29 -0700 Cedric BAIL said: > >> On May 30, 2016 22:51, "Carsten Haitzler" wrote: >> > >> > the api for promises seems pretty confusing. just look

Re: [E-devel] eina promise.... confusing

2016-05-31 Thread The Rasterman
On Tue, 31 May 2016 11:48:32 -0300 Felipe Magno de Almeida said: > On Tue, May 31, 2016 at 10:15 AM, Cedric BAIL wrote: > > On May 30, 2016 22:51, "Carsten Haitzler" wrote: > >> > >> the api for promises seems pretty

Re: [E-devel] eina promise.... confusing

2016-05-31 Thread The Rasterman
On Tue, 31 May 2016 06:15:29 -0700 Cedric BAIL said: > On May 30, 2016 22:51, "Carsten Haitzler" wrote: > > > > the api for promises seems pretty confusing. just look at this: > > > > job = efl_loop_job(obj, args); > >eina_promise_then(job,

Re: [E-devel] eina promise.... confusing

2016-05-31 Thread Felipe Magno de Almeida
On Tue, May 31, 2016 at 10:15 AM, Cedric BAIL wrote: > On May 30, 2016 22:51, "Carsten Haitzler" wrote: >> >> the api for promises seems pretty confusing. just look at this: >> >> job = efl_loop_job(obj, args); >>eina_promise_then(job,

Re: [E-devel] eina promise.... confusing

2016-05-31 Thread Cedric BAIL
On May 30, 2016 22:51, "Carsten Haitzler" wrote: > > the api for promises seems pretty confusing. just look at this: > > job = efl_loop_job(obj, args); >eina_promise_then(job, _efl_loop_args_job_cb, NULL, args); > > why do i need to pass in args... TWICE? well ok -

[E-devel] eina promise.... confusing

2016-05-30 Thread The Rasterman
the api for promises seems pretty confusing. just look at this: job = efl_loop_job(obj, args); eina_promise_then(job, _efl_loop_args_job_cb, NULL, args); why do i need to pass in args... TWICE? well ok - this specific way of using promises for jobs... both promises in efl_loop.eo do this,

[E-devel] eina test suite segault - promises

2016-05-25 Thread The Rasterman
make check is failing now because the eina promise test is crashing. :( not good (tm). :( -- - Codito, ergo sum - "I code, therefore I am" -- The Rasterman (Carsten Haitzler)ras...@rasterman.com

Re: [E-devel] Eina value optional

2016-01-26 Thread Jean-Philippe André
On 27 January 2016 at 00:03, Felipe Magno de Almeida < felipe.m.alme...@gmail.com> wrote: > On Tue, Jan 26, 2016 at 12:51 PM, Tom Hacohen wrote: > > On 26/01/16 14:42, Stefan Schmidt wrote: > >> Hello. > > [snip] > > >> JP, Tom are you happy with Felipe's explanation? If

Re: [E-devel] Eina value optional

2016-01-26 Thread Stefan Schmidt
Hello. On 13/01/16 11:56, Tom Hacohen wrote: > On 13/01/16 02:38, Jean-Philippe André wrote: >> Hi Felipe, >> >> You added the optional type to eina value. I'm not sure what it's point is. >> I understand an optional value can be empty (ie. void and not "nil" or 0 or >> whatever). >> >> But I

Re: [E-devel] Eina value optional

2016-01-26 Thread Tom Hacohen
On 26/01/16 14:42, Stefan Schmidt wrote: > Hello. > > On 13/01/16 11:56, Tom Hacohen wrote: >> On 13/01/16 02:38, Jean-Philippe André wrote: >>> Hi Felipe, >>> >>> You added the optional type to eina value. I'm not sure what it's point is. >>> I understand an optional value can be empty (ie. void

Re: [E-devel] Eina value optional

2016-01-26 Thread Felipe Magno de Almeida
On Tue, Jan 26, 2016 at 12:51 PM, Tom Hacohen wrote: > On 26/01/16 14:42, Stefan Schmidt wrote: >> Hello. [snip] >> JP, Tom are you happy with Felipe's explanation? If not we need to act >> now as we only have 6 days left before the release and once it is in >> 1.17 it

Re: [E-devel] Eina value optional

2016-01-17 Thread Felipe Magno de Almeida
Hello, Sorry to answer myself, but complementing on what I've said before. There's also ABI and optimization's concern on adding empty value for all types. Since Eina_Value is defined in eina_value.h header, it is public and needs to be backwards compatible ABI-wise, and since it can be used by

Re: [E-devel] Eina value optional

2016-01-17 Thread Felipe Magno de Almeida
On Wed, Jan 13, 2016 at 12:38 AM, Jean-Philippe André wrote: > Hi Felipe, > > You added the optional type to eina value. I'm not sure what it's point is. > I understand an optional value can be empty (ie. void and not "nil" or 0 or > whatever). > > But I don't understand why

Re: [E-devel] Eina value optional

2016-01-13 Thread Tom Hacohen
On 13/01/16 02:38, Jean-Philippe André wrote: > Hi Felipe, > > You added the optional type to eina value. I'm not sure what it's point is. > I understand an optional value can be empty (ie. void and not "nil" or 0 or > whatever). > > But I don't understand why this couldn't be implemented inside

[E-devel] Eina value optional

2016-01-12 Thread Jean-Philippe André
Hi Felipe, You added the optional type to eina value. I'm not sure what it's point is. I understand an optional value can be empty (ie. void and not "nil" or 0 or whatever). But I don't understand why this couldn't be implemented inside all standard values. Add an "empty" property to them.

[E-devel] Eina blank line mapping

2015-03-26 Thread Andrew Williams
Hi, I registered a phab ticket about eina map_lines iterator not iterating through blank links https://phab.enlightenment.org/T2237 https://phab.enlightenment.org/T2237 I have a fix for this locally which I would love to push but before that happens It thought I had better ask: Does anyone

Re: [E-devel] [eina] Eina_List and Eina_Inlist eina_inlist_prepend_relative and eina_list_prepend_relative_list

2013-11-06 Thread Tom Hacohen
On 06/11/13 00:35, Felipe Magno de Almeida wrote: n On Tue, Nov 5, 2013 at 7:02 PM, Tom Hacohen t...@stosb.com wrote: That is not going to change. The first and most important reason is that we are *NOT* going to break API no matter how right you are. OK. Can't really argue with that.

[E-devel] [eina] Eina_List and Eina_Inlist eina_inlist_prepend_relative and eina_list_prepend_relative_list

2013-11-05 Thread Felipe Magno de Almeida
Hello, Both eina_inlist_prepend_relative and eina_list_prepend_relative_list takes three arguments: list, data and relative. If relative is NULL, both execute the non-relative version: eina_inlist_prepend and eina_list_prepend. This means that there's no way to make eina_inlist_prepend_relative

Re: [E-devel] [eina] Eina_List and Eina_Inlist eina_inlist_prepend_relative and eina_list_prepend_relative_list

2013-11-05 Thread Tom Hacohen
That is not going to change. The first and most important reason is that we are *NOT* going to break API no matter how right you are. The second reason is that we also have append_relative and having two functions that do the same would be redundant. The third reason is quite subjective, and

Re: [E-devel] [eina] Eina_List and Eina_Inlist eina_inlist_prepend_relative and eina_list_prepend_relative_list

2013-11-05 Thread The Rasterman
On Tue, 5 Nov 2013 18:23:42 -0200 Felipe Magno de Almeida felipe.m.alme...@gmail.com said: there is the other view that appeanding/prepending relative TO something.. if something is NULL.. then that something does not exist, so calling these calles with NULL as relative is silly. the behavior is

Re: [E-devel] [eina] Eina_List and Eina_Inlist eina_inlist_prepend_relative and eina_list_prepend_relative_list

2013-11-05 Thread Felipe Magno de Almeida
n On Tue, Nov 5, 2013 at 7:02 PM, Tom Hacohen t...@stosb.com wrote: That is not going to change. The first and most important reason is that we are *NOT* going to break API no matter how right you are. OK. Can't really argue with that. Though I don't really see how an application that

Re: [E-devel] Eina/Evas threading support?

2013-02-13 Thread Andreas Volz
Am Wed, 13 Feb 2013 15:56:57 +0900 schrieb Carsten Haitzler (The Rasterman): Could you please tell me how to disable threading support as in 1.7.5? Maybe even by hacking build environment. I played around, but some of the m4 macros are really a nightmare! we removed the ability - we're

Re: [E-devel] Eina/Evas threading support?

2013-02-12 Thread The Rasterman
On Sun, 10 Feb 2013 22:52:19 +0100 Andreas Volz li...@brachttal.net said: Hello, In the EFL 1.7.5 I had options to disable threading support and everything worked as it should. you know we stopped testing phtread off codepaths... and thats why efl now is pthread only :) that set of bull

[E-devel] Eina/Evas threading support?

2013-02-10 Thread Andreas Volz
Hello, In the EFL 1.7.5 I had options to disable threading support and everything worked as it should. The pthread version on Android misses some features from the Linux variant. For now I just like to disable it complete and take time for this task later. But it seems there's no option for it

Re: [E-devel] [Eina PATCH] Fix directory listing in eina_file_direct_ls()

2013-02-05 Thread Cedric BAIL
Cedric Bail On Feb 5, 2013 10:47 AM, Paulo Alcantara pca...@zytor.com wrote: Gustavo Sverzut Barbieri barbi...@profusion.mobi writes: This would be a PITA. And it would be slower on FS that provide dt_type. Actually it shouldn't,as statat is only called if d_type is unknown. Direct is for

[E-devel] [Eina PATCH] Fix directory listing in eina_file_direct_ls()

2013-02-04 Thread Paulo Alcantara
XFS and other filesystems that do not support d_type field in dirent structure will get unexpected behavior when using eina_file_direct_ls() - since it relies on d_type to determine file types. Thus, an eina_file_statat() call is required in eina_file_direct_ls() only if the file type had not

Re: [E-devel] [Eina PATCH] Fix directory listing in eina_file_direct_ls()

2013-02-04 Thread Cedric BAIL
On Tue, Feb 5, 2013 at 12:46 AM, Paulo Alcantara pca...@zytor.com wrote: XFS and other filesystems that do not support d_type field in dirent structure will get unexpected behavior when using eina_file_direct_ls() - since it relies on d_type to determine file types. Thus, an

Re: [E-devel] [Eina PATCH] Fix directory listing in eina_file_direct_ls()

2013-02-04 Thread Paulo Alcantara
Cedric BAIL cedric.b...@free.fr writes: No, this is the expected behavior. You should have used eina_file_stat_ls(). eina_file_direct_ls() is called in several places - which means that all callers won't get any file type correctly. So, should I replace all eina_file_direct_ls() calls with

Re: [E-devel] [Eina PATCH] Fix directory listing in eina_file_direct_ls()

2013-02-04 Thread Gustavo Sverzut Barbieri
On Monday, February 4, 2013, Paulo Alcantara wrote: Cedric BAIL cedric.b...@free.fr javascript:; writes: No, this is the expected behavior. You should have used eina_file_stat_ls(). eina_file_direct_ls() is called in several places - which means that all callers won't get any file type

Re: [E-devel] [Eina PATCH] Fix directory listing in eina_file_direct_ls()

2013-02-04 Thread Paulo Alcantara
Gustavo Sverzut Barbieri barbi...@profusion.mobi writes: This would be a PITA. And it would be slower on FS that provide dt_type. Indeed. So I'm going to replace eina_file_direct_ls() with eina_file_stat_ls() in some places that depends on file type (e.g., elementary_config). How about it ?

[E-devel] Eina Magic Check Failed Segfault

2013-01-07 Thread Jeff Hoogland
Some of the time when this block of code is run - http://paste.debian.net/222589/ My application gets a segfault with this error message - http://paste.debian.net/222591/ Here is the full code in case something else might be causing this - http://paste.debian.net/222590 Most of the time the

Re: [E-devel] Eina Magic Check Failed Segfault

2013-01-07 Thread Jeff Hoogland
OK so I've upgraded to the 1.7.5 EFL snapshot and at kuuko's advice I've upgraded my python bindings to SVN (except for python-evas that will not build against EFL 1.7.5). I now get this dump log which gives a bunch of feedback from the EFLs and the VLC backend I am using -

Re: [E-devel] eina benchmark and eina_file patches

2012-11-19 Thread Gustavo Lima Chaves
* Jérémy Zurcher jer...@asynk.ch [2012-11-18 01:24:27 +0100]: Hi, 0002-_eina_file_escape-take-care-of.patch takes care of '/./' and breaks loop on '/..$' 0001-sanitize-eina-benchmark-compilation.patch cleans up and fixes 'make benchmark' saddly eina_bench stills segv and as I

[E-devel] eina benchmark and eina_file patches

2012-11-17 Thread Jérémy Zurcher
Hi, 0002-_eina_file_escape-take-care-of.patch takes care of '/./' and breaks loop on '/..$' 0001-sanitize-eina-benchmark-compilation.patch cleans up and fixes 'make benchmark' saddly eina_bench stills segv and as I can't find any install target for it, and I'm not a kung-auto-foo

[E-devel] eina thread create abstraction?

2012-09-24 Thread Gustavo Sverzut Barbieri
Hi all, particularly Vincent :-) Eina provides mutex abstraction with eina_lock, however the thread creation is not there as most users should be using ecore_thread. All nice, but for evas threaded render, we'll need to create threads that are not related to the main loop/ecore, then I'm

Re: [E-devel] eina thread create abstraction?

2012-09-24 Thread Vincent Torri
On Mon, Sep 24, 2012 at 11:56 PM, Gustavo Sverzut Barbieri barbi...@profusion.mobi wrote: Hi all, particularly Vincent :-) Eina provides mutex abstraction with eina_lock, however the thread creation is not there as most users should be using ecore_thread. All nice, but for evas threaded

Re: [E-devel] eina thread create abstraction?

2012-09-24 Thread Cedric BAIL
On Tue, Sep 25, 2012 at 7:43 AM, Vincent Torri vincent.to...@gmail.com wrote: On Mon, Sep 24, 2012 at 11:56 PM, Gustavo Sverzut Barbieri barbi...@profusion.mobi wrote: Hi all, particularly Vincent :-) Eina provides mutex abstraction with eina_lock, however the thread creation is not there as

Re: [E-devel] Eina with --disable-safety-checks fails in compilation

2012-03-07 Thread The Rasterman
On Thu, 1 Mar 2012 09:08:46 +0100 Thanatermesis thanatermesis.e...@gmail.com said: unless someone fixed this... it works for me (atm) :) If you add the parameter --disable-safety-checks to the configure options of libeina, the compilation fails: $ make V=1 make[3]: Entering directory

[E-devel] Eina with --disable-safety-checks fails in compilation

2012-03-01 Thread Thanatermesis
If you add the parameter --disable-safety-checks to the configure options of libeina, the compilation fails: $ make V=1 make[3]: Entering directory `/mkdeb/build/libeina/libeina/src/lib' /bin/bash ../../libtool --tag=CC --mode=compile ccache gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I../..

[E-devel] eina got broke.

2012-01-19 Thread David Seikel
I'm getting lots of these while trying to compile latest SVN. *** glibc detected *** /home/dvs1/e17_svn/SVN/trunk/elementary/doc/widgets/.libs/lt-widget_preview_fileselector_button2: corrupted double-linked list: 0x01938f40 *** === Backtrace: =

Re: [E-devel] eina got broke.

2012-01-19 Thread Cedric BAIL
On Thu, Jan 19, 2012 at 2:38 PM, David Seikel onef...@gmail.com wrote: I'm getting lots of these while trying to compile latest SVN. What are you doing to trigger that error ? Can you get us a valgrind backtrace ? Thanks, *** glibc detected ***

Re: [E-devel] eina got broke.

2012-01-19 Thread Sanjeev B.A.
Just built everything in my home PC. WFM. On Thu, Jan 19, 2012 at 10:46 PM, Cedric BAIL cedric.b...@free.fr wrote: On Thu, Jan 19, 2012 at 2:38 PM, David Seikel onef...@gmail.com wrote: I'm getting lots of these while trying to compile latest SVN. What are you doing to trigger that error ?

Re: [E-devel] eina got broke. Or maybe elementary.

2012-01-19 Thread David Seikel
Now that I can actually investigate myself, it's happening when I run make doc in elementary/doc -- A big old stinking pile of genius that no one wants coz there are too many silver coated monkeys in the world. signature.asc Description: PGP signature

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-05 Thread The Rasterman
On Thu, 1 Dec 2011 17:41:17 +0100 Joerg Sonnenberger jo...@britannica.bec.de said: Hi all, sorry for not finding the time for doing this earlier. Attached is a patch that sorts out a number of smaller issues with eina on NetBSD and Solaris. Some are noise, some are real bugs. This brings it

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread The Rasterman
On Fri, 2 Dec 2011 08:42:43 +0100 (CET) Vincent Torri vto...@univ-evry.fr said: On Fri, 2 Dec 2011, Carsten Haitzler (The Rasterman) wrote: On Fri, 2 Dec 2011 08:28:36 +0100 (CET) Vincent Torri vto...@univ-evry.fr said: On Fri, 2 Dec 2011, Carsten Haitzler (The Rasterman)

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread Joerg Sonnenberger
On Fri, Dec 02, 2011 at 04:00:46PM +0900, Carsten Haitzler wrote: On Thu, 1 Dec 2011 17:41:17 +0100 Joerg Sonnenberger jo...@britannica.bec.de said: Hi all, sorry for not finding the time for doing this earlier. Attached is a patch that sorts out a number of smaller issues with eina on

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread Joerg Sonnenberger
On Fri, Dec 02, 2011 at 03:37:30PM +0900, Carsten Haitzler wrote: On Thu, 1 Dec 2011 17:41:17 +0100 Joerg Sonnenberger jo...@britannica.bec.de said: this is really late... and a lot of these patches i find questionable. i'll question here (things not questioned seem ok): 1. __SUNPRO_C

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread Sachiel
2011/12/2 Joerg Sonnenberger jo...@britannica.bec.de: On Fri, Dec 02, 2011 at 03:37:30PM +0900, Carsten Haitzler wrote: On Thu, 1 Dec 2011 17:41:17 +0100 Joerg Sonnenberger jo...@britannica.bec.de said: this is really late... and a lot of these patches i find questionable. i'll question

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread The Rasterman
On Fri, 2 Dec 2011 13:39:41 +0100 Joerg Sonnenberger jo...@britannica.bec.de said: On Fri, Dec 02, 2011 at 03:37:30PM +0900, Carsten Haitzler wrote: On Thu, 1 Dec 2011 17:41:17 +0100 Joerg Sonnenberger jo...@britannica.bec.de said: this is really late... and a lot of these patches i

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread Joerg Sonnenberger
On Fri, Dec 02, 2011 at 09:51:12PM +0900, Carsten Haitzler wrote: 2. all the chasting and changing to unsigned char for passing into isspace () - isspace() actually takes an int, not unsigned char, so this just doesn't make sense. :( (well it does according to the manual page i have)

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread Vincent Torri
On Fri, 2 Dec 2011, Joerg Sonnenberger wrote: On Fri, Dec 02, 2011 at 09:51:12PM +0900, Carsten Haitzler wrote: 2. all the chasting and changing to unsigned char for passing into isspace () - isspace() actually takes an int, not unsigned char, so this just doesn't make sense. :( (well it

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread Joerg Sonnenberger
On Fri, Dec 02, 2011 at 10:16:17PM +0100, Vincent Torri wrote: so why not just do a 0xff instead of casting ? The cast is IMO cleaner in the intentions. It also has the theoretical advantage of working independent of CHAR_BIT==8. Joerg

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread Sachiel
2011/12/2 Joerg Sonnenberger jo...@britannica.bec.de: On Fri, Dec 02, 2011 at 10:16:17PM +0100, Vincent Torri wrote: so why not just do a 0xff instead of casting ? The cast is IMO cleaner in the intentions. It also has the theoretical advantage of working independent of CHAR_BIT==8. We are

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread Vincent Torri
On Fri, 2 Dec 2011, Joerg Sonnenberger wrote: On Fri, Dec 02, 2011 at 10:16:17PM +0100, Vincent Torri wrote: so why not just do a 0xff instead of casting ? The cast is IMO cleaner in the intentions. It also has the theoretical advantage of working independent of CHAR_BIT==8. except some

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread Joerg Sonnenberger
On Fri, Dec 02, 2011 at 07:33:15PM -0200, Iván Briano (Sachiel) wrote: 2011/12/2 Joerg Sonnenberger jo...@britannica.bec.de: On Fri, Dec 02, 2011 at 10:16:17PM +0100, Vincent Torri wrote: so why not just do a 0xff instead of casting ? The cast is IMO cleaner in the intentions. It also

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-02 Thread Sachiel
2011/12/2 Joerg Sonnenberger jo...@britannica.bec.de: On Fri, Dec 02, 2011 at 07:33:15PM -0200, Iván Briano (Sachiel) wrote: 2011/12/2 Joerg Sonnenberger jo...@britannica.bec.de: On Fri, Dec 02, 2011 at 10:16:17PM +0100, Vincent Torri wrote: so why not just do a 0xff instead of casting ?

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-01 Thread The Rasterman
On Thu, 1 Dec 2011 17:41:17 +0100 Joerg Sonnenberger jo...@britannica.bec.de said: this is really late... and a lot of these patches i find questionable. i'll question here (things not questioned seem ok): 1. __SUNPRO_C instead of __sun - explain why you dont just ADD an extra ||

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-01 Thread The Rasterman
On Thu, 1 Dec 2011 17:41:17 +0100 Joerg Sonnenberger jo...@britannica.bec.de said: Hi all, sorry for not finding the time for doing this earlier. Attached is a patch that sorts out a number of smaller issues with eina on NetBSD and Solaris. Some are noise, some are real bugs. This brings it

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-01 Thread Vincent Torri
On Fri, 2 Dec 2011, Carsten Haitzler (The Rasterman) wrote: On Thu, 1 Dec 2011 17:41:17 +0100 Joerg Sonnenberger jo...@britannica.bec.de said: this is really late... and a lot of these patches i find questionable. i'll question here (things not questioned seem ok): 1. __SUNPRO_C instead

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-01 Thread The Rasterman
On Fri, 2 Dec 2011 08:28:36 +0100 (CET) Vincent Torri vto...@univ-evry.fr said: On Fri, 2 Dec 2011, Carsten Haitzler (The Rasterman) wrote: On Thu, 1 Dec 2011 17:41:17 +0100 Joerg Sonnenberger jo...@britannica.bec.de said: this is really late... and a lot of these patches i find

Re: [E-devel] eina 1.1.0beta patches for/from pkgsrc

2011-12-01 Thread Vincent Torri
On Fri, 2 Dec 2011, Carsten Haitzler (The Rasterman) wrote: On Fri, 2 Dec 2011 08:28:36 +0100 (CET) Vincent Torri vto...@univ-evry.fr said: On Fri, 2 Dec 2011, Carsten Haitzler (The Rasterman) wrote: On Thu, 1 Dec 2011 17:41:17 +0100 Joerg Sonnenberger jo...@britannica.bec.de said:

Re: [E-devel] Eina Coverage tests

2011-05-30 Thread The Rasterman
On Tue, 17 May 2011 17:56:16 +0300 Tom Hacohen tom.haco...@partner.samsung.com said: eina_prefix actually is hard to test given the test harness as it relies on an installation following a certain standard :( can't really do it given the test setup. Hey, I just ran eina coverage and a lot of

Re: [E-devel] Eina Coverage tests

2011-05-30 Thread Tom Hacohen
:( On Mon, 2011-05-30 at 17:59 +0900, Carsten Haitzler wrote: On Tue, 17 May 2011 17:56:16 +0300 Tom Hacohen tom.haco...@partner.samsung.com said: eina_prefix actually is hard to test given the test harness as it relies on an installation following a certain standard :( can't really do it

Re: [E-devel] Eina Coverage tests

2011-05-30 Thread Gustavo Sverzut Barbieri
On Mon, May 30, 2011 at 5:59 AM, Carsten Haitzler ras...@rasterman.com wrote: On Tue, 17 May 2011 17:56:16 +0300 Tom Hacohen tom.haco...@partner.samsung.com said: eina_prefix actually is hard to test given the test harness as it relies on an installation following a certain standard :( can't

Re: [E-devel] Eina Coverage tests

2011-05-30 Thread The Rasterman
On Mon, 30 May 2011 11:54:22 -0300 Gustavo Sverzut Barbieri barbi...@profusion.mobi said: On Mon, May 30, 2011 at 5:59 AM, Carsten Haitzler ras...@rasterman.com wrote: On Tue, 17 May 2011 17:56:16 +0300 Tom Hacohen tom.haco...@partner.samsung.com said: eina_prefix actually is hard to

[E-devel] Eina Coverage tests

2011-05-17 Thread Tom Hacohen
Hey, I just ran eina coverage and a lot of the code is not tested :( Mainly: eina_file, eina_log, eina_module, eina_object, eina_prefix, eina_simple_xml_parser, and eina_str. Whoever is responsible for each part, please fix. Thanks, Tom.

Re: [E-devel] Eina Coverage tests

2011-05-17 Thread Gustavo Sverzut Barbieri
On Tue, May 17, 2011 at 11:56 AM, Tom Hacohen tom.haco...@partner.samsung.com wrote: Hey, I just ran eina coverage and a lot of the code is not tested :( Mainly: eina_file, eina_log, eina_module, eina_object, eina_prefix, eina_simple_xml_parser, and eina_str. Whoever is responsible for

Re: [E-devel] Eina Coverage tests

2011-05-17 Thread Tom Hacohen
On Tue, 2011-05-17 at 12:22 -0300, Gustavo Sverzut Barbieri wrote: all the new parts! how unexpected :-D I did a test with my initial simple_xml_parser, someone can find out in the mail and convert it in a test? Sorry, don't have time for that. Everyone who added new stuff but didn't add

Re: [E-devel] Eina Coverage tests

2011-05-17 Thread Gustavo Sverzut Barbieri
I did not add it :) On May 17, 2011 12:48 PM, Tom Hacohen tom.haco...@partner.samsung.com wrote: On Tue, 2011-05-17 at 12:22 -0300, Gustavo Sverzut Barbieri wrote: all the new parts! how unexpect... Sorry, don't have time for that. Everyone who added new stuff but didn't add tests, go and fix

Re: [E-devel] Eina Coverage tests

2011-05-17 Thread sangho park
o.. sorry.. 'eina_simple_xml_parse' was requested by me.. i'll add test to eina. Date: Tue, 17 May 2011 13:29:28 -0300 From: Gustavo Sverzut Barbieri barbi...@profusion.mobi Subject: Re: [E-devel] Eina Coverage tests To: Tom Hacohen tom.haco...@partner.samsung.com Cc: enlightenment

[E-devel] Eina: keeping .la files for mempools modules

2011-04-24 Thread Vincent Torri
Hey we are deleting the .la files for the mempool modules. I think that we should provide them. It's the packager of the distro who has to decide whether or not to delete it. What do you think ? Vincent --

Re: [E-devel] Eina: keeping .la files for mempools modules

2011-04-24 Thread The Rasterman
On Sun, 24 Apr 2011 08:13:19 +0200 (CEST) Vincent Torri vto...@univ-evry.fr said: Hey we are deleting the .la files for the mempool modules. I think that we should provide them. It's the packager of the distro who has to decide whether or not to delete it. What do you think ? only

Re: [E-devel] Eina: keeping .la files for mempools modules

2011-04-24 Thread Vincent Torri
On Sun, 24 Apr 2011, Carsten Haitzler (The Rasterman) wrote: On Sun, 24 Apr 2011 08:13:19 +0200 (CEST) Vincent Torri vto...@univ-evry.fr said: Hey we are deleting the .la files for the mempool modules. I think that we should provide them. It's the packager of the distro who has to

Re: [E-devel] Eina: keeping .la files for mempools modules

2011-04-24 Thread The Rasterman
On Sun, 24 Apr 2011 08:29:22 +0200 (CEST) Vincent Torri vto...@univ-evry.fr said: On Sun, 24 Apr 2011, Carsten Haitzler (The Rasterman) wrote: On Sun, 24 Apr 2011 08:13:19 +0200 (CEST) Vincent Torri vto...@univ-evry.fr said: Hey we are deleting the .la files for the mempool

Re: [E-devel] Eina: keeping .la files for mempools modules

2011-04-24 Thread Vincent Torri
On Sun, 24 Apr 2011, Carsten Haitzler (The Rasterman) wrote: On Sun, 24 Apr 2011 08:29:22 +0200 (CEST) Vincent Torri vto...@univ-evry.fr said: On Sun, 24 Apr 2011, Carsten Haitzler (The Rasterman) wrote: On Sun, 24 Apr 2011 08:13:19 +0200 (CEST) Vincent Torri vto...@univ-evry.fr said:

Re: [E-devel] Eina: keeping .la files for mempools modules

2011-04-24 Thread The Rasterman
On Sun, 24 Apr 2011 08:55:14 +0200 (CEST) Vincent Torri vto...@univ-evry.fr said: On Sun, 24 Apr 2011, Carsten Haitzler (The Rasterman) wrote: On Sun, 24 Apr 2011 08:29:22 +0200 (CEST) Vincent Torri vto...@univ-evry.fr said: On Sun, 24 Apr 2011, Carsten Haitzler (The Rasterman)

[E-devel] eina error proposal

2011-03-12 Thread Mike Blumenkrantz
Is there a reason why we don't use stringshare for allocated eina_error messages? I am planning to add a function like eina_error_eq(Error1, Error2) that would be nice to use with stringshare to avoid strcmp where possible... -- Mike Blumenkrantz Zentific: NULL pointer dereferences now 50% off!

Re: [E-devel] eina error proposal

2011-03-12 Thread Mike Blumenkrantz
On Sat, 12 Mar 2011 14:02:15 -0500 Mike Blumenkrantz m...@zentific.com wrote: Is there a reason why we don't use stringshare for allocated eina_error messages? I am planning to add a function like eina_error_eq(Error1, Error2) that would be nice to use with stringshare to avoid strcmp where

[E-devel] [eina] warning-- on Windows 64 bits

2010-10-18 Thread Vincent Torri
Hey, below is a patch to remove warnings on Windows 64. Indeed, on this arch, long int is 32bits and not 64bits long. No objection to commit it ? Vincent Index: src/lib/eina_magic.c === --- src/lib/eina_magic.c(revision

[E-devel] Eina Arg Not NULL should die

2010-09-07 Thread Brett Nash
Hello All, I'm proposing we remove all the EINA_ARG_NOTNULLs floating around in public headers. In short they may generate a warning, but they subvert any checks we have in the API, and can cause valid code to break. The not null macro expands to the GCC attribute notnull. From the GCC

Re: [E-devel] Eina Arg Not NULL should die

2010-09-07 Thread Gustavo Sverzut Barbieri
On Tue, Sep 7, 2010 at 4:36 AM, Brett Nash n...@nash.id.au wrote: Hello All, I'm proposing we remove all the EINA_ARG_NOTNULLs floating around in public headers. In short they may generate a warning, but they subvert any checks we have in the API, and can cause valid code to break. The not

[E-devel] eina broken in alpha

2010-08-27 Thread Michael Blumenkrantz
Just fyi, the alpha release of eina is broken when you try to compile some of the modules due to the move of eina_private.h from include/ to lib/. Whoever moved this should have done a test compile. -- Mike Blumenkrantz Zentific: Our boolean values are huge.

Re: [E-devel] eina broken in alpha

2010-08-27 Thread Vincent Torri
On Fri, 27 Aug 2010, Michael Blumenkrantz wrote: Just fyi, the alpha release of eina is broken when you try to compile some of the modules due to the move of eina_private.h from include/ to lib/. Whoever moved this should have done a test compile. I think that now we are in alpha, before

Re: [E-devel] Eina - location of the .h's and .x's

2010-08-17 Thread Vincent Torri
On Tue, 17 Aug 2010, Tom Hacohen wrote: Hey all, I have heard today that in Eina, all the .h's and .x's should be located in the src/include dir, even if they are *not* shipped. That's ugly, stupid and confusing. There's no reason why to put headers or include files that should not be

[E-devel] eina types and thread safety

2010-08-14 Thread Vincent Torri
Hey Can someone explain me the interest of having the eina data types thread safe ? For me, there is no reason until they are used in the EFL and thread safety is then a requirement (for a reason or another) So, i'm wondering if we should have thread safety in them. It adds more complexity

[E-devel] eina array: missing guards

2010-08-13 Thread Vincent Torri
hey, in eina_array.c, lines 761 and 808, some threaded code is not guarded Vincent -- This SF.net email is sponsored by Make an app they can't live without Enter the BlackBerry Developer Challenge

Re: [E-devel] eina array: missing guards

2010-08-13 Thread Michael Blumenkrantz
On Fri, 13 Aug 2010 19:21:12 +0200 (CEST) Vincent Torri vto...@univ-evry.fr wrote: hey, in eina_array.c, lines 761 and 808, some threaded code is not guarded Vincent -- This SF.net email is sponsored by

Re: [E-devel] eina array: missing guards

2010-08-13 Thread Michael Blumenkrantz
On Fri, 13 Aug 2010 22:37:40 -0400 Michael Blumenkrantz m...@zentific.com wrote: On Fri, 13 Aug 2010 19:21:12 +0200 (CEST) Vincent Torri vto...@univ-evry.fr wrote: hey, in eina_array.c, lines 761 and 808, some threaded code is not guarded Vincent

  1   2   3   >