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 622247 - Refactored code to use CleanRoomStartup
Refactored code to use CleanRoomStartup
Status: RESOLVED FIXED
Product: f-spot
Classification: Other
Component: General
GIT
Other Linux
: Normal enhancement
: 0.7.1
Assigned To: F-spot maintainers
F-spot maintainers
Depends on:
Blocks:
 
 
Reported: 2010-06-20 23:01 UTC by Evan
Modified: 2010-06-22 07:17 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch for CleanRoomStartup (5.81 KB, patch)
2010-06-20 23:03 UTC, Evan 		none Details | Review
fixed a conditional (5.83 KB, patch)
2010-06-21 00:13 UTC, Evan 		committed Details | Review

Description Evan 2010-06-20 23:01:54 UTC 
Refactored out code in the Main method into a Startup method that is called by the CleanRoomStartup.
Comment 1 Evan 2010-06-20 23:03:55 UTC 
Created attachment 164183 [details] [review]
Patch for CleanRoomStartup
Comment 2 Evan 2010-06-21 00:13:57 UTC 
Created attachment 164187 [details] [review]
fixed a conditional
Comment 3 Ruben Vermeersch 2010-06-22 07:16:42 UTC 
Review of attachment 164187 [details] [review]:

Patch looks good, merging. I opened the related bug 622352.

Thanks a lot, I really appreciate cleanup patches!

::: src/main.cs
@@ +135,3 @@
 			args = FixArgs (args);
 
+			ApplicationContext.TrySetProcessName (Defines.PACKAGE);

Good catch!