Re: [E-devel] Model-View-Controller

2014-07-18 Thread Carlos Carvalho
lready declared enum Foo? > IMO this is an ugly thing in C (typedef struct _A A), no need to repeat it > in Eo. > > enum Foo {} would be simpler, no? > > Unless there's a very good reason for it, of course :) > > Cheers, > > > > > > > > > > &

Re: [E-devel] Model-View-Controller

2014-07-07 Thread Carlos Carvalho
x27;t seem correct and I am thinking to make the API to return a NULL-terminated array of strings instead. What do you think? []s On Wed, Jul 2, 2014 at 7:35 PM, Gustavo Sverzut Barbieri wrote: > On Tue, Jul 1, 2014 at 7:08 PM, Carlos Carvalho > wrote: > > Hi gustavo, > > &

Re: [E-devel] Model-View-Controller

2014-07-07 Thread Carlos Carvalho
Hi Raster! On Tue, Jul 1, 2014 at 6:36 AM, Carsten Haitzler wrote: > On Tue, 24 Jun 2014 12:25:53 -0300 Carlos Carvalho > said: > > child_select() and child_select_eval() are basically not documented. i can > guess that they may create a EMODEL_EVENT_CHILD_SELECTED later..

Re: [E-devel] Model-View-Controller

2014-07-01 Thread Carlos Carvalho
odel Eio was make a "real > assync" model, to have a best case to make/tests "views" and examples, not > make a definitive model for "file objects" and therefore this choice for > name > > If nobody oppose we will fix/change first the points 1, 2, 3,

Re: [E-devel] Model-View-Controller

2014-06-27 Thread Carlos Carvalho
Hi! Some review items already pushed: 4, 7, 13.1, 14 and partially 9 (const void* and doc). []s On Thu, Jun 26, 2014 at 3:14 PM, Carlos Carvalho < ccarva...@expertisesolutions.com.br> wrote: > Hi Gustavo, > > First of all thank you very much for your review! > > I, person

Re: [E-devel] Model-View-Controller

2014-06-26 Thread Carlos Carvalho
to incremental calls, in > that case you can save lookup time by providing the last item as > start_pos). >- children_match(start, match_cb): allows matching every children > that matches, like to provide filtering in "search as you type", etc. > > All of thos

Re: [E-devel] Model-View-Controller

2014-06-24 Thread Carlos Carvalho
sesolutions/elementary.git branch eo_mvc I'll appreciate if you can take a second look :) []s! On Thu, Jun 19, 2014 at 11:39 PM, Carsten Haitzler wrote: > On Thu, 19 Jun 2014 12:20:21 -0300 Carlos Carvalho > said: > > > Hi raster :) > > > > please see below. >

Re: [E-devel] Model-View-Controller

2014-06-19 Thread Carlos Carvalho
s/don't though/didn't think/s :) On Thu, Jun 19, 2014 at 12:20 PM, Carlos Carvalho < ccarva...@expertisesolutions.com.br> wrote: > Hi raster :) > > please see below. > > On Thu, Jun 19, 2014 at 4:09 AM, Carsten Haitzler > wrote: > >> On Mon, 16 Jun

Re: [E-devel] Model-View-Controller

2014-06-19 Thread Carlos Carvalho
Hi raster :) please see below. On Thu, Jun 19, 2014 at 4:09 AM, Carsten Haitzler wrote: > On Mon, 16 Jun 2014 12:00:20 -0300 Felipe Magno de Almeida > said: > > Hey guys. just reading now. looking at emodel.eo a lot of these methods > have no > return just looking at properties for starter

Re: [E-devel] Model-View-Controller

2014-06-18 Thread Carlos Carvalho
Please before anyone review the code make sure to be up-to-date because I'm pushing some fixes, code cleanup and so on... []s! On Wed, Jun 18, 2014 at 6:28 AM, Tom Hacohen wrote: > On 17/06/14 20:25, Cedric BAIL wrote: > > Hello, > > > > So I did just a very quick high level review for now and

Re: [E-devel] Model-View-Controller

2014-06-17 Thread Carlos Carvalho
Hi Cedric, Please read in-line. On Tue, Jun 17, 2014 at 4:25 PM, Cedric BAIL wrote: > Hello, > > So I did just a very quick high level review for now and the following > is just my opinion. But I do think it makes more sense to have what > you name emodel be part of either Eo directly or a new