Commits
- Commit:
34b6a1d0c02432efe96cf9dc19916375f2d4f4dd
- From:
- Omar Polo <op@omarpolo.com>
- Date:
rename gen-emoji-matcher.sh -> genemoji.sh
- Commit:
c8c572e436db2059fd96ca1b0d10cdfe95c4090e
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fix compilation; add missing header
- Commit:
e95a72488ce30c8d26ffe8c5f6fc0a902cd879ac
- From:
- Omar Polo <op@omarpolo.com>
- Date:
document reply-last-input and M-r
- Commit:
acf9defe6aa430581cda55be848d06c76da738ad
- From:
- Omar Polo <op@omarpolo.com>
- Date:
make `reply-last-input' work on gopher too
- Commit:
3ecf170abf3a0062ff745ebe29c1e18e2ef361f3
- From:
- Omar Polo <op@omarpolo.com>
- Date:
reply-last-input was an idea by thfr
- Commit:
9d85aaf0829fa51614a5b36d4bd891283cce78fe
- From:
- Omar Polo <op@omarpolo.com>
- Date:
sync changelog
- Commit:
ed504b9e1a0307a13e869cb3296f07ec072dae42
- From:
- Omar Polo <op@omarpolo.com>
- Date:
reply-last-input: reply input requests on demand
Telescope now remebers the last URL (per tab!) that replied with
1X (input request.) The new command reply-last-input allows to
resend a query to that URL.
It's particularly useful with capsules that make a heavy uses of
10 replies (search engines or similar, interactive pages, etc)
because it both saves a network roundtrip and the user from looking
for the "search" link all across the page ;-)
idea from a conversation with thfr@, thanks!
- Commit:
693188cbff678ee0934f5fe5da91365614864888
- From:
- Omar Polo <op@omarpolo.com>
- Date:
ui_require_input: take a fn pointer instead of the protocol number
- Commit:
98b6dee90d3fd7300cc834972decd2c123af999c
- From:
- Omar Polo <op@omarpolo.com>
- Date:
only memcpy into imsg->data if datalen is not zero
Passing a NULL pointer to memcpy is UB no matter if len is 0 (commit by
claudio@, backporting for telescope.)
- Commit:
6d5348f32514004ad33d27ece8dec68ffed9b209
- From:
- Omar Polo <op@omarpolo.com>
- Date:
set_scroll_position: further semplifications
Simplify it even further. There's no need to keep the curfound variable
*at all*: if we end the loop, current_line is not found.
- Commit:
2ef8fcaedd93b1a56d0940ddd58a43a6d6c321f0
- From:
- Omar Polo <op@omarpolo.com>
- Date:
set_scroll_position: always set current_line
There's a small chance we may end up finding the topline but not the
current line if a page content is abruptedly changed.
While here also drop `curfound = 1': it's a dead assignment since it
returns in that conditions.
Bug found while reading the output of clang' scan-build, but not thanks
to clang. scan-build complained about the `curfound' variable, and by
re-reading the code I spotted the missing case.
- Commit:
0e49c9bbcbfef05d5bdf9ad6d851f51fe25c2eaa
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gemini_parse_reply: return code and don't close connection
clang static analyzer found a possible use after free: if the code is
not 2X in gemini_parse_reply we call close_conn. Then, in net_read we
set req->done_header, but req may have been free'd!
Actually, this is almost impossible to trigger. close_conn never ends
up calling free(req) on its first try, because tls_close takes a while
and is rescheduled by libevent. (The check req->ctx != NULL in
close_conn is always true if it's a gemini request.)
Nevertheless, it's clear to move close_conn out of gemini_parse_reply
and simply return the response code: it feels wrong that a "parsing
function" takes logic decisions.
- Commit:
3591e6dbd5cf94dae2b0a438542bf542ae094c01
- From:
- Omar Polo <op@omarpolo.com>
- Date:
declare data as uint8_t *; pointer arith on void * is a GNU extension
- Commit:
6ecb1353d2c2d97f08d74510f3067c7fd25a28db
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add some more warnings to the mix
- Commit:
6241aa26f34274ae8b9b7934c88d5686bcb2c7a2
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fix tests for build out-of-tree
now `make distcheck' works as advertised