Phy. vs Str. equality strikes again

Real coordinates

Addresses https://github.com/moosotc/llpp/issues/34 also too ugly to live

Remove redundant entry

Fix visibility conditions

Types...

Uhm... shell is hard too

Fix vertical bar scrolling when maxfit is false

Smart(er) mouse wheel zooming

Maybe some day should be enabled as a config option.

Stop gap listview scrollbars solution

Fix horizontal scrollbar

Hide veritcal scroll bar when not needed

If the document fits the window vertically. This also matches
horizontal behavior.

Begin scrollbar overhaul

This is "wrong", but the old approach was very ad hoc and caused major
pains.

Random points:

a. Behavior with this patch lends itself to an en vogue way of fading
   out bars

b. Scrollbars are hard - https://bugzilla.mozilla.org/show_bug.cgi?id=1248118

c. Perhaps view and bar areas should be covered by different view
   ports. This would be the "right" thing to do, but will probably
   require substantial changes all over the place

Use centered zooming pervasively

Fix visible area calculation

Be greedy

Address #33

Takeaways:
a. scrollbars ought to be re-engineered
b. scrollbar doesn't end with an 'e'

Fix selection zooming

Fix placement of list view selection rectangle

Prevent scroll bar width double accounting

Do not pivotzoom when there are pending geometry changes

This is a sledgehammer approach but nothing better springs to mind at
the moment.

Fix vertical positioning with pivot zooming

Wrong calculations inside represent went unnoticed for far too long
due to the fact that by default window dimensions are the same
(900x900).

Fix rect zooming

And probably a lot more.

Zoom centering

Needs more work for (at least) mouse wheel I suppose.

Simplify

Use correct x coordinate when layouting

Sigh...

Use hypot instead of open coding it

Given that other C99 features crept in anyways.

Cosmetics

Morf gotoy into gotoxy

When old code needed to adjust horizontal position it just merrily set
the state.x to the needed value and called gotoy, that was wrong in
case of throttling - gotoy might fail, so that state.y wasn't updated
but state.x was uncodnitionally set and the result was in inconsistent
behavior - one could end up in a situation where some tiles were
rendered - immediately evicted from the cache and then scheduled to be
rendered again.

Suppress redisplay on invalidation

We can end up in a situation when throttling is active, redisplay is
scheduled (if post-processing fails) and then invalidate is called -
resulting in an empty layout and consequently flickering.

This is probably the third time when post-processing intricacies
lead to nasty visual issues.

Reproducer:

mkfifo /tmp/fifo
llpp some.pdf -remote /tmp/fifo &
echo 'keys ctrl-0' >/tmp/fifo
for i in $(seq 1 100); do
    echo 'keys ctrl-+' >/tmp/fifo;
    sleep 0.01;
done;

Cosmetics

Fix logcurrently (angle was forgotten)

Just lock, do not warn buyers

Drop timed lock inside post-processing

This introduced nasty usability regression - any failure to take the
lock during post-processing results in a "freeze" so to speak. Hence
indicate the failure due to magical -1 post-processing return code and
schedule re-display if seen from the OCaml side, meaning the display
and post-processing will be triggered on any future event (most likely
the notification from the rendering/loading side that will likely
release the lock and allow forward progress)

Things work fine with original offender (visualFAQ.pdf -
http://tug.ctan.org/info/visualFAQ/visualFAQ.pdf) as does scrolling
around (mainly horizontally) in sufficiently large and zoomed in
image, such as:
https://upload.wikimedia.org/wikipedia/commons/c/c4/Vasily_Surikov_-_%D0%91%D0%BE%D1%8F%D1%80%D1%8B%D0%BD%D1%8F_%D0%9C%D0%BE%D1%80%D0%BE%D0%B7%D0%BE%D0%B2%D0%B0_-_Google_Art_Project.jpg)

Robustify

Sync with upstream

Give ml_postprocess a chance to succeed

Admittedly this is fugly, but no clean solution immediately springs to
mind.

(Register continuation that will be called upon unlock and will notify
caml code to retry? Use cloned contexts? Something else?)

All other trylocks are just as ugly, but probably much less noticeable.

Fix locking

Add clang guard

Update ignore list

Tramp auxiliary files.

Sync with upstream

Also fix some issues reported by valgrind.

Revert a bunch of syncs

Things are rather sad. Segfaults, compilation problems, etc, on top of
that Tor is on vaccation. So let's take it slow and stick to a
slightly outdated mupdf, but at least it works (that's not to say that
those are mupdf problems)

Safety net

null pointer dereference with "Frank Frazetta - Fantasy Illustrated #2.pdf"

"Fix" trimctm

I don't have any idea what's going on here. Details:

mupdf 834243dbfc8ab3e9269b7a789d61e3fe0f8e71f7 - good
      e2a85460370a3110b89fb838bc924c73f87f5b3c - bad

Most things work with either of those, except for:
http://www.gly.uga.edu/railsback/11111misc/SizeofThings36.pdf

Sync with upstream

Sync with upstream

Refactor

rotate is initialized the same way in both branches, so initialize it
once.

Sync with upstream

The trimctm stuff and the whole poling deep into fitz internals is
pretty fragile and disgusting, maybe this should be revisited.

Raison d'être for some of those "optimizations" was speed,
DDI0406B_arm_architecture_reference_manual_errata_markup_8_0.pdf
and
ECMA-376, Second Edition, Part 1 - Fundamentals And Markup Language Reference.pdf

[maybe things aren't as atrocious now (speed wise) as they were when some
 of those "hacks" were introduced]

Consistency

Cosmetics

Abstract keyboard handling a bit more

Cosmetics

Kill stray space

Change splitatspace to return a pair

Simplify

Add remote `keys' command

Remove diagnostic pragmas around fitz/mupdf headers

Apparently the issues with warnings were fixed upstream.

Demacrofication

Replace MIN/MAX usage with calls to fz_min/max[i].

Sync with upstream

Fix typo

Right... Coud those sort of typos thinkos be avoided? No idea...

Sync with upstream

Cosmetics

We only need to build libs part of mupdf.

Refactor

Fix native builds

Thanks to Tuncer Ayaz for heads up. (Also this can use some refactoring).

Make hlint happy

Dedup

Cosmetics

Coincidentally fixes dyslexia.

Add fb2

Quoting

Strangely enough this works better.

Sync with upstream

More dedup

Avid duplication

Merge branch 'ghennequin-master'

Merge branch 'master' of https://github.com/ghennequin/llpp into ghennequin-master

Added more precise building instructions for mupdf in [BUILDING]

Trim more stray whitespace

Kill stray space

Add fz_var annotations where appropriate

Thanks to Tuncer Ayaz for heads up.

Worth metnioning that the warnings only trigger when source is
compiled with optimizations (those were set to 0 (i.e. -O0) over here
so were absent when the source was compiled locally)

Use proper matrix

Takes trimming into account among other things.

Remove pointless code

Fix hovering over final char in a region of interest

Thanks for heads up to Tor Andersson.

Add svg

Compress more

Compress

Merge branch 'tuncer-mupdf-release-native'

Add target build/llpp.murel.native

This expects a mupdf tree that's been built in release mode and is
needed because -march=native objects can use instructions that are not
supported by all CPUs.

Silence the compiler

w.r.t. lablGL

Cosmetics

Roll own ASCII lowercase functionality

4.03 deprecates lowercase functions, and lowercase_ascii is not
available on prior compilers, roll our own.

cf. https://sympa.inria.fr/sympa/arc/caml-list/2016-04/msg00005.html

Collapse extended parameters before exiting

Something has to be done with this mess.

Wikipedia mode

mupdf can handle bmps on its own now

Sync with upstream

Merge pull request #14 from nojb/fix-osx-x11-socket-name

OS X: recognize X11 unix socket path in $DISPLAY

Indentation

Sync with upstream

Shrug

Allow specifying output path name via command line

Use command line instead of clipboard to specify urls

Enable css hack for shake builds

Update