Claudio Cambra authored 3 weeks ago and Felix Paul Kühne committed 3 weeks ago

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>

Claudio Cambra authored 3 weeks ago and Felix Paul Kühne committed 3 weeks ago

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>

Claudio Cambra authored 3 weeks ago and Felix Paul Kühne committed 3 weeks ago

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>

Claudio Cambra authored 3 weeks ago and Felix Paul Kühne committed 3 weeks ago

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>

egwg

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>

afefa

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>

Claudio Cambra authored 4 weeks ago and Felix Paul Kühne committed 3 weeks ago

Signed-off-by: Claudio Cambra <developer@claudiocambra.com>

Marvin Scholz authored 4 months ago and Steve Lhomme committed 3 weeks ago

This is needed with the sha512sum introduced on recent macOS versions
and matches what is already done for contribs too.

We don't need vlc_strtof_c() since we already have the float value.
We also don't need to check what is after the number (a comment for example).
As long as it's a number it should be FPS. Anything else other than
"FORMAT=TIME" is bogus. So we don't need to get the remaining part of the string.

We still need to look for the wait & duration values after that.

Most of the time that's the first line we will get.
If not the line can contain different things other than a FORMAT setting.

And calling vlc_strtof_c without checking the lines contains 2 values
was bound to use 0 if the line did not actually contain numbers.

Fatih Uzunoğlu authored 3 months ago and Steve Lhomme committed 3 weeks ago

For unresolved items, currently the title might be "Unknown
Title". In this case, there is no information provided to the
user regarding the item to recognize what it is.

Marvin Scholz authored 4 weeks ago and Steve Lhomme committed 3 weeks ago

HAVE_BROKEN_QSORT_R implies HAVE_QSORT_R, so checking both is not
necessary.

Alexandre Janniaux authored 4 weeks ago and Steve Lhomme committed 3 weeks ago

This was not supposed to be committed (hence the lack of casts to string
chars).

Prince Gupta authored 1 month ago and Steve Lhomme committed 3 weeks ago

otherwise you've interface scale show up in UI

Prince Gupta authored 1 month ago and Steve Lhomme committed 3 weeks ago

fixes interface scale change on setting dialog show

Fatih Uzunoğlu authored 1 month ago and Steve Lhomme committed 3 weeks ago

qt: introduce `MatrixChangeObserverNode` and use it to signal `ViewBlockingRectangle::scenePositionHasChanged()`

Fatih Uzunoğlu authored 1 month ago and Steve Lhomme committed 3 weeks ago

This reverts commit af27693d.

Fatih Uzunoğlu authored 1 month ago and Steve Lhomme committed 3 weeks ago

Previously the window handle was retrieved through private platform
interface. It was recently changed to use `QWindow::winId()` instead,
as it is public and more convenient to use.

However, `QWindow::winId()` creates the platform window if it does
not exist. This is normally not a problem, but since `WinId()` is
may be called within the native event handler, it is a problem.

I did not realize that `WinId()` was called at inappropriate times
(either during destruction of the platform window when the window
is destroyed, or after that as we don't want to re-create the platform
window when closing the interface), that's why I did not pay attention
on having a check here. Now that the problem is obvious, we should
fix this mistake made in 7e063c67.

This is trivial to fix, we can simply check if the pointer returned
by either `QWindow::handle()` or `QSurface::surfaceHandle()` is a
null pointer, and return early if it is before calling `QWindow::winId()`.

As a side note, I believe that having a const method named as `winId()`
which mutates the window by creating platform window if it does not
exist is a really bad idea. I really wonder why Qt behaves like that,
especially considering there is already `QWindow::create()` that
creates the platform window.

Johannes Kauffmann authored 1 month ago and Steve Lhomme committed 3 weeks ago

vlc_pw_stream_new() / pw_stream_new() takes ownership of props and calls
pw_properties_free() in case of any error.

Johannes Kauffmann authored 1 month ago and Steve Lhomme committed 3 weeks ago

vlc_pw_stream_new() / pw_stream_new() takes ownership of props, but we
need to delete the properties ourselves in case of any error before that
function is called.

Florian Albrechtskirchinger authored 1 month ago and Steve Lhomme committed 3 weeks ago

In the 2024.11.28 release (version 1732752000), live555 introduced
`EventLoopWatchVariable` as a typedef for `std::atomic<char>`, replacing
direct use of `char` in earlier versions. Add a conditional typedef to
define it as `char` for older versions, and update `event_rtsp` and
`event_data` to use `EventLoopWatchVariable`.

Prince Gupta authored 1 month ago and Steve Lhomme committed 4 weeks ago

fixes artist loading with large medialibrary