Wed, 21 Dec 2005 18:36:19 +0000
[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.
|
11332
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
1 | #!/bin/bash |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
2 | |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
3 | METHOD_NAME=$1 |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
4 | |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
5 | if test -z "$METHOD_NAME" |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
6 | then |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
7 | cat <<EOF |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
8 | This program calls gaim API functions using DBus. As opposed to gaim-send, |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
9 | it does not print the return value. |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
10 | |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
11 | Usage: |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
12 | |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
13 | $0 method-name type1:parameter1 type2:parameter2 ... |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
14 | |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
15 | This shell script just invokes dbus-send, see man dbus-send for how |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
16 | to specify the parameters. |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
17 | |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
18 | Examples: |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
19 | |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
20 | $0 GaimCoreQuit |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
21 | |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
22 | Use dbus-viewer to get the list of supported functions and their parameters. |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
23 | EOF |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
24 | exit 1 |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
25 | fi |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
26 | |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
27 | shift |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
28 | dbus-send --dest=org.gaim.GaimService --type=method_call /org/gaim/GaimObject org.gaim.GaimInterface.$METHOD_NAME "$@" |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
29 | |
|
35bbaaee70a7
[gaim-migrate @ 13545]
Piotr Zielinski <zielaj@users.sourceforge.net>
parents:
diff
changeset
|
30 | echo |