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 617215 - Merge the gio document loader and saver into document loader and saver
Merge the gio document loader and saver into document loader and saver
Status: RESOLVED FIXED
Product: gedit
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Gedit maintainers
Gedit maintainers
Depends on:
Blocks:
 
 
Reported: 2010-04-29 21:01 UTC by Garrett Regier
Modified: 2010-04-30 07:36 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Merge it together (98.77 KB, patch)
2010-04-29 21:01 UTC, Garrett Regier 		none Details | Review
Merge the gio document loader and saver into document loader and saver (98.89 KB, patch)
2010-04-30 06:19 UTC, Garrett Regier 		committed Details | Review

Description Garrett Regier 2010-04-29 21:01:12 UTC 
Created attachment 159945 [details] [review]
Merge it together

Don't need the abstraction anymore so merge it together.
Comment 1 Paolo Borelli 2010-04-29 21:14:21 UTC 
Review of attachment 159945 [details] [review]:

this was discussed on irc and Garret promptly provided a patch.

It looks good to me, though I did not test it (yet)
Comment 2 jessevdk@gmail.com 2010-04-29 22:46:37 UTC 
The patch doesn't apply. Could you try to rebase it on the latest master?
Comment 3 Garrett Regier 2010-04-30 06:19:58 UTC 
Created attachment 159959 [details] [review]
Merge the gio document loader and saver into document loader and saver
Comment 4 jessevdk@gmail.com 2010-04-30 07:36:31 UTC 
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.