Commits
- 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
- Commit:
23a912c214e3f9a8bbb2bf03e65c2d3a538f869f
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add test of parsing and deserialization of gophermaps
- Commit:
1bcb8303e1b913df31980ca332a5619424a72b01
- From:
- Omar Polo <op@omarpolo.com>
- Date:
move load_page_from_str to telescope.c
I'd like to use parser{,_gophermap,_gemtext}.c in standalone testing
programs, and this function is getting in the way since it depends on
ui.c
- Commit:
fe84b7508f733106d346acf1b120951c78ad77a1
- From:
- Omar Polo <op@omarpolo.com>
- Date:
switch the cache to the new parser_serialize API
- Commit:
069c9a92f36df9d307d734ffa8903035c7181129
- From:
- Omar Polo <op@omarpolo.com>
- Date:
parsers: add serialize function
Add a parser_serialize function and an optional function `serialize' for
parsers. The default implementation just prints the text line, which is
suitable for parser_textplain and parser_textpatch; gemtext and
gophermap uses their own specific unparse functions.
This is intended for an incoming cache refactoring: we'll cache the page
serialized in the correct format, not once it has been translated to
gemtext. This will allow in the future to provide commands such as
``save page to disk'' or interface with an on-disk cache.
- Commit:
5a21e0eb51554d72c1b6465c2b4afc8cdef5dcf1
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add fn pointer to the parser init function to struct parser
will come in handy in the future cache refactoring.
- Commit:
b96fa8fe23277bee0c42d55dae506449d83909bc
- From:
- Omar Polo <op@omarpolo.com>
- Date:
use parser_init instead of directly calling the parser init fn
- Commit:
bf5b33f40e876afe41fd486ff5d3eb182dfaf046
- From:
- Omar Polo <op@omarpolo.com>
- Date:
correctly save scrolling position
Before we didn't save the scrolling position before a refresh
(nohist=1) and that's annoying. This saves the scrolling position
only in the correct case, which is when changing from a page to
another, expect when switching to TAB_LAZY off.
- Commit:
fb8dcd1c494b0339067ca13f5bfc8ac8a1a43114
- From:
- Omar Polo <op@omarpolo.com>
- Date:
move TAB_LAZY flipping from switch_to_tab in load_url
load_url needs to know when a tab passes from lazy to not lazy to
correctly saves the scrolling position (see follow-up commit.)
- Commit:
4aaf5de005720fd3086c249f6854b51a6c86ea92
- From:
- Omar Polo <op@omarpolo.com>
- Date:
typo in parse_session_line
without +4 strtonum sees top=X and cur=Y which are invalid numbers, so
the scroll position is lost.
- Commit:
8d39ecac31b149e6d285c931679c7463daebf273
- From:
- Omar Polo <op@omarpolo.com>
- Date:
sync changelog
- Commit:
638823f847d99e105ee6671fe1c51f8f048a0e48
- From:
- Omar Polo <op@omarpolo.com>
- Date:
don't reset scroll position if the cursor changed line
If the user managed to move the cursor before the page load, don't
restore the scroll position. This is to avoid unnecessary jumping and
causing possible confusion on the user.