Make AddMouseRegion's index unsigned

This was previously already done for wmbiff/wmgeneral: the index
argument is used as an index into an array, and is always only tested to
be below the upper bound. Passing in a negative value would pass that
test, and would result in an out-of-bounds access. All dockapps appear
to get that right, but libdockapp could see this violated by a (future)
user.

Temperature.app: fix temperature data URL

Hello All,

I am new to Window Maker. I wanted to install the Temperature.app dockapp
and found out that the ftp address that retrieves temperature data is
invalid. Investigating further I found that noaa has changed the address. I
have created a patch and am attaching it to this email.

I did not create a pull request, It wanted me to create a fork off of the
dockapps repository.

Maybe someone might find the dockapp useful and might use it.

Regards
Riza

Fix building with clang 16

From Clang 16 (and GCC 13) -Wimplicit-function-declaration has been
turned on by default. This resutls in warnings such as

song_hash.c: In function ‘insert_song’:
song_hash.c:22:39: warning: implicit declaration of function ‘malloc’ [-Wimplicit-function-declaration]
   22 |         new_elt = (struct hash_elt *) malloc(sizeof(struct hash_elt));
      |                                       ^~~~~~

This patch should fix these warnings. The warnings can be reproceded by
both recent version of Clang and GCC.

First discovered on Gentoo Linux: https://bugs.gentoo.org/882321

Signed-off-by: Brahmajit Das <brahmajit.xyz@gmail.com>

wmusic: bump version to 2.1.1 and update change-log

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmusic: fix overflow when calculating remaing track time

Track position and length are 64-bit integer quantities scaled in
microseconds, so use appropriate types.

Link: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1038264
Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmusic: divide time once in `DrawTime`.

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmusic: use appropriate variable names for track positions

Currently track posiions are called "time".

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmusic: use appropriate variables and function names for track numbers

Currently track numbers are referred to as "position".

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmressel: fix refresh rate check and add interlace mode detection

Interlace video mode was not supported and the refresh rate check using
only dotclock was inaccurate, which was selecting multiple current modes
and freezing the app.

Fixed build error on missing libiconv

ld error.
Error discovered while building in Freebsd 13.1. Uncertain if change
will cause problems on other platforms.

Updated deps error info

information to user updated with working link

Makefile cleanup - wmSMPmon

Disregard previous patch - forgot to update comment in makefile.
Related to issue #43.
Next on the list.

Changed path for man-file installation

Fixes problem with washerDryer / make install - broken path for man-pages
Changed path to what seems to make more sense.
Perhaps another approach might be to programmaticly examine manpath.
At the same time, if multiple paths are available a new problem presents itself.

Fixed make install/uninstall

washerDryer dockapp
related to issue #43
Next on the list created by d-torrance

Update README - Download link out of date.

Changed to current

wmcalclockkbd: Move variable definitions from xutil.h -> xutil.c

Otherwise we get "multiple definition" linking errors when compiling
with "-fno-common", which has been the default since GCC 10.

wmarchup: Add X11 linker flags

Otherwise, we get an "undefined reference" error during linking.

wmarchup: Fix path to libdockapp header

buttonmaker: Link against libdockapp instead of vendored wmgeneral

cputnik: Drop "file_types" variable name for enum.

It was causing a "multiple definition" linking error.  This could be
fixed using "static", but then we get an "unused variable" warning.
The F_* macros it defines are used, however.

cputnik: Update "make install" to support prefix/DESTDIR variables

Also other variables as recommended by GNU make docs.

cputnik: Drop unused "led_colors" enum.

It was causing a "multiple definition" linking error.

pclock: Update "make install" to support prefix/DESTDIR variables

Also other variables as recommended by GNU make docs.

asmon: Update "make install" to support prefix/DESTDIR variables

Also other variables as recommended by GNU make docs

asbeats: Add "install" target to Makefile.

wmdots: Update "make install" to support prefix/DESTDIR variables

Also several other variables as recommended by GNU make docs.

wmusic: Drop "-1.0" from playerctl pkg-config check.

It was removed when playerctl 2 was released.

This commit is included in the dockapps.net tarball for wmusic 2.1.0,
but for some reason I missed it when I submitted the previous series
of wmusic patches to the mailing list.

wmusic: Bump to version 2.1.0

wmusic: Update copyright header

wmusic: Switch player with eject button

wmusic: Switch player when current player is disconnected.

wmusic: Clear all errors when finished with them.

wmusic: Fix major memory leak; don't always create new players

Calling g_main_context_iteration will update the existing player.

Closes: #42

wmusic: Use playerctl_player_set_volume instead of "volume" property

Old behavior was deprecated.

wmusic: Free memory used by length and position strings

wmusic: Use "player-status" property instead of "status".

"status" was deprecated in version 2.  Also simpler since this is an
enum and not a string.  This fixes a memory leak since we weren't
freeing the string.

wmusic: Don't cut off first character of player name.

libdockapp: Free string to fix memory leak in DAWarning and DAError

libdockapp: Free string to fix memory leak in DACreateIcon

wmhdplop: update change-log and bump version

wmhdplop: build: add quoting and maintainer e-mail address to `AC_INIT`

wmhdplop: build: fix obsolete usage of `AC_OUTPUT`

These days `AC_OUTPUT` takes no arguments, and one uses `AC_CONFIG_FILES` to
indicate the files to be generated from their .in precursors.

wmhdplop: build: remove `AC_HEADER_STDC`

It's obsolete.

wmhdplop: build: replace `AM_CONFIG_HEADER`

It is obsolete and has been superseded by `AC_CONFIG_HEADERS`.

wmhdplop: build: use pkg-config to configure imlib2

imlib2 has had pkg-config support for many years, and recent versions no longer
contain imlib2-config.

wmhdplop: add .gitignore

wmbattery: bump version to 2.56 and update change-log.

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbattery: add config.sub and config.guess to dist tar-ball

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbattery: remove install-sh

Let autoreconf create it.

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbattery: bump version and update change-log

wmbattery: update .gitignore

wmbattery: avoid truncating ACPI info

Hitherto, the following code has been used to read from ACPI files:

  end = read(fd, buf, sizeof(buf));
  buf[end - 1] = '\0';

This can cause problems, if we read the whole file and there is no new-line:

  $ cat /sys/module/acpi/parameters/acpica_version
  20190816$

In this case, the '\0' will overwrite the last digit, leading to spurious
error reports:

  ACPI subsystem 2019081 too is old, consider upgrading to 20011018.

Instead, read one fewer byte and write the '\0' at the next index:

  end = read(fd, buf, sizeof(buf) - 1);
  buf[end] = '\0';

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbattery: avoid truncating ACPI file paths

Instead of writing the static `SYSFS_PATH` string to a buffer, and
passing the buffer to `opendir`, remove the buffer and pass the static
string directly.

Instead of using a fixed size for the ACPI file path buffer, size it
based on the sizes of the arguments that snprintf will write to it.

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbattery: update configure.ac

Run configure.scan and update accordingly.

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbattery: quote autoconf macro arguments

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbattery: remove superfluous Makefile rules

The default '%.o : %c' rule suffices for simplehal.c and upower.c.

Signed-off-by: Jeremy Sowden <jeremy@azazel.net>

wmbattery: pass `--install` to autoreconf

wmix: Fix for inaccurate volume adjustments

This has been applied to the Debian package since version 3.1 in 2006.

Authored-By: Dan Pascu <dan@ag-projects.com>

wmbiff: Convert AUTHORS from ISO-8859-1 to UTF-8.

wmtv: Fix typo

wmcliphist: fix FTCBFS by making pkg-config configurable.

Authored-By: Helmut Grohne <helmut@subdivi.de>
Bug: https://bugs.debian.org/941312

wmcliphist: Fix -Wmaybe-uninitialized compiler warning.

In particular, the following warning:
 wmcliphist.c: In function 'main':
 wmcliphist.c:386:11: warning: 'icon_file' may be used uninitialized in this function
 [-Wmaybe-uninitialized]
    surface = cairo_image_surface_create_from_png(icon_file);
            ^

wmhdplop: Fix "misleadingly indented" compiler warnings.

wmhdplop: Fix typos.

wmifinfo: Use CC as default value for LD Makefile variable

wmifinfo fails to cross build from source, because its Makefile uses
two variables (CC and LD) to supply the compiler, but dh_auto_build
only passes the former.

Authored-By: Helmut Grohne <helmut@subdivi.de>
Bug: https://bugs.debian.org/929758

Note necessary changes to the configure line in a new readme.md in the repo root. Add .gitignore files (safely: The repo's root .gitignore doesn't ignore files that are present rather than generated as of current versions of any apps). Resolves #36.

Co-authored-by: OldCoder <1663134+OldCoder@users.noreply.github.com>

Fix multiple definitions errors - resolves #34

Co-authored-by: OldCoder <1663134+OldCoder@users.noreply.github.com>

Add autoreconf input files and .gitignore. resolves #33

Rename the old Makefile.

wmressel: Bump to version 0.10

wmressel: Check return values from XF86VidMode* for submenus

If these functions fail to get the video mode information, then later
code may cause bad things to happen, e.g., "munmap_chunk(): invalid
pointer "when freeing vm_modelines.

wmressel: Display error message if XF86VidModeGetModeLine fails

Otherwise, the program crashes with a floating point exception when we
try to divide by 0 in the call to BlitNum a few lines later.

wmressel: Use correct type for dotclock from XF86VidModeModeInfo

wmressel: Increase buffer size for string to avoid compiler warning.

wmressel: Fix multiscreen support

Patch by Malcolm Parsons <malcolm@ivywell.screaming.net>.  See
https://bugs.debian.org/158674.  Has been applied to the Debian
package since 2002.

wmressel: Switch manpage section from deprecated 1x to 1.

From Debian (https://sources.debian.org/src/wmressel/0.9-1/debian/patches/
manpage-section.patch/)

wmressel: Convert README from ISO-8859-1 to UTF-8.

wmlongrun: display frequencies >= 1 Ghz

This patch has been applied to the Debian wmlongrun package since
March 2005.  From https://bugs.debian.org/277066:

> From: guillaume.ansanay-alex@esm2.imt-mrs.fr
> To: submit@bugs.debian.org
> Subject: wmlongrun can't display frequencies >= 1 Ghz
> Date: Mon, 18 Oct 2004 12:50:29 +0200
>
> I am using a Crusoe TM5800 (1 GHz) CPU. When it is running at 1 Ghz,
> wmlongrun displays 00 Mhz.
>
> I wrote a patch applying to files from wmlongrun_0.3.0-pre1.orig.tar.gz
> (it modifies only files in the src/ directory) fixing this bug.
> This patch is sent with this e-mail.

wmtop: Fix typo in manpage.

wmsun: Bump to version 1.06

wmsun: Add 'dist' target to Makefile to generate tarball

wmsun: Move version number definition to Makefile

wmsun: Fix typo in manpage.

wmmixer: 1.9

wmmixer: dynamic exception specifications were removed in C++17.

wmmixer: add autogen.

wmxres: Bump to version 1.4.

wmxres: Add manpage from Debian

https://sources.debian.org/src/wmxres/1.3-1/debian/wmxres.1/

wmxres: Port to libdockapp instead of using its own version of wmgeneral

This also centers the dockapp in its icon window instead of being placed
in the upper left hand corner.

wmxres: Use autotools for build

This will make the following patches unnecessary:
* https://sources.debian.org/src/wmxres/1.3-1/debian/patches/no-root.patch/
* https://bugs.debian.org/968433

Also move source files from wmxres directory to root directory; otherwise
this directory will conflict with the binary that's built.

wmxres: Add "-v" command line option and bump to version 1.3

wmxres: Increase buffer size to avoid format-overlow warnings.

wmxres: Replace deprecated xf86dga.h with Xxf86dga.h

wmxres: diff for NMU version 1.2-10.2

Control: tags 957959 + patch
Control: tags 957959 + pending

Dear maintainer,

I've prepared an NMU for wmxres (versioned as 1.2-10.2) and
uploaded it to DELAYED/5. Please feel free to tell me if I
should cancel it.

Added Doug Torrance in Cc as he intends to adopt this package.

--
Regards
Sudip

Add wmxres 1.2 to the repository

Obtained from
http://deb.debian.org/debian/pool/main/w/wmxres/wmxres_1.2-10.1.tar.gz
with mercurial and debian files removed.

Note that the wmgeneral included with wmxres includes a RedrawWindowXYWH
function that isn't present in the libdockapp version of wmgeneral, so
we can't use it.

Description: Fix ftbfs with GCC-10

Bug-Debian: https://bugs.debian.org/957941
Forwarded: no

wmbiff: parse IMAP configuration on FreeBSD

The upper limit of a bounded repetition is given by RE_DUP_MAX, which is
255 on FreeBSD (and OpenBSD), but 32,767 on Linux, so this should
suffice:

wmbiff: parse IMAP configuration on FreeBSD

From https://github.com/window-maker/dockapps/issues/29, raised
by mpastor:

Hi,
wmbiff compiled from source is not working on FreeBSD when using IMAP4, POP3 is working:

$ wmbiff
error in compiling regular expression: invalid repetition count(s)
error in compiling regular expression: invalid repetition count(s)
wmbiff/ imap4: Couldn't parse line imaps:mymail@domain.tld:MyPaSsWd@imap.domain.tld plaintext (0)
  If this used to work, run wmbiff with the -relax option, and
  send mail to wmaker-dev@googlegroups.com with the hostname
  of your mail server.
wmbiff: creator for mailbox 0 returned failure
$ wmbiff -relax
error in compiling regular expression: invalid repetition count(s)
error in compiling regular expression: invalid repetition count(s)
wmbiff/ imap4: Couldn't parse line imaps:mymail@domain.tld:MyPaSsWd@imap.domain.tld plaintext (0)
  If this used to work, run wmbiff with the -relax option, and
  send mail to wmaker-dev@googlegroups.com with the hostname
  of your mail server.
wmbiff: creator for mailbox 0 returned failure
$ wmbiff -debug
wmbiff: config_file = /home/me/.wmbiffrc.
wmbiff: @interval.-1=300@
wmbiff: @label..0=mail@
wmbiff: @path..0=imaps:mymail@domain.tld:MyPaSsWd@imap.domain.tld plaintext@
wmbiff: @action..0=claws-mail@
error in compiling regular expression: invalid repetition count(s)
error in compiling regular expression: invalid repetition count(s)
wmbiff/ imap4: Couldn't parse line imaps:mymail@domain.tld:MyPaSsWd@imap.domain.tld plaintext (0)
  If this used to work, run wmbiff with the -relax option, and
  send mail to wmaker-dev@googlegroups.com with the hostname
  of your mail server.
wmbiff: creator for mailbox 0 returned failure
wmbiff: looking for /usr/local/share/wmbiff/skins/wmbiff-master-led.xpm
wmbiff: found /usr/local/share/wmbiff/skins/wmbiff-master-led.xpm
wmbiff: running 1 mailboxes w 64 h 20
So I have changed Imap4Client.c according to Pop3Client.c and now it's ok:

wmcore: bump to 0.0.4.

wmcore: remove reference to deleted code.