fix find-following

this bug manifested in freecell as the ace of clubs not recognized as
placable on the foundation (from the tableu) when all four free cells
were occupied.
due to us not checking that find_top() found a card at all, we
accidentally checked foundation[-1], which is the fourth free cell.

there are probably more bugs stemming from this misuse of find_top().

make 'no take from empty' more dry

oof, that's an awful macro.

prevent taking from empty foundation

a direct copy of the previous commit, but for the other non-tableu
piles.

freecell: fix direct addressing 'take from cell'

the 'no take from empty stock/cell' rule had a bug that only checked the
first cell when accessed with '9'. this is because with direct
addressing mode, active.opt stays at its initial value of 0. this check
was split in two parts:

- direct addressing: only abort when all cells are empty
- other: check that struct cursor active was actually modified.

a third test was then added to make this work with mouse addressing
also.

make freecell obey the 'no selecting of empty stock' rule

blink waste/cells

'.' implementation (ugly and not portable)

uses ungetc(3) twice, which "works on my machine"™

find and search commands prototypes

NOTE: both need a considerable amount of cleaning up

ifdef off inverted move logic

that proved to be more annoying than useful for me. pass -DINVERSE_MOVE
to the compiler to reenable.

fix freecell->free tableu

if only 1 free cell was occupied and moved to an empty tableu using
direct addressing the user was asked which cell to take from.

shuffle around TODOs

fix c2f() emtpy check

gah! function didn't check if the cell was empty. this caused extra
items on the undo list (and the wrong retval)

freecell: better join() cell->foundation fix

freecell: join() freecells->foundation

freecell: join() to foundation: fix fencepost error

update TODOs

smart card selector -> DONE

spider: fix undo stack-completed-pile bug

if the user draws from the stock and this completes a cascade, the
undo event for the stock deal _must_ be stored before the undo event for
the cascade removal. otherwise the undo history will be broken!!1

updated foundation placeholder

the braille characters are widely enough supported, and the placeholder
should still make sense even if they aren't on a system.

not commited: alternate "10" symbols. ⒑ (tofu in xterm), ⏨ (too small,
tofu in xterm), Ю, (cyrillic yu, not looking too good)

freecell: join to empty tableu

also removed redundant checks for join()-ability in get_cmd()

use color_ok macro everywhere

freecell: smart card selector, part 4

all done! \o\ /o/ \o/
nice test seed: 1550690183
also a new helper functoin color_ok(), yet to be used everywhere else

freecell: smart card selector, part 3

foundation -> tableu is the same as in klondike, so i reordered the code
to take advantage of this

note that the if inactive.opt check inside the klondike||freecell block
(sol.c:1169) is only correct for klondike, but freecell checks further
up the if/elseif branches for inactive.opt, so this is never executed.

freecell: smart card selector, part 2

also fixed wrong return value in spider's card selector dialog

freecell: smart card selector, part 1

implements smart move detector for foundation->freecell and
freecell->foundation. missing: *->tableu

implement join() for freecell

also fix the outstanding todo in klondike.join()

remove freecell backspace

freecell stable enough for default target

differentiate foundation from placeholder

alternative designs:
╭╌╌╌╮╭╌╌╌╮╭╌╌╌╮
╎ ╳ ╎╎▕▔▏╎╎▛▀▜╎
╎╱‾╲╎╎▕▔▏╎╎▛▀▜╎
╰╌╌╌╯╰╌╌╌╯╰╌╌╌╯

argh! a semicolon too many!

freecell: respect max_move() in cursor selection

swap win_anim width/height

fix fencepost in undo_pop, mouse-foundation also in klondike

fix fallthough at -V

thanks dax-dev

freecell: mouse select for cells/foundation

freecell: finished t2t, work on card selector dialog

playable, but clunky version of the get_cmd() post processing stage.

more work on freecell t2t()

it works now, although direct addressing still needs to be addressed
(ba-dum-tshhh) in get_cmd() and some new TODOs have to be DONE.

freecell: t2t(), calculate number of movable

see also: https://boardgames.stackexchange.com/a/45157/26498

win_anim: print everywhere on the terminal

merge freecell todos into readme

spider: fix move to empty tableu

"no of the additions affect klondike or spider, so its safe to commit"
my ass! this is the second bug I've found that I introduced with this.

remove debugging seed

that shouln't've been committed.

Add a colorful version of the mini colorscheme

WIP: freecell implementation (unfinished)

what's missing:
 - t2t() and corresponding undo_pop()
 - get_cmd() "select from which {cell,foundation}?" dialog
 - some mouse details (term2pile, wait_mouse_up, set_mouse)
 - join() for freecell

no of the additions affect klondike or spider, so its safe to commit.

join() fixes for w2t

you could join onto the waste and if the waste was -1.

remove stupid {1, ...} initialisation

new todo: freecell

allow disabling visual bell (might change)

thanks to github user mikezila for the idea. will be replaced by a
less-annoying visbell in the future

remove spider.c symlink

let's do the straight-forward thing and just spell out sol.c.

fix 80 columns, todo->done

finish active-cursor-hider

yeah, it's a hack.

WIP: hide active cursor when using mouse (/u/TooEarlyForMe)

`:help` to show keyhelp (/u/Valley6660)

allow using `10` instead of `X` on card face (/u/Valley6660)

hide old internal notes

add keybindings info

'join' with right mouse, update keybindings help

fix segfault when mouse-moving multiple cards to empty tableu

the "up to?" dialog calls getch(NULL), which caused a segfault.
> Program received signal SIGSEGV, Segmentation fault.
> 0x00000000004030fb in getch (buf=0x0) at spider.c:940
> 940             if (buf[3]) {
> (gdb) bt
> #0  0x00000000004030fb in getch (buf=0x0) at spider.c:940
> #1  0x0000000000402c97 in get_cmd (from=0x7ffeba433bfc, to=0x7ffeba433bf8,
>     opt=0x7ffeba433bf4) at spider.c:808
> #2  0x0000000000400ebf in sol () at spider.c:108
> #3  0x0000000000400e2f in main (argc=2, argv=0x7ffeba433d28) at spider.c:88

Add LICENSE

this is the first version covered by and released under the terms of the
GNU GPL version 3. Older versions are _not_ relicensed.

improve readme

friendly readme opening

elevator pitch of the controls

mouse: allow dragging

one can now drag the mouse cursor from one pile to another and it'll
register as two events and do the right thing.

handle SIGTSTP, fix mouse ctrlseq

cleanup mouse

still needs some features like dragging and 'if to is empty, how many
cards?'

mouse mode (needs cleanup)

fix overlong lines

fix spider foundation printing

clean up reworked join

rewrite join() prediction engine

fix 'make test'

symlink duplicated all TODOs in sol.c

join: in SPIDER, prefer completing a stack

better join for klondike

join to foundation, better to-tableu join

allow joining to foundation

allow inverted moves

use getch() everywhere

except for inside getch(), of course. where no mouse is needed, one can
pass NULL instead of a buffer; KEY_INVAL will then be returned if a
mouse click was received.

support 'real' cursor keys

keyhandling imported from minesviiper (minor changes); mouse support
stubbed.

make Makefile portable

apparently, the "prerequisite name != target rule name" only works in
GNU make. symlinking sol.c to spider.c and changing the makefile works.

implement restart same game, increase portability, update TODOs

 * use rand() instead of random() to get in line with mines and viiper
 * reorder playfield struct for better alignment
 * remove ignore warnings pragma (not needed any more)
 * remove _POSIX_C_SOURCE feture test (not needed and breaks freebsd)

fix off-by-one

cursor could 'escape' the playfield in KLONDIKE

actually useful CMD_JOIN implementation

first version of join() -- to be replaced

this version is very suboptimal. we shouldn't take from longest piles,
as that will rip apart already constructed piles. instead, take from
piles where the card we want to move has the least amount of movable
cards _above_ it. even better if the card above is closed. even better
yet if there is no card above (resulting in an empty pile)

automagic conserve space

only when the window size would be too small to display a pile
completely. still has the problem that sometimes the terminal is too
small for even the extreme overlap.

implement 'M' keybinding

moves active cursor to the centre pile

rework command line arguments

don't require long string arguments

fix BULCARD() padding

using ESC produces tofu on some terminal emulators. switching to SI
(Switch To Standard Charset) for padding seems to work better. (SI
should have no effect on Unicode terminal emulators). this also made
detecting the padded lines easier, as \017 (SI) is only appearing in
those.

fix foundation rendering (SPIDER)

by using a static int to store the foundation index, it couldn't be
reset when a new game was started. this caused the first pile to be e.g.
to end up on the second foundation if the game before had 1 completed.

fix bold hack; extreme overlap: always show top selected card

fix broken halfcard rendering

due to the faint hack, black cards were 1 byte shorter than red cards.
this meant, that the offset for halfcard was off for red colors (as it
has been adjusted for black cards), which caused printing errors in
spider's foundation for overlapping red cards. can't pad with zero bytes
(as that would end the string), but using an extra escape seems to be OK
for xterm and vte-based terminals.

update TODOs

assimilate remove_if_complete

so spider's "double-click" action has the same feedback as sol's.

awful hack that un-breaks faint+bold printing

bold and faint can bet set individually, but only reset both at once.
this causes the bold attribute (useful for determining with cards are
movable) to be reset when faint was. we are splitting the string at a
known-whitespace position only if we are at the faint printed lines
(str[n]==033).

fix turn over bug

undo for spider

note that there is a bug in turn-over-detection when to==FOUNDATATION[0]

properly free undo allocations

implement undo for KLONDIKE

for now, redo has been scrapped (would require an additional field in
f., and I don't see the need)

implement 'up to' space/return

remove redundant display calls

undo_pop pseudo implementation

move data structure definition into .h

can't declare functions otherwise :|
forward declarations are a mess!

rename append_undo to undo_push; subs for undo_pop, free_undo

squash another bug

what did that semicolon here!?
note that WASTE is not -1 but 0, as it is used as the second index into
action[][]