forked from baresip/re
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update #2
Open
bacterik
wants to merge
324
commits into
sverigesradio:main
Choose a base branch
from
baresip:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
update #2
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The SDP negotiation state is now tracked in a single enum which is used as a state machine to keep track of the current state of the SDP negotiation.
* test: move test_rtcp_decode_badmsg() to separate testcase * test: fix test_rtcp_decode_badmsg for OOM
* rtp: lock more rtcp_sess fields and code * rtp: replace rtp_source lock by extended rtp_sess lock
* httpauth: digest verification function using RFC 7616 * httpauth: test cases for http digest verification * httpauth: use re_atomic for nc to avoid data race in multithreaded environment * httpauth: fix win onversion warning
* tmr: prevent timer race condition on cancel If `tmr_cancel` is called from two threads it's enough to cancel once. * tmr: rename lock to list lock and improve comments
* rtp: make struct rtp_source public * add rtp_ prefix
If qentp is set, the caller of sip_transp_send is responsible for derefing qent.
original error for C++ file on Windows-ARM64: C:\PROGRA~1\MICROS~2\2022\ENTERP~1\VC\Tools\MSVC\1442~1.344\bin\Hostx64\arm64\cl.exe /nologo /TP -DARCH=\"AMD64\" -DHAVE_ATOMIC -DHAVE_IO_H -DHAVE_PQOS_FLOWID -DHAVE_QOS_FLOWID -DHAVE_SELECT -DHAVE_UNIXSOCK=1 -DOS=\"Windows\" -DRE_VERSION=\"3.18.0\" -DVER_MAJOR=3 -DVER_MINOR=18 -DVER_PATCH=0 -DWIN32 -D_CRT_SECURE_NO_WARNINGS -D_GNU_SOURCE -D_WIN32_WINNT=0x0600 -Dretest_EXPORTS -ID:\a\re\re\test\. -ID:\a\re\re\include /DWIN32 /D_WINDOWS /EHsc /Ob0 /Od /RTC1 -MDd -Zi /W3 /showIncludes /Fotest\CMakeFiles\retest.dir\cplusplus.cpp.obj /Fdtest\CMakeFiles\retest.dir\ /FS -c D:\a\re\re\test\cplusplus.cpp D:\a\re\re\include\re_atomic.h(602): error C2664: 'char __iso_volatile_load8(volatile const char *)': cannot convert argument 1 from 'const unsigned char *' to 'volatile const char *' D:\a\re\re\include\re_atomic.h(602): note: Types pointed to are unrelated; conversion requires reinterpret_cast, C-style cast or parenthesized function-style cast I am not an expert on atomics, perhaps there are better solutions.
* test: add test for C++ applications * cmake: move some compile options to C only * cmake: move -Wshorten-64-to-32 to C-only * sanitizer: add clang++ * test: move compile options to C only * test: fix cmake lint * ci: rename job * atomic: attempt to fix __iso_volatile_load8() compile error * atomic: change unsigned to volatile
re_fhs_flush is used to cleanup OOM tests
* fmt/pl: add pl_strncasecmp and tests
Finds an RTP extension by its ID within an array of RTP extensions
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.