| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
Testing Done:
Touched all source files and compiled; saw no new warnings.
Reviewed at https://reviews.imfreedom.org/r/2122/
|
|
|
|
|
|
|
|
|
| |
See [this development thread for a future Fedora change](https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/thread/CJXKTLXJUPZ4F2C2VQOTNMEA5JAUPMBD/) or [the proposed change page](https://fedoraproject.org/wiki/Changes/PortingToModernC). These may be made stronger errors in GCC 14.
Testing Done:
Configured with `-Dc_args='-Werror=implicit-int -Werror=implicit-function-declaration -Werror=int-conversion -Werror=strict-prototypes -Werror=old-style-definition'` and compiled.
Reviewed at https://reviews.imfreedom.org/r/1974/
|
|
|
|
|
|
|
|
|
| |
It is a one line wrapper for it.
Testing Done:
Compiled.
Reviewed at https://reviews.imfreedom.org/r/1865/
|
|
|
|
|
|
|
| |
Testing Done:
Compiled, but didn't test on windows. As such I left all the networking stuff untouched as that's where windows will really need it and that can be done in another rr.
Reviewed at https://reviews.imfreedom.org/r/1764/
|
|
|
|
|
|
|
| |
Testing Done:
Compiled
Reviewed at https://reviews.imfreedom.org/r/1750/
|
|
|
|
|
|
|
|
|
| |
And also fix all properties/signals/non-docs that were incorrect.
Testing Done:
Built docs and found no new warnings. Checked some docs for newly-available property/signal docs.
Reviewed at https://reviews.imfreedom.org/r/1568/
|
|
|
|
|
|
|
| |
Testing Done:
Sent ims in both pidgin3 and finch3, also joined a chat in pidgin3 and verified that the preferences window is okay.
Reviewed at https://reviews.imfreedom.org/r/1077/
|
|
|
|
|
|
|
|
|
| |
Ran `codespell -w`, ignoring some stuff, or making things better.
Testing Done:
Compile only.
Reviewed at https://reviews.imfreedom.org/r/1016/
|
|
|
|
|
|
|
| |
Testing Done:
Compiled and ran without an obvious issues.
Reviewed at https://reviews.imfreedom.org/r/888/
|
|
|
|
|
|
|
| |
Testing Done:
Full build after a `meson --wipe`
Reviewed at https://reviews.imfreedom.org/r/863/
|
|
|
|
|
|
|
| |
Testing Done:
Compile only
Reviewed at https://reviews.imfreedom.org/r/791/
|
|
|
|
|
|
|
|
|
| |
subclasses
Testing Done:
Tested a bonjour im and irc chat.
Reviewed at https://reviews.imfreedom.org/r/646/
|
|
|
|
|
|
|
|
|
| |
Testing Done:
Compiled and ran locally.
Bugs closed: PIDGIN-17472
Reviewed at https://reviews.imfreedom.org/r/538/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This add a static inline version of `g_memdup2` if the version of glib we're
compiling against doesn't have the function.
GHSL-2021-045 was originally reported to glib at
https://gitlab.gnome.org/GNOME/glib/-/issues/2319.
More information about the entire situation can be found on the gnome
desktop-devel-list at
https://mail.gnome.org/archives/desktop-devel-list/2021-February/msg00000.html
Testing Done:
Compiled and ran unit tests.
Reviewed at https://reviews.imfreedom.org/r/482/
|
|
|
|
|
|
|
| |
Testing Done:
Compile only, mostly; began receiving an irc transfer from grim, though ports were closed, so it never started.
Reviewed at https://reviews.imfreedom.org/r/274/
|
|
|
|
| |
with the repercussions of that.
|
|
|
|
| |
some pieces to purpleprivate.h
|
| |
|
| |
|
|
|
|
|
|
|
| |
UI data is stored on GObjects and automatically destroyed, so this isn't
really needed as a UI op for *our* UIs. There will be a replacement soon
if needed by other UIs.
|
| |
|
|
|
|
|
| |
As a GObject, UIs can use g_object_[gs]et_data instead.
|
| |
|
|
|
|
|
| |
UIs can connect to notify::status on PurpleXfer instances instead.
|
|
|
|
|
|
| |
Progress can be monitored by connecting to notify::progress on
PurpleXfer instances.
|
|
|
|
|
|
| |
It can be monitored by connecting to notify::status on an individual
PurpleXfer instead.
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Some redundant assignment, missing NULL-check, and missing frees.
|
|
|
|
|
| |
Fixes unused, and possibly unitialized variable warnings.
|
|
|
|
|
|
| |
Note, since that function uses base 10 units, I also changed the
transfer speeds to base 10 (KB/s, not KiB/s).
|
|
|
|
|
| |
These are used to calculate elapsed transfer times.
|
|
|
|
|
|
| |
The `*_get_instance_private` function just returns input value + offset,
so it is impossible for `priv` to be `NULL`.
|
|
|
|
|
|
| |
It should not be necessary in the private one, because these checks were
done in the public one.
|
| |
|
|
|
|
| |
that support file transfers
|
|
|
|
|
|
|
|
| |
This patch ports the almost self-contained libpurple GObjects away
from the deprecated g_type_class_add_private() function in favor of
the G_DEFINE_TYPE_WITH_PRIVATE() macro. None of these require
changes to anything other than their own .c and .h files. It removes
the const qualifier from some of the functions for simplicity.
|
| |
|
|
|
|
|
|
|
|
| |
With the addition of PurpleProtocolXferInterface, building in Windows
fails due to the redefinition of send() as wpurple_send(). Since a
lot of the prpls already call it this their implementations, this
patch renames it to send_file(). This fixes building on Windows
again.
|
| |
|
|
|
|
| |
auxiliary work that needs to be done to make it work
|
| |
|
| |
|
|
|
|
| |
instance as well
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch fixes a mixed declarations and code warning and a discards
'const' qualifier from pointer target type warning:
xfer.c:1927:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
gpointer *old_thumbnail_data = priv->thumbnail_data;
xfer.c:1942:9: warning: passing argument 1 of ‘g_free’ discards ‘const’ qualifier from pointer target type [-Wdiscarded-qualifiers]
g_free(old_mimetype);
It also changes the old_thumbnail_data variable to match the type of
thumbnail_data in the priv structure.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Consider sending a small (say 10 bytes) file in do_transfer().
First, the buffer is filled in purple_xfer_read_file(), which also calls
purple_xfer_set_bytes_sent(); because the file is small, bytes sent now
equals the file size. Later in purple_xfer_write(), the size of
the write buffer is adjusted so that it's not bigger than the remaining
portion of the file being sent - but at that point
purple_xfer_get_bytes_remaining() already returns 0, nothing is written
and the file transfer gets stuck.
As I understant it, the meaning of buffer size adjustment in
purple_xfer_write() is to ensure that external protocol plugin which
handles file transfer input and output on its own can't by accident send
more data than the size of the file. In do_transfer(), though, that
check is redundant and can be skipped.
|
|
|
|
| |
purple_xfer_set_bytes_sent() and purple_xfer_set_completed(). Also, remove needless member bytes_remaining from PurpleXferPrivate. Works reliably for me on XMPP. However, the redundant purple_xfer_set_completed() calls may be relied upon by other protocols. This needs to be tested. Ok'd by elb. Refs #16001
|