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 769880 - retro-gobject: Refactor retro-environment* files
retro-gobject: Refactor retro-environment* files
Status: RESOLVED FIXED
Product: gnome-games
Classification: Core
Component: retro-gtk
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Games maintainers
Depends on:
Blocks:
 
 
Reported: 2016-08-14 08:57 UTC by Adrien Plazas
Modified: 2016-08-27 20:31 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
retro-gobject: Move retro-environment-core.h into retro-environment.c (10.61 KB, patch)
2016-08-14 08:58 UTC, Adrien Plazas
none Details | Review
retro-gobject: Move retro-environment-video.h into retro-environment.c (5.26 KB, patch)
2016-08-14 08:58 UTC, Adrien Plazas
none Details | Review
retro-gobject: Move retro-environment-input.h into retro-environment.c (4.34 KB, patch)
2016-08-14 08:59 UTC, Adrien Plazas
none Details | Review
retro-gobject: Move retro-environment-variable.h into retro-environment.c (4.84 KB, patch)
2016-08-14 08:59 UTC, Adrien Plazas
none Details | Review
retro-gobject: Move retro-environment-core.h into retro-environment.c (10.61 KB, patch)
2016-08-14 09:34 UTC, Adrien Plazas
committed Details | Review
retro-gobject: Move retro-environment-video.h into retro-environment.c (5.26 KB, patch)
2016-08-14 09:34 UTC, Adrien Plazas
committed Details | Review
retro-gobject: Move retro-environment-input.h into retro-environment.c (4.34 KB, patch)
2016-08-14 09:35 UTC, Adrien Plazas
committed Details | Review
retro-gobject: Move retro-environment-variable.h into retro-environment.c (4.84 KB, patch)
2016-08-14 09:35 UTC, Adrien Plazas
committed Details | Review
retro-gobject: Move retro-environment-interfaces.[ch] into retro-environment.c (29.27 KB, patch)
2016-08-14 09:35 UTC, Adrien Plazas
committed Details | Review

Description Adrien Plazas 2016-08-14 08:57:50 UTC
Libretro environment calls are handled by inline functions in private .h, which is a bit dirty.

Having these as static functions in a .c file would be cleaner as they would be private, in a singel file and the compiler would probably inline them anyway if they are static as they are meant to be used in only one place.
Comment 1 Adrien Plazas 2016-08-14 08:58:44 UTC
Created attachment 333267 [details] [review]
retro-gobject: Move retro-environment-core.h into retro-environment.c

This helps making the code cleaner.
Comment 2 Adrien Plazas 2016-08-14 08:58:51 UTC
Created attachment 333268 [details] [review]
retro-gobject: Move retro-environment-video.h into retro-environment.c

This helps making the code cleaner.
Comment 3 Adrien Plazas 2016-08-14 08:59:03 UTC
Created attachment 333269 [details] [review]
retro-gobject: Move retro-environment-input.h into retro-environment.c

This helps making the code cleaner.
Comment 4 Adrien Plazas 2016-08-14 08:59:20 UTC
Created attachment 333270 [details] [review]
retro-gobject: Move retro-environment-variable.h into retro-environment.c

This helps making the code cleaner.
Comment 5 Adrien Plazas 2016-08-14 09:34:50 UTC
Created attachment 333272 [details] [review]
retro-gobject: Move retro-environment-core.h into retro-environment.c

This helps making the code cleaner.
Comment 6 Adrien Plazas 2016-08-14 09:34:56 UTC
Created attachment 333273 [details] [review]
retro-gobject: Move retro-environment-video.h into retro-environment.c

This helps making the code cleaner.
Comment 7 Adrien Plazas 2016-08-14 09:35:01 UTC
Created attachment 333274 [details] [review]
retro-gobject: Move retro-environment-input.h into retro-environment.c

This helps making the code cleaner.
Comment 8 Adrien Plazas 2016-08-14 09:35:07 UTC
Created attachment 333275 [details] [review]
retro-gobject: Move retro-environment-variable.h into retro-environment.c

This helps making the code cleaner.
Comment 9 Adrien Plazas 2016-08-14 09:35:12 UTC
Created attachment 333276 [details] [review]
retro-gobject: Move retro-environment-interfaces.[ch] into retro-environment.c

This helps making the code cleaner.
Comment 10 Bastien Nocera 2016-08-14 16:00:35 UTC
Review of attachment 333272 [details] [review]:

Sure.
Comment 11 Bastien Nocera 2016-08-14 16:01:21 UTC
Review of attachment 333273 [details] [review]:

++
Comment 12 Bastien Nocera 2016-08-14 16:03:56 UTC
Review of attachment 333274 [details] [review]:

Sure.
Comment 13 Bastien Nocera 2016-08-14 16:04:29 UTC
Review of attachment 333275 [details] [review]:

++
Comment 14 Bastien Nocera 2016-08-14 16:05:09 UTC
Review of attachment 333276 [details] [review]:

++
Comment 15 Adrien Plazas 2016-08-27 20:31:31 UTC
Attachment 333272 [details] pushed as aa93463 - retro-gobject: Move retro-environment-core.h into retro-environment.c
Attachment 333273 [details] pushed as f119bad - retro-gobject: Move retro-environment-video.h into retro-environment.c
Attachment 333274 [details] pushed as 9a86bcb - retro-gobject: Move retro-environment-input.h into retro-environment.c
Attachment 333275 [details] pushed as 6e0ad76 - retro-gobject: Move retro-environment-variable.h into retro-environment.c
Attachment 333276 [details] pushed as 442a9b5 - retro-gobject: Move retro-environment-interfaces.[ch] into retro-environment.c