always redraw images

Previously when covering the sent window with an image slide selected and
exposing it again, the image was lost. This change always redraws the image and
also removes the now redundant bit flag in the state enum.

improve some more error messages

- remove \n in die() messages in drw.c
- add a few more ':' suffix for functions that set errno.
- minor typo.

Add a manpage

Use Plan 9 style for ARGEND

Fix error-messages

Fix memory leak and print a warning on invalid regex

This will minimize the already low memory usage of sent.

_exit() instead of exit() after failed exec

die() calls exit(), but actually at this point the program could be in a
state not allowing this. It's better to call _exit to stop the libc from
calling any handlers that might fail at this moment anyway.

Simplify LICENSE wording

Now, before you call your lawyers, here is some explanation:
The MIT/X Consortium-License and ISC-License are equivalent
according to the Berne-convention. This means that a court
of law presented with either licenses will act the same.

The advantage of the ISC license is that it has a simpler wording
and is recommended by the OpenBSD-project for new projects[0].
And as we at suckless.org love removing cruft, this also includes
removing legal cruft wording that is not needed any more.

[0]: http://www.openbsd.org/policy.html

Simplify usage()

Update example presentation

- Add a small transparent farbfeld test image

add ff and ff.bz2 support in config.def.h

Also call ffload on the current slide in advance()

This fixes a bug where the image would not be loaded if the user setup a
keybinding which advances multiple slides at once. Then the preloading for the
previous/next slide in advance() would not catch that case and instead of an
image the text version (`@filename`) would be rendered on the slide.

clean ffread and rename to ffload

- also remove unneeded memmove in load()

load slide image on-demand

On Sat, Jun 04, 2016 at 05:28:31PM +0200, Markus Teich wrote:
> Hiltjo Posthuma wrote:
> > previously an image file would be opened but only ffread when advancing to
> > the slide, but when the slide was not used it gave an error:
> >
> >  /usr/local/bin/2ff: failed to convert image/png
>
> Heyho Hiltjo,
>
> thanks for the patch. Unfortunately it does not work if the first slide contains
> an image (ffopen is only called in advance). I think it would be good to merge
> ffopen and ffread instead into a single function ffload. This makes the `LOADED`
> state clearer and also enforces that the fd is closed in the same function where
> it is opened. This ffload function should then be called in advance() replacing
> the ffread() calls if the image is not loaded yet and once in load() for the
> first slide if it is an image.
>

Ah yes, sorry for the oversight.

> If you want to take this new approach, go for it, otherwise I'll look into it
> myself.
>

I have attached a patch that does this, I hope you'll like it.

Also I'm not sure if we need the below code in advance(), I have removed it in
the patch:

        if (slidecount > idx + 1 && slides[idx + 1].img)
                ffread(slides[idx + 1].img);
        if (0 < idx && slides[idx - 1].img)
                ffread(slides[idx - 1].img);

That seems to preload the next and previous slide image right? A minor issue I
notice also is that images seem to flicker, it uses XPutImage directly to
xw.win. Maybe it can be replaced with a backbuffer then XCopyArea? What do you
think?

In advance() it should also not always be needed to rescale the image.

--
Kind regards,
Hiltjo

From 97bebdcab4003f9acdfdd4bdf424449299ffd61d Mon Sep 17 00:00:00 2001
From: Hiltjo Posthuma <hiltjo@codemadness.org>
Date: Sat, 4 Jun 2016 21:34:25 +0200
Subject: [PATCH] merge ffread and ffopen into one function

load slide image on-demand

previously an image file would be opened but only ffread when advancing to
the slide, but when the slide was not used it gave an error:

/usr/local/bin/2ff: failed to convert image/png

this changes it to load the image on-demand once and "cache" it.

import new drw and util from libsl.

fix include and lib paths for the BSDs

support FreeBSD

On FreeBSD the lib and include directories are in /usr/local rather than /usr.

Import arg.h OOB fix from Lucas Gabriel Vuotto.

This is a small fix for arg.h where it's possible to perform an out-of-boundary
memory access in argv's internal string.

Fix cmdline argument check

There was a segfault when sent was called without arguments. Now we use stdin
when there's no argument or - is used. Thanks to izabera for the report.

no need to reimplement cat here

improve error handling

- errors are now fatal
- no need to clean up when dying
- no need for status return codes => ffread and ffprepare now return void

style: move function return types to own line

Use 2ff exclusively in config.def.h

Fix alpha blending

- "/ 257", because 255 * 257 = UINT16_MAX
- "/ 255", because that's the maximum possible RGB value

readme: update reference to farbfeld

Change an eprintf to a die to stop child from running its own slideshow

The child thread was created because execlp will not return if successful.
The eprintf was placed after the call to execlp to catch any error, but the
child continued to return a (closed) fds[0], resulting in a second slideshow
being run by the child.

This commit fixes the issue by killing the child.

filter via shell pipeline

Instead of requiring an executable, allow building arbitrary shell
pipelines to filter filetypes through.

unbreak the build

fix 2 uninitialized variables and bin == NULL check

check for bin == NULL

minor syntax, style and README fixes

support transparent images

Support farbfeld as an intermediate format

Sent now uses farbfeld[0] as an intermediate format.  A series of
filters is specified in config.h that matches file extensions to
filter programs.  The programs will convert between formats such as
png to farbfeld.  Internally in sent we do not need to worry on how
to parse png or any other format.

This also works with jpg and gif and others.  The 2ff wrapper will
use imagemagick conversion tools.  This is temporary as jpg2ff and
gif2ff will also be implemented.

To make this work, you will have to clone[0] and put png2ff and 2ff
in your PATH.

[0] http://git.2f30.org/farbfeld/

add 'sent: ' prefix to errors

Add 'n' and 'p' keys to navigate slides

Refactor die() to use eprintf()

Use consistent wording for error messages

Use right click as previous and enable scrolling to switch slides

Bail out before allocating slides if file is empty

In load() we allocated slides before checking if we actually read
anything from the FILE fp and then continue with an allocated but
“empty” space wich would lead to errors.

fix dying on empty slidefile

drw.c: Avoid potential memory leak in drw_cur_create()

If drw was NULL, memory was still allocated for cur.

Add Escape key to quit sent

die if no font could be loaded

Register space to advance to the next slide

shorten bulletpoints in example presentation

fix dejavu font name

support even larger screens

prevent flickering when changing slides

When the window background is left at WhitePixel (from initialization) and the
user has set a dark background color in config.h, the window shortly displays
the white default background when clearing after advancing to another slide.
This was very disturbing and is fixed now by setting the default window
background color to the bgcol chosen in config.h.

update README

fix another index overflow issue

fix several number overflow issues

update version

add multiline support

fix more memory leaks

free all fonts correctly

free temporary fontstr arrays

simplify png cleanup

make usable screensize config const

don't assume we get a ConfigureNotify event

If the window manager maps our window at the requested size, we may not get a
ConfigureNotify event.  This happens e.g. under ratpoison.

The result is that xw.uw and xw.uh are never set, so text is too small and
pngscale() tries to divide by 0.

initialize .img

without the attached patch ./sent example segfaults here

don't use EXIT_… cpp defines

fix comment wording in drw.c

make colors configurable

update license year

fix default fonts in config.def.h

remove unused font configuration, update license year

fix font width calculations

remove unused type qualifiers

update example presentation

fonts are automatically cleaned up

drw_resize should be called on mapping as well

remove debug statement

typedef image struct

minor style adaptions

fonts cleanup

remove unused dc/gc variables

remove utf8 support from todo list

remove old stuff and fix warnings

blank screen before drawing text

fix calculation of correct font size to use

drw: fixup font handling

use new font system from drw

minor style fix and new var for fonts

drw: fixup drw_rect function

use die() where it should be used

use drw color scheme fixup

drw: fixup color schemes

some work on using drw.c done

drw: fixup font_getexts

add utf8 support, scaling and positioning missing.

init and clear drw context

prepare switch to libsl

update TODO

update example

support for comments to ignore lines

use # as first character in a line to skip this line
from the slides.

make: fix warnings because of BSD_SOURCE

make: don't build cscope by default

update README

large jumps don't work well with presenters