John Darrington [Sat, 19 Sep 2020 06:54:40 +0000 (08:54 +0200)]
Select Cases Dialog: Make the 'All Cases' button easier to select.
The radiobutton area is very small and hard to click.
This change adds an Eventbox under the corresponding
label, So clicking on the label will have the same effect.
Ben Pfaff [Sat, 12 Sep 2020 16:53:52 +0000 (09:53 -0700)]
i18n: Fix use of const qualifiers in create_iconv().
Only the inbuf parameter of iconv() takes a const qualifier, on some POSIX
nonconforming systems, but all of its parameters were qualified here, which
caused warnings on those nonconforming systems. This fixes the problem.
John Darrington [Sat, 12 Sep 2020 12:35:18 +0000 (14:35 +0200)]
Regression Dialog: Reduce the size of the Independent Variables widget.
It is uncommon to perform a linear regression on many predictor variables.
But it is quite common to have many depdendent variables. This change
reduces the size of the predictor variable view widget and increases the
size of the regressor variable view widget.
psppire-output-view: modified the way how copy and paste works
Copy (no Paste in output window) works now the usual gui way. You
can select an item in the output window like a table or a chart and
it will be highlighted. When you now do the copy operation via menu
or key, then the item is copied to the clipboard. With the change
the clipboard now provides the additional formats svg and various image
formats. You can now paste to inkscape, libreoffice suite and gimp.
Ben Pfaff [Mon, 7 Sep 2020 21:18:14 +0000 (14:18 -0700)]
HOST: Implement timeouts on Hurd.
The existing implementation didn't work and I do not know why.
Hurd isn't documented to have different behavior for signals on
execl(). The timeouts were not firing, however.
This implementation does work.
Thanks to Friedrich Beckmann for reporting the issue (via Debian
buildd).
Ben Pfaff [Mon, 7 Sep 2020 17:45:12 +0000 (10:45 -0700)]
po: Simplify wget command.
https://translationproject.org/html/maintainers.html shows all
kinds of recursive options, but that's for retrieving all of the
po files in one go. This makefile doesn't do that, so the
recursive options don't help and provoke warnings from wget.
Ben Pfaff [Sat, 5 Sep 2020 23:15:27 +0000 (16:15 -0700)]
Release version 1.4.1.
This commit moves the version number *backward*, because we decided
that it was necessary to release 1.4.1 with all of the commits that
were currently on master.
Ben Pfaff [Sat, 5 Sep 2020 22:08:33 +0000 (15:08 -0700)]
Makefiles: Use $(PHONY) instead of .PHONY.
Avoids a warning from Automake in Smake:
src/ui/gui/automake.mk:463: warning: .PHONY was already defined in condition TRUE, which includes condition HAVE_GUI ...
Makefile.am:91: 'src/automake.mk' included from here
src/automake.mk:55: 'src/ui/automake.mk' included from here
src/ui/automake.mk:21: 'src/ui/gui/automake.mk' included from here
po/automake.mk:128: ... '.PHONY' previously defined here
Makefile.am:87: 'po/automake.mk' included from here
appstream provides the metainfo.its file which is required for
xgettext to extract the translation strings of the
src/ui/gui/org.fsf.pspp.metainfo.xml.in file
The metainfo.its file is located in /usr/share/gettext/its on
debian and on opensuse. It is provided by the packages
John Darrington [Sat, 5 Sep 2020 03:08:12 +0000 (05:08 +0200)]
Remove the --enable-debug configure option.
In practice, it is faster and simpler to make local modifications
to the respective files which debugging rather than reconfiguring
with --enable-debug. Furthermore this is just another configure
option which confuses users and serves little purpose.
John Darrington [Sat, 5 Sep 2020 02:31:28 +0000 (04:31 +0200)]
po/automake.mk: Fixed typo in variable names.
This resulted in po/pspp.pot not getting updated when it should have been.
This is a very good demonstration of the reason why carefully desigined
languages should insist on a variable being declared before it is used.
Also make the file depend on Makefile, so that it gets rebuilt when the
PACKAGE VERSION changes.
Ben Pfaff [Wed, 2 Sep 2020 04:42:35 +0000 (21:42 -0700)]
make-file: Pass O_BINARY or O_TEXT to open() call in replace_file_start().
It's hard to imagine why this would make a difference on a real system,
because the file descriptor obtained from open() is passed to fdopen(),
which is documented to determine whether the stream is text or binary
based on the mode string. However, bug #40223 implies that it makes a
difference. I hope that this commit fixes the problem (if any!).
Ben Pfaff [Mon, 31 Aug 2020 16:03:24 +0000 (09:03 -0700)]
Makefile.am: Suppress GCC 10.x warning in gl/careadlinkat.c.
This suppresses an error in gl/careadlinkat.c that otherwise can't be
avoided in GCC 10.x:
../../gl/careadlinkat.c: In function ‘careadlinkat’:
cc1: error: function may return address of local variable
John Darrington [Sun, 30 Aug 2020 05:43:32 +0000 (07:43 +0200)]
dummy.c: Conform to exact prototypes
Apparently certain toolchains complain if the symbols in dummy.c do not
exactly match their declarations. This should not be a problem unless
the compiler has enabled a lot of warnings, and is treating those warnings
as errors, (which is inadvisable except under very special circumstances).
However, it appears that some people are doing that.
John Darrington [Sun, 30 Aug 2020 05:45:01 +0000 (07:45 +0200)]
Remove _base_init functions from custom widget definitions.
These served no purpose, and are in Gtk+ only for reasons of
legacy so far as I can make out. Also use G_TYPE_DEFINE macros
instead of explicit *_get_type function definitions.
xml-parser-generator - iterate over sorted dict - #58975
I sort the dict before iteration because the normal
iterator over a dict is not reproducible and will generate
different code in each run. This will work but the
build is not reproducible.
John Darrington [Sat, 29 Aug 2020 06:26:23 +0000 (08:26 +0200)]
src/ui/gui/psppire-conf.c: Use GError instead of strerror.
Apparently the use of strerror was causing problems on certain
platforms. In this case, since the function being checked provides
a GError argument it makes sense to use that instead.
Warning: adhere to const qualifier for initializations and assignments
This commit avoids the warnings:
warning: initialization discards ‘const’ qualifier from pointer target type
warning: assignment discards ‘const’ qualifier from pointer target type
Warnings: missing initializer for value_tables and function type cast (GObject)
I fixed the warnings:
warning: missing initializer for field ‘value_table’ of ‘GTypeInfo’
warning: cast between incompatible function types
by replacing the handwritten type info code with G_DEFINE_TYPE...
macros (preferred). When this was not possible due to usage of
base init functions I added casts and the initializer.