2012/5/30 Kai Huuhko <kai.huu...@gmail.com>: > 2012/5/30 Davide Andreoli <d...@gurumeditation.it>: >> 2012/5/29 Kai Huuhko <kai.huu...@gmail.com>: >>> Currently the python bindings have get/set functions as in the C api, >>> in addition to this it has object properties which have their >>> functionality implemented by simply calling those functions, such as >>> in the below example: >>> >>> def remember_position_set(self, remember): >>> elm_video_remember_position_set(self.obj, remember) >>> >>> def remember_position_get(self): >>> return bool(elm_video_remember_position_get(self.obj)) >>> >>> property remember_position: >>> def __get__(self): >>> return self.remember_position_get() >>> def __set__(self, remember): >>> self.remember_position_set(remember) >>> >>> I propose that the get/set functions are removed from the python api >>> and replaced by the properties, so the above example simply becomes: >>> >>> property remember_position: >>> def __get__(self): >>> return bool(elm_video_remember_position_get(self.obj)) >>> def __set__(self, remember): >>> elm_video_remember_position_set(self.obj, remember) >>> >>> Please tell me what you think about this change. >>> >> >> I disagree here, we have compatibility layer in all the efl pybindings, >> don't see a reason to not follow the scheme here. >> I like/prefer to write using the c style functions usually, please >> leave at the user the choice :P >> ...I know is annoying to write properties > > Having both the get/set functions and a property for one functionality > is confusing for new pythonic efl developers. We've already revamped > much of the py-elm bindings breaking many existing applications in the > process, this would be a good time to do more damage by simplifying > the python api.
I have 2 applications written in py, one is 10000 lines of py code... do you want to rewrite ALL my apps for me ?? :P Really: You are proposing to remove 90% of the functions from the bindings. I would say: no way. Also we discussed this long time ago and we choosed to ALWAYS keep c "compatibility" in all the bindings > > In the spirit of this, I'm also looking into replacing the py-elm > callback_xxx_add/del functions with a single C api equivalent evas > smart_callback_add/del("xxx"... function. This also means that we > won't need to maintain the functions when signals are added or removed > from C. We discussed this some years ago, we decided to keep all the cb functions for the user, so you don't have to browse the docs to know what callbacks are available for the widget. I agree to add a generic func (smart_callback_add/del("xxx"...)) But I prefer to keep also the explicit ones for reference. > > If we are ever going to push for a release and wider acceptance of the > bindings, we need to make the py bindings api as simple as possible, > as feature complete as possible, and as well documented as possible. > Having them easier to maintain is a nice bonus too. In general I agree, BUT I don't want to break stuff more than necessary. IMO removing stuff just to make maintaining easier is not a good approach. > ------------------------------------------------------------------------------ > Live Security Virtual Conference > Exclusive live event will cover all the ways today's security and > threat landscape has changed and how IT managers can respond. Discussions > will include endpoint security, mobile security and the latest in malware > threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ > _______________________________________________ > enlightenment-devel mailing list > enlightenment-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel ------------------------------------------------------------------------------ Live Security Virtual Conference Exclusive live event will cover all the ways today's security and threat landscape has changed and how IT managers can respond. Discussions will include endpoint security, mobile security and the latest in malware threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/ _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel