]> Pileus Git - ~andy/gtk/commitdiff
No need for INCLUDE_INTERNAL_SYMBOLS anymore.
authorMatthias Clasen <mclasen@redhat.com>
Tue, 17 Aug 2004 18:24:06 +0000 (18:24 +0000)
committerMatthias Clasen <matthiasc@src.gnome.org>
Tue, 17 Aug 2004 18:24:06 +0000 (18:24 +0000)
2004-08-17  Matthias Clasen  <mclasen@redhat.com>

* gtk/abicheck.sh: No need for INCLUDE_INTERNAL_SYMBOLS anymore.

* gdk/gdk.symbols: Don't use #if defined().

* gdk/Makefile.am (gdkalias.h):
* gtk/Makefile.am (gtkalias.h): Don't use cpp to filter gtk.symbols.

* gdk/makegdkalias.pl:
* gtk/makegtkalias.pl: Move the #ifdef processing into the perl script, and
keep the #ifdefs which differentiate between platforms.

* gtk/Makefile.am (gtk_private_h_sources): Remove gtkinternals.h, it is no
longer needed.

16 files changed:
ChangeLog
ChangeLog.pre-2-10
ChangeLog.pre-2-6
ChangeLog.pre-2-8
gdk-pixbuf/ChangeLog
gdk-pixbuf/Makefile.am
gdk-pixbuf/abicheck.sh
gdk-pixbuf/gdk-pixbuf.symbols
gdk-pixbuf/makegdkpixbufalias.pl
gdk/Makefile.am
gdk/gdk.symbols
gdk/makegdkalias.pl
gtk/Makefile.am
gtk/abicheck.sh
gtk/gtkinternals.h [deleted file]
gtk/makegtkalias.pl

index 009da4090649fc943825c4e5493a7ee79da23b9e..a306d32efa83869076b37cbc188142b1390b706f 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,19 @@
+2004-08-17  Matthias Clasen  <mclasen@redhat.com>
+
+       * gtk/abicheck.sh: No need for INCLUDE_INTERNAL_SYMBOLS anymore.
+
+       * gdk/gdk.symbols: Don't use #if defined().
+
+       * gdk/Makefile.am (gdkalias.h): 
+       * gtk/Makefile.am (gtkalias.h): Don't use cpp to filter gtk.symbols.
+
+       * gdk/makegdkalias.pl: 
+       * gtk/makegtkalias.pl: Move the #ifdef processing into the perl script, and
+       keep the #ifdefs which differentiate between platforms.
+
+       * gtk/Makefile.am (gtk_private_h_sources): Remove gtkinternals.h, it is no 
+       longer needed.
+
 2004-08-17  Matthias Clasen  <mclasen@redhat.com>
 
        * tests/testfilechooserbutton.c: Use g_message() throughout, add RTL envar for
index 009da4090649fc943825c4e5493a7ee79da23b9e..a306d32efa83869076b37cbc188142b1390b706f 100644 (file)
@@ -1,3 +1,19 @@
+2004-08-17  Matthias Clasen  <mclasen@redhat.com>
+
+       * gtk/abicheck.sh: No need for INCLUDE_INTERNAL_SYMBOLS anymore.
+
+       * gdk/gdk.symbols: Don't use #if defined().
+
+       * gdk/Makefile.am (gdkalias.h): 
+       * gtk/Makefile.am (gtkalias.h): Don't use cpp to filter gtk.symbols.
+
+       * gdk/makegdkalias.pl: 
+       * gtk/makegtkalias.pl: Move the #ifdef processing into the perl script, and
+       keep the #ifdefs which differentiate between platforms.
+
+       * gtk/Makefile.am (gtk_private_h_sources): Remove gtkinternals.h, it is no 
+       longer needed.
+
 2004-08-17  Matthias Clasen  <mclasen@redhat.com>
 
        * tests/testfilechooserbutton.c: Use g_message() throughout, add RTL envar for
index 009da4090649fc943825c4e5493a7ee79da23b9e..a306d32efa83869076b37cbc188142b1390b706f 100644 (file)
@@ -1,3 +1,19 @@
+2004-08-17  Matthias Clasen  <mclasen@redhat.com>
+
+       * gtk/abicheck.sh: No need for INCLUDE_INTERNAL_SYMBOLS anymore.
+
+       * gdk/gdk.symbols: Don't use #if defined().
+
+       * gdk/Makefile.am (gdkalias.h): 
+       * gtk/Makefile.am (gtkalias.h): Don't use cpp to filter gtk.symbols.
+
+       * gdk/makegdkalias.pl: 
+       * gtk/makegtkalias.pl: Move the #ifdef processing into the perl script, and
+       keep the #ifdefs which differentiate between platforms.
+
+       * gtk/Makefile.am (gtk_private_h_sources): Remove gtkinternals.h, it is no 
+       longer needed.
+
 2004-08-17  Matthias Clasen  <mclasen@redhat.com>
 
        * tests/testfilechooserbutton.c: Use g_message() throughout, add RTL envar for
index 009da4090649fc943825c4e5493a7ee79da23b9e..a306d32efa83869076b37cbc188142b1390b706f 100644 (file)
@@ -1,3 +1,19 @@
+2004-08-17  Matthias Clasen  <mclasen@redhat.com>
+
+       * gtk/abicheck.sh: No need for INCLUDE_INTERNAL_SYMBOLS anymore.
+
+       * gdk/gdk.symbols: Don't use #if defined().
+
+       * gdk/Makefile.am (gdkalias.h): 
+       * gtk/Makefile.am (gtkalias.h): Don't use cpp to filter gtk.symbols.
+
+       * gdk/makegdkalias.pl: 
+       * gtk/makegtkalias.pl: Move the #ifdef processing into the perl script, and
+       keep the #ifdefs which differentiate between platforms.
+
+       * gtk/Makefile.am (gtk_private_h_sources): Remove gtkinternals.h, it is no 
+       longer needed.
+
 2004-08-17  Matthias Clasen  <mclasen@redhat.com>
 
        * tests/testfilechooserbutton.c: Use g_message() throughout, add RTL envar for
index 8ce859d84d4afa6f04f3a723b2f45267f03d4ca3..9bf55a29d85b08a9023845dd0673b317c03a0ad0 100644 (file)
@@ -1,3 +1,16 @@
+2004-08-17  Matthias Clasen  <mclasen@redhat.com>
+
+       * abicheck.sh: No need for INCLUDE_INTERNAL_SYMBOLS any more.
+
+       * Makefile.am (gdk-pixbuf-alias.h): Don't use cpp to filter the symbols.
+
+       * makegdkpixbufalias.pl: Move the #ifdef processing into the perl script,
+       and keep the #ifdefs which differentiate between platforms.
+
+       * gdk-pixbuf.symbols: Remove the marshalers.
+
+       * Makefile.am (stamp-gdk-pixbuf-marshal.h): _-prefix the marshalers.
+
 Thu Aug 12 22:19:12 2004  Matthias Clasen  <maclas@gmx.de>
 
        * io-bmp.c (DecodeHeader): Properly determine the number of 
index 21dfe16f1aef42d986d93a95f86c7e9d3cbed003..9c370f6328e838365601ab42c815696d0c6266ed 100644 (file)
@@ -43,7 +43,7 @@ gdk_pixbuf.def: gdk-pixbuf.symbols
        (echo -e EXPORTS; cpp -P -DINCLUDE_VARIABLES gdk-pixbuf.symbols | sed -e '/^$$/d' -e 's/^/\t/') > gdk_pixbuf.def
 
 gdk-pixbuf-alias.h: gdk-pixbuf.symbols
-        cpp -P -DINCLUDE_INTERNAL_SYMBOLS gdk-pixbuf.symbols | ./makegdkpixbufalias.pl > gdk-pixbuf-alias.h
+        ./makegdkpixbufalias.pl < gdk-pixbuf.symbols > gdk-pixbuf-alias.h
 
 if OS_UNIX
 TESTS = abicheck.sh
@@ -392,7 +392,7 @@ gdk-pixbuf-marshal.h: @REBUILD@ stamp-gdk-pixbuf-marshal.h
        @true
 
 stamp-gdk-pixbuf-marshal.h: @REBUILD@ $(srcdir)/gdk-pixbuf-marshal.list
-       $(GLIB_GENMARSHAL) --prefix=gdk_pixbuf_marshal $(srcdir)/gdk-pixbuf-marshal.list --header >> xgen-gmh \
+       $(GLIB_GENMARSHAL) --prefix=_gdk_pixbuf_marshal $(srcdir)/gdk-pixbuf-marshal.list --header >> xgen-gmh \
        && (cmp -s xgen-gmh gdk-pixbuf-marshal.h || cp xgen-gmh gdk-pixbuf-marshal.h) \
        && rm -f xgen-gmh xgen-gmh~ \
        && echo timestamp > $(@F)
index 44df9df39d246be694fb18d75e9dcf8ea3a6e7c4..041677276f423bc125962b029d99699302556262 100755 (executable)
@@ -1,5 +1,5 @@
 #! /bin/sh
 
-cpp -P -DINCLUDE_INTERNAL_SYMBOLS gdk-pixbuf.symbols | sed -e '/^$/d' | sort > expected-abi
+cpp -P gdk-pixbuf.symbols | sed -e '/^$/d' | sort > expected-abi
 nm -D .libs/libgdk_pixbuf-2.0.so | grep " T " | cut -c12- | sort > actual-abi
 diff -u expected-abi actual-abi
index 100116355894367b91a401e6cf223ee82cbd0e64..f42c960239050ec2c2292c524bef659cb3c38a06 100644 (file)
@@ -57,10 +57,6 @@ gdk_pixbuf_loader_new_with_mime_type
 gdk_pixbuf_loader_new_with_type
 gdk_pixbuf_loader_set_size
 gdk_pixbuf_loader_write
-#ifdef INCLUDE_INTERNAL_SYMBOLS
-gdk_pixbuf_marshal_VOID__INT_INT
-gdk_pixbuf_marshal_VOID__INT_INT_INT_INT
-#endif
 gdk_pixbuf_new
 gdk_pixbuf_new_from_data
 gdk_pixbuf_new_from_file
index a76b61599ed439e93299634be5a71fab53bc10de..9bca4f7b4b9632c3ab5290c4e0bc849445a9b172 100755 (executable)
@@ -19,7 +19,6 @@ print <<EOF;
 
 #include "gdk-pixbuf.h"
 #include "gdk-pixdata.h"
-#include "gdk-pixbuf-marshal.h"
 
 EOF
 
@@ -28,14 +27,63 @@ while (<>) {
   # ignore empty lines
   next if /^\s*$/;
 
+  # skip comments
+  if ($_ =~ /^\s*\/\*/)
+  {
+      $in_comment = 1;
+  }
+  
+  if ($in_comment)
+  {
+      if ($_ =~  /\*\/\s$/)
+      {
+         $in_comment = 0;
+      }
+      
+      next;
+  }
+
+  # handle ifdefs
+  if ($_ =~ /^\#endif/)
+  {
+      if (!$in_skipped_section)
+      {
+         print $_;
+      }
+
+      $in_skipped_section = 0;
+
+      next;
+  }
+
+  if ($_ =~ /^\#ifdef\s+INCLUDE_VARIABLES/)
+  {
+      $in_skipped_section = 1;
+  }
+
+  if ($in_skipped_section)
+  {
+      next;
+  }
+
+  if ($_ =~ /^\#ifdef\s+G/)
+  {
+      print $_;
+      
+      next;
+  }
+
+
   my $str = $_;
   chomp($str);
   my $alias = $str."__internal_alias";
   
-  print "extern __typeof ($str) $alias __attribute((visibility(\"hidden\"))); \n";
-  print "extern __typeof ($str) $str __attribute((alias(\"$alias\"), visibility(\"default\"))); \n";
-  print "#define $str $alias \n";
-  print "\n";
+  print <<EOF
+extern __typeof ($str) $alias __attribute((visibility("hidden")));
+extern __typeof ($str) $str __attribute((alias("$alias"), visibility("default")));
+\#define $str $alias
+
+EOF
 }
 
 print <<EOF;
index fbf7ba481a6f3b2ab2b84556596bd05f4e6c829e..c8578a00fc54c742b8d5af0c8f29a50e1c18518e 100644 (file)
@@ -180,7 +180,7 @@ gdk.def: gdk.symbols
        (echo -e EXPORTS; cpp -P -DGDK_WINDOWING_WIN32 -DINCLUDE_VARIABLES gdk.symbols | sed -e '/^$$/d' -e 's/^/\t/') > gdk.def
 
 gdkalias.h: gdk.symbols
-        cpp -P gdk.symbols -DGDK_WINDOWING_X11 | ./makegdkalias.pl > gdkalias.h
+        ./makegdkalias.pl <gdk.symbols > gdkalias.h
 
 if OS_UNIX
 TESTS = abicheck.sh
index 16103aed83dd2ac90aea4c9eec4e69ccb5b38f0a..8669507c3ef35807db5f2494fc147b02f336cd39 100644 (file)
@@ -285,7 +285,10 @@ gdk_line_style_get_type
 gdk_list_visuals
 gdk_mbstowcs
 gdk_modifier_type_get_type
-#if defined(GDK_WINDOWING_X11) || defined(GDK_WINDOWING_WIN32)
+#ifdef GDK_WINDOWING_X11
+gdk_net_wm_supports
+#endif
+#ifdef GDK_WINDOWING_WIN32
 gdk_net_wm_supports
 #endif
 gdk_notify_startup_complete
index f46d21cc165d7f59bc22e709df0493ede3fb100b..4a1185f46871a4711e45897b0a3ec07d12ed30fa 100755 (executable)
@@ -29,23 +29,79 @@ print <<EOF;
 
 #include "gdk.h"
 
+#ifdef GDK_WINDOWING_X11
 #include "x11/gdkx.h"
+#endif
+#ifdef GDK_WINDOWING_WIN32
+#include "win32/gdkwin32.h"
+#endif
 
 EOF
 
+my $in_comment = 0;
+my $in_skipped_section = 0;
+
 while (<>) {
 
   # ignore empty lines
   next if /^\s*$/;
 
+  # skip comments
+  if ($_ =~ /^\s*\/\*/)
+  {
+      $in_comment = 1;
+  }
+  
+  if ($in_comment)
+  {
+      if ($_ =~  /\*\/\s$/)
+      {
+         $in_comment = 0;
+      }
+      
+      next;
+  }
+
+  # handle ifdefs
+  if ($_ =~ /^\#endif/)
+  {
+      if (!$in_skipped_section)
+      {
+         print $_;
+      }
+
+      $in_skipped_section = 0;
+
+      next;
+  }
+
+  if ($_ =~ /^\#ifdef\s+INCLUDE_VARIABLES/)
+  {
+      $in_skipped_section = 1;
+  }
+
+  if ($in_skipped_section)
+  {
+      next;
+  }
+
+  if ($_ =~ /^\#ifdef\s+G/)
+  {
+      print $_;
+      
+      next;
+  }
+
   my $str = $_;
   chomp($str);
   my $alias = $str."__internal_alias";
-  
-  print "extern __typeof ($str) $alias __attribute((visibility(\"hidden\"))); \n";
-  print "extern __typeof ($str) $str __attribute((alias(\"$alias\"), visibility(\"default\"))); \n";
-  print "#define $str $alias \n";
-  print "\n";
+  print <<EOF
+extern __typeof ($str) $alias __attribute((visibility("hidden")));
+extern __typeof ($str) $str __attribute((alias("$alias"), visibility("default")));
+\#define $str $alias
+
+EOF
 }
 
 print <<EOF;
index 2feac995f117fac8a8d1a33c26a041232942cb88..eb2659bba7d363c463f90b0248d1824b6820ae84 100644 (file)
@@ -74,7 +74,7 @@ gtk.def: gtk.symbols
        (echo -e EXPORTS; cpp -P -DINCLUDE_VARIABLES -DG_OS_WIN32 gtk.symbols | sed -e '/^$$/d' -e 's/^/\t/') > gtk.def
 
 gtkalias.h: gtk.symbols
-        cpp -P -DG_OS_UNIX -DGTK_WINDOWING_X11 -DINCLUDE_INTERNAL_SYMBOLS gtk.symbols | ./makegtkalias.pl > gtkalias.h
+         ./makegtkalias.pl < gtk.symbols > gtkalias.h
 
 if OS_UNIX
 TESTS = abicheck.sh
@@ -295,7 +295,6 @@ gtk_private_h_sources =             \
        gtkfilechooserutils.h   \
        gtkfilesystemunix.h     \
        gtkfilesystemmodel.h    \
-       gtkinternals.h          \
        gtkpathbar.h            \
        gtkrbtree.h             \
        gtksequence.h           \
index 45198b627ee5d50d880a976a332b9a93fd754c8c..26b81edd9141444604a92ede0a1a5b9de040ce7c 100755 (executable)
@@ -1,5 +1,5 @@
 #! /bin/sh
 
-cpp -P -DG_OS_UNIX -DGTK_WINDOWING_X11 -DINCLUDE_INTERNAL_SYMBOLS gtk.symbols | sed -e '/^$/d' | sort > expected-abi
+cpp -P -DG_OS_UNIX -DGTK_WINDOWING_X11 gtk.symbols | sed -e '/^$/d' | sort > expected-abi
 nm -D .libs/libgtk-x11-2.0.so | grep " T " | cut -c12- | sort > actual-abi
 diff -u expected-abi actual-abi
diff --git a/gtk/gtkinternals.h b/gtk/gtkinternals.h
deleted file mode 100644 (file)
index b3e1c2a..0000000
+++ /dev/null
@@ -1,38 +0,0 @@
-/* GTK - The GIMP Toolkit
- * Copyright (C) 1995-1997 Peter Mattis, Spencer Kimball and Josh MacDonald
- *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; either
- * version 2 of the License, or (at your option) any later version.
- *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.         See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public
- * License along with this library; if not, write to the
- * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
- * Boston, MA 02111-1307, USA.
- */
-
-/*
- * Modified by the GTK+ Team and others 1997-2000.  See the AUTHORS
- * file for a list of people on the GTK+ Team.  See the ChangeLog
- * files for a list of changes.  These files are distributed with
- * GTK+ at ftp://ftp.gtk.org/pub/gtk/. 
- */
-
-#ifndef __GTK_INTERNALS_H__
-#define __GTK_INTERNALS_H__
-
-/* Uninstalled header defining types and functions internal to GDK */
-
-#include <glib-object.h>
-
-G_BEGIN_DECLS
-
-G_END_DECLS
-
-#endif /* __GTK_INTERNALS_H__ */
index 170832fc406c749a9e752e0e5d422b31c1c1d9d8..9fb298b578154a490be223ecb10cb4413cb550cf 100755 (executable)
@@ -30,9 +30,13 @@ print <<EOF;
 #include "gtk.h"
 
 #include "gtkfilesystem.h"
+#ifdef G_OS_UNIX
 #include "gtkfilesystemunix.h"
+#endif
+#ifdef G_OS_WIN32
+#include "gtkfilesystemwin32.h"
+#endif
 #include "gtkhsv.h"
-#include "gtkinternals.h"
 #include "gtkpathbar.h"
 #include "gtktextdisplay.h"
 #include "gtktextlayout.h"
@@ -43,19 +47,71 @@ print <<EOF;
 
 EOF
 
+my $in_comment = 0;
+my $in_skipped_section = 0;
+
 while (<>) {
 
   # ignore empty lines
   next if /^\s*$/;
 
+  # skip comments
+  if ($_ =~ /^\s*\/\*/)
+  {
+      $in_comment = 1;
+  }
+  
+  if ($in_comment)
+  {
+      if ($_ =~  /\*\/\s$/)
+      {
+         $in_comment = 0;
+      }
+      
+      next;
+  }
+
+  # handle ifdefs
+  if ($_ =~ /^\#endif/)
+  {
+      if (!$in_skipped_section)
+      {
+         print $_;
+      }
+
+      $in_skipped_section = 0;
+
+      next;
+  }
+
+  if ($_ =~ /^\#ifdef\s+INCLUDE_VARIABLES/)
+  {
+      $in_skipped_section = 1;
+  }
+
+  if ($in_skipped_section)
+  {
+      next;
+  }
+
+  if ($_ =~ /^\#ifdef\s+G/)
+  {
+      print $_;
+      
+      next;
+  }
+
   my $str = $_;
   chomp($str);
   my $alias = $str."__internal_alias";
   
-  print "extern __typeof ($str) $alias __attribute((visibility(\"hidden\"))); \n";
-  print "extern __typeof ($str) $str __attribute((alias(\"$alias\"), visibility(\"default\"))); \n";
-  print "#define $str $alias \n";
-  print "\n";
+  print <<EOF
+extern __typeof ($str) $alias __attribute((visibility("hidden")));
+extern __typeof ($str) $str __attribute((alias("$alias"), visibility("default")));
+\#define $str $alias
+
+EOF
 }
 
 print <<EOF;