| Commit message (Collapse) | Author | Age | Lines |
|
|
|
|
|
|
|
|
|
| |
Now that we are no longer blocking on Readline calls, we can set up a
self-pipe that will let us break out of the Readline loop upon receiving
a "quit" command on our local socket, thus (finally) fixing issue #143.
(Thanks to https://stackoverflow.com/a/27662212 for the tip!)
Fixes #143
|
|
|
|
|
| |
By only invoking rl_callback_read_char() when there is actual input on
stdin, we can now avoid blocking on Readline calls.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When using Readline's callback interface, signals are (by default) only
captured within the duration of the rl_callback_read_char() call. It is
therefore expected of the application to capture SIGWINCH on its own,
and notify Readline of the fact.
(While it's possible to change this with rl_persistent_signal_handlers,
some signals, such as SIGHUP, will still only be *processed* during that
call, making it a somewhat unappealing solution. This all could be
alleviated by calling rl_check_signals() periodically, but that function
was only introduced in Readline 8.0, which is far too recent.)
Note that we are using signal(2) and not sigaction(2), depite the
various warnings that come with it, mostly because it's what is already
present in the codebase.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When Twinkle is running in CLI mode and is sent a "quit" command to its
local socket, it will currently not respond immediately, but rather wait
until the next line has been read from its standard input (issue #143).
This is due to the blocking nature of readline(), which only returns
once a complete line has been read. Switching to Readline's "alternate"
callback interface is the first step in addressing this issue.
(As a bonus, this also fixes a bug where the line pointer returned by
readline() was not freed correctly.)
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
Since Qt 5.11, generated ui_getprofilename.h no longer includes QHeaderView
which breaks the chain that included qvalidator.h in getprofilename.cpp.
As it feels rather fragile to rely on such indirect includes, let's include
<QRegExpValidator> explicitly in each file using QRegExpValidator class.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
There's a potential ambiguity between ::bind() and std::bind() if
<functional> happens to be included beforehand (as is the case with
libc++).
|
| |
|
|\
| |
| | |
PAI Header option
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
When you reject a call, either
A. the forwarding rule for User-Busy kicks in,
like an answering machine or another extension (of a personal secretary or a deputy). Or
B. the caller hears normal ringing first, and then the busy tone.
To allow case A, a SIP user-agent client (UAC) has to return the status User-Busy (486). The same should happen in case B, because the caller shall hear a busy tone on rejection. 486 is mandated for SIP clients in mobile phones by the GSMA. Furthermore, other SIP-client creators give a 486 in that case as well; I tested Acrobits, Atlinks, Counterpath, Gigaset, Grandstream, RTX, Snom, Yealink, and Vtech.
Before this change, twinkle rejected a call with the SIP-Status 603. The class 6xx requires that *all* other registered phones stop to ring. Furthermore, some SIP user-agent servers (UAS) follow RFC 3398 and map a status 603 to the cause-code 21, which is mapped back to status 403 (Forbidden). Cisco and Digium Asterisk do this. For example in Asterisk, after you rejected the call in twinkle, the caller did not get 603 or 486, but 403. However, in case of 403, the original caller is allowed to re-try the call setup. For example, I have a Nokia Symbian/S60 based phone which tries via IPv4 first, then after it received the 403, that phone tries again via IPv6. Consequently, a user of twinkle had to reject the call twice when 603 was returned.
|
| | |
|
|/ |
|
| |
|
|
|
| |
I've updated the German translation file.
|
| |
|
|
|
|
| |
(This is just cut-and-paste, also removing some trailing whitespace.)
|
|
|
|
| |
Fixes #82
|
|\
| |
| | |
Fix two issues with SelectUserForm
|
| | |
|
| | |
|
|\ \
| | |
| | | |
Fix two issues with multiple number conversion rules in UserProfileForm
|
| | | |
|
| |/ |
|
|\ \
| | |
| | | |
Skip over the mutex field when copying t_call_info data
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
t_line::get_call_info() returns a copy of a (locked) t_call_info;
copying the mutex, aside from being undefined in POSIX, returns an
unexpectedly locked copy, possibly resulting in a deadlock.
This fix is pretty much a copy of the one applied to t_call_record in
38bb6b7, with an added check for self-assignment. Like its predecessor,
it is still susceptible to deadlock, though (as unlikely as it may be).
Fixes #73
|
|\ \
| | |
| | | |
Replace (old) FSF postal address with URL in header files
|
| | |
| | |
| | |
| | |
| | |
| | | |
Wording copied from https://www.gnu.org/licenses/gpl-howto.en.html
Fixes #25, fixes #71
|
| |/
|/|
| |
| | |
Fixes #74
|
| |
| |
| |
| |
| |
| |
| | |
The correct French term for "address book" is "carnet d'adresses", as
documented in the following term record:
http://www.gdt.oqlf.gouv.qc.ca/ficheOqlf.aspx?Id_Fiche=8365993
|
|/
|
|
|
|
| |
The confirmation prompt when deleting a contact from the local address
book (added in 57dc032) contains two new translatable strings; this
commit provides their French translation.
|
|
|
|
| |
during conversations
|
| |
|
|
|
|
|
|
|
| |
DTMF event constants handling is a bit messy as three different types
are used in different source files, triggering various compiler
warnings. Introduce t_dtmf_ev type and use it everywhere, also replace
helper macros by inline functions.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|\
| |
| | |
Local address book: re-sort after adding/editing, and confirm before deleting
|
| | |
|
| |
| |
| |
| |
| | |
(The proper solution is probably to create a filter proxy model, but
this nevertheless does the job.)
|
|/ |
|