Date: Fri, 10 Feb 2017 02:02:54 +0000 (UTC) From: Jung-uk Kim <jkim@FreeBSD.org> To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r433781 - in head/editors/libreoffice: . files Message-ID: <201702100202.v1A22sV5088600@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: jkim Date: Fri Feb 10 02:02:54 2017 New Revision: 433781 URL: https://svnweb.freebsd.org/changeset/ports/433781 Log: Refine r433663. If "use-header-bar" property of the dialog is set to TRUE, it uses a GtkHeaderBar for action buttons instead of the action area and gtk_dialog_get_header_bar() should not return NULL. If it is set to FALSE, gtk_dialog_get_header_bar() always returns NULL. In other words, we should not remove buttons from both header bar and action area in any case. This patch also removes pointless assertions from the previous patch. PR: 203563 Modified: head/editors/libreoffice/Makefile head/editors/libreoffice/files/patch-vcl_unx_gtk_fpicker_SalGtkFilePicker.cxx Modified: head/editors/libreoffice/Makefile ============================================================================== --- head/editors/libreoffice/Makefile Fri Feb 10 00:23:05 2017 (r433780) +++ head/editors/libreoffice/Makefile Fri Feb 10 02:02:54 2017 (r433781) @@ -2,7 +2,7 @@ .include "${.CURDIR}/Makefile.common" -PORTREVISION= 4 +PORTREVISION= 5 MASTER_SITES= http://download.documentfoundation.org/libreoffice/src/${PORTVERSION}/ \ http://dev-www.libreoffice.org/src/:src \ http://dev-www.libreoffice.org/extern/:ext Modified: head/editors/libreoffice/files/patch-vcl_unx_gtk_fpicker_SalGtkFilePicker.cxx ============================================================================== --- head/editors/libreoffice/files/patch-vcl_unx_gtk_fpicker_SalGtkFilePicker.cxx Fri Feb 10 00:23:05 2017 (r433780) +++ head/editors/libreoffice/files/patch-vcl_unx_gtk_fpicker_SalGtkFilePicker.cxx Fri Feb 10 02:02:54 2017 (r433781) @@ -1,26 +1,16 @@ --- vcl/unx/gtk/fpicker/SalGtkFilePicker.cxx.orig 2017-01-12 00:54:33 UTC +++ vcl/unx/gtk/fpicker/SalGtkFilePicker.cxx -@@ -418,8 +418,11 @@ shrinkFilterName( const OUString &rFilte - static void - dialog_remove_buttons(GtkWidget *pActionArea) - { -- GList *pChildren = -- gtk_container_get_children( GTK_CONTAINER( pActionArea ) ); -+ GtkContainer *pContainer = GTK_CONTAINER( pActionArea ); -+ -+ g_return_if_fail( pContainer != nullptr ); -+ -+ GList *pChildren = gtk_container_get_children( pContainer ); - - for( GList *p = pChildren; p; p = p->next ) - { -@@ -436,10 +439,10 @@ dialog_remove_buttons( GtkDialog *pDialo +@@ -436,10 +436,13 @@ dialog_remove_buttons( GtkDialog *pDialo { g_return_if_fail( GTK_IS_DIALOG( pDialog ) ); -#if GTK_CHECK_VERSION(3,0,0) #if GTK_CHECK_VERSION(3,12,0) - dialog_remove_buttons(gtk_dialog_get_header_bar(pDialog)); +- dialog_remove_buttons(gtk_dialog_get_header_bar(pDialog)); ++ GtkWidget *pHeaderBar = gtk_dialog_get_header_bar( pDialog ); ++ if( pHeaderBar != nullptr ) ++ dialog_remove_buttons( pHeaderBar ); ++ else #endif +#if GTK_CHECK_VERSION(2,14,0) dialog_remove_buttons(gtk_dialog_get_action_area(pDialog));
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201702100202.v1A22sV5088600>