| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Sometimes this happened:
`ox d<tab>` → `/ox request`
`ox a<tab>` → `/ox request`
`ox c<tab>` → `/ox keys`
`ox s<tab>` → `/ox keys`
`ox e<tab>` → `/ox keys`
We didn't reset the ox_*_ac variables.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Fix https://github.com/profanity-im/profanity/issues/1693
```
@StefanKropp @DebXWoody please check `_handle_ox_chat()` I don't
understand what you are doing there.
1)
First plain is assigned `message->plain =
p_ox_gpg_decrypt(xmpp_stanza_get_text(ox));`
and then in the same if block you overwrite this with `message->plain =
xmpp_stanza_get_text(b);` without freeing the old value as far as I can
see.
2)
Sometimes even doing `message->plain = "OX error: No payload found";`.
Shouldn't there be a `strdup()`? I think later on we try to free the
whole message struct. So we can't mix this static things.
```
|
| |\ \
| | | |
| | | | |
Fix SIGABRT when using wrong argument order for receiving ox key
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Asserting here is not the right thing to do at all.
A person could have a typo in the fingerprint. Or like in the case of
the reported bug just confuse the arguments.
An additional check for valid jid should be added later to the calling
function maybe.
Fix https://github.com/profanity-im/profanity/issues/1698
|
| | | | |
|
| | |/ |
|
| |/ |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When editing multi-line messages or comments everything past the first
newline becomes invisible.
This patch fixes it by substituting a Unicode symbol for "return"
instead of printing the newline as is. On locales where it's not
available single backslash is used instead.
|
| |
| |
| |
| |
| | |
Allow Readline to control when to redisplay the input line. This makes
text entered via /editor visible after editor is closed.
|
| |
| |
| |
| |
| | |
When doing an interactive search the prompt is needed to show the
current state of the search to avoid confusion.
|
| |
| |
| |
| |
| | |
Since /editor can be used also for commands it doesn't make sense to
restrict it to only connected state.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Dont show presence status changes by default in chat and muc windows.
Users repeatedly ask how to change this setting in the MUC.
We have it in the FAQ since a long time.
And most people I know change this setting, including me.
So it looks like it's time to change this default.
|
| |\
| | |
| | | |
Allow bold for default colours
|
| | | |
|
| |\ \
| | | |
| | | |
| | | |
| | | | |
Fix theme load
When we execute `/theme load somethingthatdoesntexist` it still loaded some colors wrongly.
|
| | |/
| | |
| | |
| | | |
doesn't exist
|
| |\ \
| | | |
| | | | |
Use correct colour for their nick when showing history
|
| | |/ |
|
| | |
| | |
| | |
| | | |
See <https://github.com/profanity-im/profanity/issues/1512>
|
| |/
| |
| |
| | |
The variable `ccount->tls_policy` was not null-tested before `strcmp`.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
... I hope ...
We also need to create the path where the logs are stored.
Fixup of d82f2f293b056965fc0bed2b24820961885ab90b
Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
|
| |
| |
| |
| |
| |
| |
| | |
43e5f15e66b83ac8f3d57d2ffc7babf57873c51c broke the creation of the path
where chatlogs are stored. This is fixed now.
Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
|
| |\
| | |
| | | |
Tab completion code simplification
|
| | |
| | |
| | |
| | |
| | |
| | | |
Factor out common code for Tab and Shift-Tab in a separate function.
No functional change.
|
| |\ \
| | | |
| | | | |
Add Alt/Ctrl-arrows combinations for GNU Screen
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
GNU Screen terminal emulator is modelled after VT100, its key bindings
for "application mode" (which profanity enables on startup) can be
seen with this: info screen "Input Translation".
|
| |\ \ \
| | | | |
| | | | | |
Make all existing input functions bindable from config
|
| | |/ / |
|
| |\ \ \
| | | | |
| | | | | |
Even moar cleanup
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
|
| | | | |
| | | | |
| | | | |
| | | | | |
Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Print error message from `_string_matches_one_of()` instead of forming
an error message manually that contains the same entries that were checked
in `_string_matches_one_of()`.
Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
|
| | |/ /
| | | |
| | | |
| | | |
| | | |
| | | | |
I missed them the last time ...
Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
|
| |/ /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The current code is inherently racy: if screen update takes
considerable time (e.g. when working over network) and a user
performed a series of resizes the final event might get ignored and
the display will be left in inconsistent state.
Fix the race by unsetting the flag first so if the next WINCH signal
is received while display is resizing it'll be processed on the next
iteration.
|
| | |
| | |
| | |
| | | |
Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
|
| | |
| | |
| | |
| | |
| | |
| | | |
... instead of copying the same pattern over and over again.
Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
|
| |/
| |
| |
| |
| |
| |
| |
| |
| | |
* use custom memory descriptor that `abort()`s on `malloc()` failure
* use static log descriptor
* don't always re-create all contexts
* de-duplicate code of `.._connect()` and `.._register()`
Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The current code enters an infinite loop if the input string happens
to get an invalid utf-8 sequence somehow. For me it was reproducible
by running profanity in a Screen session and pressing Alt-т (cyrillic
letter).
Fix it the way borrowed from 0501e49623f68aa39508e4e622924c1dd8147588
where mbrlen is used for the same purposes.
|
| | |
|
| |
| |
| |
| |
| | |
PREF_INCOMING_STR -> PREF_INCOMING_STAMP
PREF_OUTGOING_STR -> PREF_OUTGOING_STAMP
|
| | |
|
|/
|
|
| |
command allow override standard stamps of incoming and outgoing messages
|
|\
| |
| | |
Add none option for /who
|
| |
| |
| |
| |
| | |
Fixes https://github.com/profanity-im/profanity/issues/1425
`/who none` now displays all users in an muc without an affiliation
|
|/
|
|
|
|
|
| |
When preferences do not specify a program to be used for /editor
command, try getting it from EDITOR (which POSIX.1-2017 calls one of
"variables that are frequently exported by widely used command
interpreters and applications"), fall back to "vim" if not set.
|
|\
| |
| | |
Reset url autocompletion after open/save
|
| |
| |
| |
| |
| |
| |
| | |
I guess we should reset the position after we ran `/url open|save`.
So that next time `/url open <tab>` starts with the latest entry.
Fix https://github.com/profanity-im/profanity/issues/1654
|