After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 584431 - Wrap g_uri_list_extract_uris
Wrap g_uri_list_extract_uris
Status: RESOLVED FIXED
Product: pygobject
Classification: Bindings
Component: general
Git master
Other Linux
: Normal normal
: ---
Assigned To: Nobody's working on this now (help wanted and appreciated)
Python bindings maintainers
Depends on:
Blocks:
 
 
Reported: 2009-06-01 10:13 UTC by Tomeu Vizoso
Modified: 2010-09-24 07:24 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
proposed wrapper (2.83 KB, patch)
2009-06-01 10:14 UTC, Tomeu Vizoso 		needs-work Details | Review
address concerns adn add testcase (4.08 KB, patch)
2009-07-18 17:36 UTC, Tomeu Vizoso 		committed Details | Review

Description Tomeu Vizoso 2009-06-01 10:13:56 UTC 
This format is simple but not completely trivial and if all receivers of uri lists used this func to extract the individual uris, several interoperability bugs could be avoided.
Comment 1 Tomeu Vizoso 2009-06-01 10:14:51 UTC 
Created attachment 135714 [details] [review]
proposed wrapper
Comment 2 Gian Mario Tagliaretti 2009-07-18 16:05:14 UTC 
Tomeu, thanks for the patch, from a quick look here a couple of comments:

you should free the returned string array with g_strfreev()
it'd prefer to return None instead of an empty tuple
please provide a test case for the wrapper
Comment 3 Tomeu Vizoso 2009-07-18 17:36:18 UTC 
Created attachment 138690 [details] [review]
address concerns adn add testcase
Comment 4 Tomeu Vizoso 2010-05-23 10:00:57 UTC 
Ping :)
Comment 5 johnp 2010-09-23 16:35:06 UTC 
Comment on attachment 138690 [details] [review]
address concerns adn add testcase

Looks good