22 Commits (d184dc2f61e047e548b2e3daa11968dbc675258e)

Author	SHA1	Message	Date
Jethro Grassie	b254539efb	readline: don't dereference possible NULL pointer	5 years ago
Martijn Otto	bd98e99c80	Removed a lot of unnecessary includes	6 years ago
moneromooo-monero	8f3c793749	readline_buffer: fix "cursor in prompt" bug ... It happens when readline displays a prompt just before switching to a shorter one	6 years ago
Jethro Grassie	21bd21576d	remove unnecessary include	6 years ago
moneromooo-monero	09ce03d612	move includes around to lessen overall load	7 years ago
Howard Chu	c088d38a57	Simplify readline support ... And don't use std::mutex	7 years ago
Jethro Grassie	1249a2a550	Construct on first use for completion_commands	7 years ago
Jethro Grassie	67dd4933e6	Add sync lock on stop ... Also added and moved two free's hoping to fix leaks.	7 years ago
Jethro Grassie	d75cff1a0e	Fix readline prompt when command does not output	7 years ago
Riccardo Spagni	ab594cfee9	Merge pull request #2159 ... 6955976b Add various readline related fixes (Jethro Grassie)	7 years ago
Jethro Grassie	6955976b2d	Add various readline related fixes ... - Add missing unbind key - Fix colored messages - Add command completion - Preserve last command input - Fix cursor position issues - Fix trailing whitespace in commands - Synchronize set_prompt	7 years ago
Jethro Grassie	6a23cd07aa	Add notification on stop	7 years ago
Riccardo Spagni	7995dcff94	Merge pull request #2112 ... 76043b17 monero-wallet-cli: hang on exit in readline code (#2117) (moneromooo-monero) a73a42a6 monero-wallet-cli: hang on exit in readline code (#2117) (moneromooo-monero) be9d4f04 Fix multiline wallet cli output with readline (Jethro Grassie)	7 years ago
moneromooo-monero	76043b17fd	monero-wallet-cli: hang on exit in readline code (#2117) ... readline_buffer: move a local to local scope Also limit the select fd limit to what we use Signed-off-by: Jethro Grassie <jtg@xtrabass.com>	7 years ago
moneromooo-monero	a73a42a6b0	monero-wallet-cli: hang on exit in readline code (#2117) ... readline_buffer: fix start/stop threads being starved by process process could run for quite some time re-acquiring the process lock, leaving start/stop starving. Yielding after unlock in process is much better but doesn't seem to be enough to reliably yield, so we sleep for a millisecond, which should be transparent for user input anyway. Signed-off-by: Jethro Grassie <jtg@xtrabass.com>	7 years ago
Jethro Grassie	be9d4f0411	Fix multiline wallet cli output with readline ... monero-wallet-cli commands which have multine output sometimes causes issues with the readline support. This patch fixes show_transfers, payments and incoming_transfers.	7 years ago
Jethro Grassie	fdccf7e6c3	Add readline improvements ... Color prompt now working and no reprompting on exit command.	7 years ago
Riccardo Spagni	0c6ea4f8a6	Merge pull request #2111 ... 8261ba69 readline_buffer: fix busy wait (moneromooo-monero)	7 years ago
moneromooo-monero	8261ba69e6	readline_buffer: fix busy wait ... It'd eat up a core constantly, due to spending its time jumping back and forth between userland and kernel. We now wait for up to a millisecond in kernel, which will be transparent to the user and drop to idle most of the time.	7 years ago
moneromooo-monero	21b939e1e6	readline: fix invalid memory access ... m_cout_buf was not initialized	7 years ago
moneromooo-monero	5f8f290505	readline_buffer: add a couple const	7 years ago
jethro	e1f3dfccc8	Add readline support to cli ... This PR adds readline support to the daemon and monero-wallet-cli. Only GNU readline is supported (e.g. not libedit) and there are cmake checks to ensure this. There is a cmake variable, Readline_ROOT_DIR that can specify a directory to find readline, otherwise some default paths are searched. There is also a cmake option, USE_READLINE, that defaults to ON. If set to ON, if readline is not found, the build continues but without readline support. One negative side effect of using readline is that the color prompt in the wallet-cli now has no color and just uses terminal default. I know how to fix this but it's quite a big change so will tackle another time.	7 years ago