[gtk+] gtkquartz: don't free the string returned by get_bundle_path()
- From: Michael Natterer <mitch src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gtk+] gtkquartz: don't free the string returned by get_bundle_path()
- Date: Sat, 22 Oct 2011 21:25:18 +0000 (UTC)
commit 88ad614c735a92f8e0b029e2b5070bf0f8db5016
Author: Michael Natterer <mitch gimp org>
Date: Sat Oct 22 23:08:32 2011 +0200
gtkquartz: don't free the string returned by get_bundle_path()
It's statically allocated.
(cherry picked from commit c040b03c2e32a773a4d9cf4019050c2f8a5b91ce)
gtk/gtkquartz.c | 43 +++++++++++++++++++------------------------
1 files changed, 19 insertions(+), 24 deletions(-)
---
diff --git a/gtk/gtkquartz.c b/gtk/gtkquartz.c
index 771309c..7f36e9e 100644
--- a/gtk/gtkquartz.c
+++ b/gtk/gtkquartz.c
@@ -318,24 +318,28 @@ _gtk_quartz_set_selection_data_for_pasteboard (NSPasteboard *pasteboard,
* to test for that and remove the last element.
*/
-static gchar *
-get_bundle_path ()
+static const gchar *
+get_bundle_path (void)
{
static gchar *path = NULL;
+
if (path == NULL)
{
- gchar *base;
NSAutoreleasePool *pool = [[NSAutoreleasePool alloc] init];
gchar *resource_path = g_strdup ([[[NSBundle mainBundle] resourcePath] UTF8String]);
+ gchar *base;
[pool drain];
+
base = g_path_get_basename (resource_path);
if (strcmp (base, "bin") == 0)
path = g_path_get_dirname (resource_path);
else
path = strdup (resource_path);
+
g_free (resource_path);
g_free (base);
}
+
return path;
}
@@ -343,12 +347,10 @@ const gchar *
_gtk_get_datadir (void)
{
static gchar *path = NULL;
+
if (path == NULL)
- {
- gchar *resource_dir = get_bundle_path ();
- path = g_build_filename (resource_dir, "share", NULL);
- g_free (resource_dir);
- }
+ path = g_build_filename (get_bundle_path (), "share", NULL);
+
return path;
}
@@ -356,12 +358,10 @@ const gchar *
_gtk_get_libdir (void)
{
static gchar *path = NULL;
+
if (path == NULL)
- {
- gchar *resource_dir = get_bundle_path ();
- path = g_build_filename (resource_dir, "lib", NULL);
- g_free (resource_dir);
- }
+ path = g_build_filename (get_bundle_path (), "lib", NULL);
+
return path;
}
@@ -369,12 +369,10 @@ const gchar *
_gtk_get_localedir (void)
{
static gchar *path = NULL;
+
if (path == NULL)
- {
- gchar *resource_dir = get_bundle_path ();
- path = g_build_filename (resource_dir, "share", "locale", NULL);
- g_free (resource_dir);
- }
+ path = g_build_filename (get_bundle_path (), "share", "locale", NULL);
+
return path;
}
@@ -382,12 +380,10 @@ const gchar *
_gtk_get_sysconfdir (void)
{
static gchar *path = NULL;
+
if (path == NULL)
- {
- gchar *resource_dir = get_bundle_path ();
- path = g_build_filename (resource_dir, "etc", NULL);
- g_free (resource_dir);
- }
+ path = g_build_filename (get_bundle_path (), "etc", NULL);
+
return path;
}
@@ -396,4 +392,3 @@ _gtk_get_data_prefix (void)
{
return get_bundle_path ();
}
-
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]