pidgin3: changelog

Wed, 21 Dec 2005 21:59:18 +0000
by Richard Laager <rlaager@pidgin.im> [Wed, 21 Dec 2005 21:59:18 +0000] rev 12605
[gaim-migrate @ 14940]
Part of SF Patch #1378464 from Sadrul

This removes code duplication in the format toggling functions.

Wed, 21 Dec 2005 21:48:38 +0000
by Sadrul Habib Chowdhury <sadrul@pidgin.im> [Wed, 21 Dec 2005 21:48:38 +0000] rev 12604
[gaim-migrate @ 14939]
SF Patch #1383857 from Sadrul

"This patch makes the display-msg signals to be emitted
from gtkconv instead of conversation. The display-msg
signals were not too useful before, especially because
they were being emitted right after writing-msg
signals, and not being emitted for messages being sent
at all.

This patch also makes necessary changes in the plugins,
and in Changelog.API. With this change, the notify
plugin can set the URGENT hint for a new received
message which creates a conversation.

marv and charkins seemed to think this was a good thing
to do."

committer: Richard Laager <rlaager@pidgin.im>

Wed, 21 Dec 2005 20:59:16 +0000
by Richard Laager <rlaager@pidgin.im> [Wed, 21 Dec 2005 20:59:16 +0000] rev 12602
[gaim-migrate @ 14937]
Adding a case sensitive option as per SF #1387378.

Wed, 21 Dec 2005 18:43:39 +0000
by Richard Laager <rlaager@pidgin.im> [Wed, 21 Dec 2005 18:43:39 +0000] rev 12601
[gaim-migrate @ 14935]
Jason LeBrun wrote to gaim-devel:

"I have found a small quirk in the way gdk_pixbuf_loader works. When you
are using it without signalling, the proper way to use it is to call
gdk_pixbuf_loader_close *before* calling gdk_pixbuf_loader_get_animation
or gdk_pixbuf_loader_get_pixbuf. The call to gdk_pixbuf_loader_close
signals that no more writes will be occuring.

In particular, this affects images that are less than 1k in size. If
gdk_pixbuf_loader_close is not called before _get_animation, the loader
will not return anything unless it has received more than 1k of data
(the file type sniffing buffer size) or it has been closed.

So, the proper order of calls for loaders in the gtk*.c code is:

gdk_pixbuf_loader_new();
gdk_pixbuf_loader_write();
gdk_pixbuf_loader_close();
gdk_pixbuf_loader_get_animation();"

I know we fixed a bug by changing this in one place. I've gone through and updated the rest.

Wed, 21 Dec 2005 18:36:19 +0000
by Richard Laager <rlaager@pidgin.im> [Wed, 21 Dec 2005 18:36:19 +0000] rev 12600
[gaim-migrate @ 14934]
Enable the extra warnings regardless of --enable-debug.

Enable FORTIFY_SOURCE regardless of --enable-debug, adding a --disable-fortify flag to configure.

Enable (well, stop disabling) the missing initializer warnings.
This leads to warnings with: GValue v = {0,}; that must be worked around.
Basically, instead of:
GValue v = {0,};
...
g_value_init(&v, G_TYPE_FOO); /* or other use of the GValue */
We'd need to do:
GValue v;
...
v.g_type = 0;
g_value_init(&v, G_TYPE_FOO); /* or other use of the GValue */

Fix several cases of missing initializers. I don't think any of these are bugs, but having this warning seems like a good idea. It might prevent us from making a mistake in the future.

While I was fixing missing initializers, I optimized substitute_simple_word in plugins/spellchk.c, in the same way as I did substitute_word before. Yes, I'm bad for committing these together.

Added a --enable-fatal-asserts flag to configure. As the name implies, this makes g_return_... guards fatal. This is a useful flag to run on a debug copy of Gaim. It will make it very clear if your changes have triggered one of these guards. It's also useful in detecting g_return_... abuse, which helps prevent crashes if Gaim is compiled with G_DISABLE_ASSERT defined.

Wed, 21 Dec 2005 13:37:18 +0000
by Nathan Walp <nwalp@pidgin.im> [Wed, 21 Dec 2005 13:37:18 +0000] rev 12599
[gaim-migrate @ 14929]
i don't know how this is happening, but apparently it is, so now we'll protect against it

Wed, 21 Dec 2005 12:22:04 +0000
by Richard Laager <rlaager@pidgin.im> [Wed, 21 Dec 2005 12:22:04 +0000] rev 12598
[gaim-migrate @ 14928]
Fix an assertion failure. Remove a useless cast.

Wed, 21 Dec 2005 12:08:51 +0000
by Sadrul Habib Chowdhury <sadrul@pidgin.im> [Wed, 21 Dec 2005 12:08:51 +0000] rev 12597
[gaim-migrate @ 14927]
SF Patch #1357805 from Sadrul

"Statusbox: resize
This patch resizes the statusbox-entry after the entry.
It shows a max. of 4 lines, otherwise resizes the
height so that there's no empty place. Try it now!!"

This one has lots of changes from me. It works over here.

committer: Richard Laager <rlaager@pidgin.im>

Wed, 21 Dec 2005 10:56:07 +0000
by Richard Laager <rlaager@pidgin.im> [Wed, 21 Dec 2005 10:56:07 +0000] rev 12596
[gaim-migrate @ 14926]
Squeeze that statusbox!

Mercurial > pidgin3 / changelog