'q66' on IRC; Software Engineer at Samsung Electronics Czech (working for SRUK Open Source Group); OctaForge project maintainer, Elua maintainer, and a heavy Lua (ab)user.
- User Since
- Apr 12 2013, 5:06 PM (335 w, 4 d)
Fri, Sep 13
as @segfaultxavi said
Thu, Sep 12
@raster that's because requires is not extends, requires pretty much specifies a list of classes that are required to be in inheritance tree of the thing using the mixin; by specifying it, you tell eolian that it's guaranteed it'll be there, and the mixin can safely implement it.
Tue, Sep 10
As of afc011d8931006bd020ba5130a581580f709edcf, usage of ptr() is banned in stable API. That's bound to be good enough for now, and removing it from beta is a separate task, so closing this as resolved.
Mon, Sep 9
Fri, Sep 6
this doesn't correctly fix the eolian_type_c_type_get usage yet, which means this will behave wrong with the tag; i still see EOLIAN_C_TYPE_DEFAULT/PARAM/RETURN being passed around.
OK, I did that for now, but we'll need to figure out a proper one...
Thu, Sep 5
I think we've got this down, eolian properly checks and verifies this
this is now banned in stable API... removal still pending
Thu, Aug 29
Hm, what should be do about this? Do we want this to be thread-safe? Because the error API is not...
Closing this as the system is generally sound. Syntax changes may happen but no design alterations
Wed, Aug 28
Please rebase for the @class -> @static method change
Okay, but I'm not sure why you assigned this to me, considering I'm not dealing with widgets.
Aug 16 2019
Aug 14 2019
Aug 9 2019
Hm? why not just rename current @class to @static (which is easy and quick) and then only re-add @class later if it ends up being needed?
If it's just renaming to @static, then that's fine and easy
Jul 26 2019
Jul 23 2019
I think this is good
Jul 8 2019
But the proper thing to do later on would be to create a wrapper type with the same layout as Eina_Slice (or Eina_Rw_Slice) but make it templated in order to respect Eolian's typeinfo. But for now this is fine
Jun 26 2019
Jun 24 2019
I added an initial syntax for this in the commit above. Please comment if you have any suggestions, objections etc.
May 30 2019
May 28 2019
I landed the patch so this can proceed probably.
Just running tests before landing this now.
Hm, I think it's fine.
May 16 2019
May 14 2019
Looking good to me
May 9 2019
Any update on this? Can it be closed? As far as Eolian is concerned it's been done for a while
This has been done for a while.
May 6 2019
May 5 2019
May 3 2019
Apr 24 2019
No, it's not intended, but all you need to do is simply remove the first is_enum check and leave the second one where it is.
Apr 23 2019
The goto is unnecessary, syntax error means a long jump
Apr 19 2019
that helps, i'll add guards for it once I am back
Apr 14 2019
Mar 21 2019
Mar 20 2019
you still missed one
You can't just eina_hash_add into nhash without checking !oobj because if you do and the entry is already present, that's undefined behavior for our hash implementation, so either keep the if or make them eina_hash_set
Mar 18 2019
That works too
I intend to do away with const() as well as ptr() in the current form entirely, so having a new syntax is the better choice for it, besides, immutable by default is a better choice than mutable by default