| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
It works fine as long as a preallocated GArray doesn't move its memory
buffer for no reason, but it's good to be obviously correct. Spotted while
working on the previous commit.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71093
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In order for this to work, TpProtocol now ignores parameters whose
D-Bus signatures are not a syntactically valid single complete type.
This is helpful for Mission Control to be able to migrate parameters
from untyped to typed storage - it stores parameters in terms of
GVariant.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71093
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
[added (transfer full) as requested in review -smcv]
|
|\
| |
| |
| |
| |
| | |
Conflicts:
NEWS
configure.ac
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
TelepathyGLib-1 isn't going to be fully compatible.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=49737
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
|
| |
| |
| |
| |
| | |
Please mention bug numbers in your commits, particularly if they're
for stable.
|
| | |
|
| | |
|
|\ \
| |/ |
|
| | |
|
| |
| |
| |
| |
| |
| |
| | |
TelepathyGLib-1 isn't going to be fully compatible.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=49737
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| |
| |
| |
| |
| |
| |
| | |
Based on a patch by Guillaume Desmottes.
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=68390
|
| |
| |
| |
| |
| | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| |
| |
| |
| |
| |
| |
| |
| | |
It was deprecated in GLib 2.36, and we already have a hard dependency
on that version.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
tp_file_transfer_channel_accept_file_async() and
tp_file_transfer_channel_provide_file_async() operations are supposed
to complete as soon as the CM returns from AcceptFile or ProvideFile.
That means that we cannot call operation_failed() for streaming errors.
We also have to keep a ref on self while streaming the file to
avoid a crash in the callback when we dereference self.
This means that the client app cannot cancel the ongoing streaming
by unreffing the channel, replying on dispose calling g_cancellable_cancel().
It can still be doing using g_object_run_dispose() though. To make it cleaner
we should probably add tp_file_transfer_channel_cancel().
The spec does not provide any way for the streaming client to inform
the CM and other clients about the error occured while streaming.
TpFileTransferChannel API does not even have a way to propagate that
error to the user.
|
|\ \
| |/ |
|
| |\ |
|
| | |
| | |
| | |
| | |
| | | |
glib 2.39.0 started to print a g_critical() message when calling
g_source_remove() on a source that does not exist anymore.
|
| | |
| | |
| | |
| | |
| | | |
Also link to https://bugzilla.gnome.org/show_bug.cgi?id=697348 to
explain why we still have to unset STARTER env variables.
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Our D-Bus API no longer supports anything else.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
To make the test a little more interesting and a little more realistic,
we normalize the 'account' parameter to lower-case.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
We previously passed the value when parsed as a string, but that
makes very little sense. The raw value is at least used for something
by the function itself, and it's better for debug messages too.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | | |
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71048
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
I needed this for TpExportable (which is still work in progress),
but it's already a small code reduction.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59024
Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
Having 'UNRELEASED' in a source file makes the release checker unhappy.
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
_TP_DEPRECATED_IN_0_24 was already defined but not this one.
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|