Am Wed, 5 Nov 2008 21:50:09 +0100 schrieb Jonathan MULLER:
Hello Jonathan,
I reviewed and commited your patch. Well done!
One comment:
Please provide another patch with some doxygen comments where you think
it's useful. Other will be very thankful. :-)
regards
Andreas
> Hi Andreas,
>
> You c
> Hello Jonathan,
Hi,
> I reviewed and commited your patch. Well done!
> One comment:
> Please provide another patch with some doxygen comments where you think
> it's useful. Other will be very thankful. :-)
No problem, be sure I will do it this week.
Regards.
Jonathan Muller.
--
Drylm Found
Am Sat, 1 Nov 2008 21:14:57 +0100 schrieb Jonathan MULLER:
Hello Jonathan,
I reviewed the source code short and this it looks good. But there's a
little problem. Until now the eflpp library hasn't any boost dependency.
Using your eet wrapper would create a dependency to boost for all
application
Hi Andreas,
I understand those problematics of course.
The dependency is on tuple classe with 3 elements ... we could in a
first time considering a std::pair of std::pair
boost::tuple < Ty, int, bool > ~ std::pair < std::pair< Ty, int >, bool >
It will be a bit more heavy for the user ... but I
Ok andreas,
I will do it with a pair of pair ... but I found the syntax very
heavier than with boost::tuple.
Take a look at the example in attached file, this is the approach I
will take to replace the boost::tuple.
And this approach is not very usable if we have to extend the "tuple".
Regards,
Am Mon, 3 Nov 2008 09:56:45 +0100 schrieb Jonathan MULLER:
Hello Jonathan,
would be nice if you do that. And best provide a path to integrate it
into eflpp.
BTW: Seems you're not a member of
enlightenment-devel@lists.sourceforge.net list. You should join the
list if you like to post anything
re
Am Sat, 1 Nov 2008 21:14:57 +0100 schrieb Jonathan MULLER:
Hello Jonathan,
I reviewed the source code short and this it looks good. But there's a
little problem. Until now the eflpp library hasn't any boost dependency.
Using your eet wrapper would create a dependency to boost for all
application