Wed, 15 Aug 2001 18:09:48 +0000
[gaim-migrate @ 2154]
coding style. mmm. Also got rid of po/ChangeLog, it's autogenerated and we don't use it.
| 639 | 1 | A lot of people have tried to hack gaim, but haven't been able to because |
| 2 | the code is just so horrid. Well, the code isn't getting better anytime | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
3 | soon (I hate GNU indent), so to help all you would-be hackers help out |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
4 | gaim, here's a brief tutorial on how gaim works. I'll quickly describe |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
5 | the logical flow of things, then what you'll find in each of the source |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
6 | files. As an added bonus, I'll try and describe as best I can how multiple |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
7 | connections and multiple protocols work. Depending on how much I want |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
8 | to avoid my final tomorrow I may even describe other parts of gaim that |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
9 | I particularly want to brag about. Hopefully that's enough to get most |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
10 | of you going. |
| 639 | 11 | |
|
708
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
12 | If you're going to hack gaim, PLEASE, PLEASE PLEASE PLEASE send patches |
|
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
13 | against the absolute latest CVS. I get really annoyed when I get patches |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
14 | against the last released version, especially since I don't usually |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
15 | have a copy of it on my computer, and gaim tends to change a lot between |
|
708
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
16 | versions. (I sometimes get annoyed when they're against CVS from 3 days |
|
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
17 | ago, but can't complain because it's usually my fault that I haven't |
|
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
18 | looked at the patch yet.) To get gaim from CVS (if you haven't already), |
|
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
19 | run the following commands: |
|
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
20 | |
|
774
97a94a9fd7c0
[gaim-migrate @ 784]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
749
diff
changeset
|
21 | $ export CVSROOT=:pserver:anonymous@cvs.gaim.sourceforge.net:/cvsroot/gaim |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
22 | $ cvs login (hit enter as the password) |
|
708
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
23 | $ cvs co gaim |
|
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
24 | (you'll see it getting all of the files) |
|
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
25 | $ cd gaim |
|
1863
af03c531e79c
[gaim-migrate @ 1873]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1710
diff
changeset
|
26 | $ ./autogen.sh |
|
708
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
27 | |
|
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
28 | You'll now have your normal gaim tree with ./configure and all. (If you |
|
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
29 | want to make your life really simple, learn how CVS works. CVS is your |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
30 | friend.) To make a patch, just edit the files right there in that tree |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
31 | (don't bother with two trees, or even two copies of the same file). Then |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
32 | when you're ready to make your patch, simply run 'cvs diff -u >my.patch' |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
33 | and send it off. |
|
708
c4285ce27acc
[gaim-migrate @ 718]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
706
diff
changeset
|
34 | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
35 | This file was last modified by $Author: warmenhoven $ on |
|
2144
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
36 | $Date: 2001-08-15 14:09:48 -0400 (Wed, 15 Aug 2001) $. |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
37 | |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
38 | |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
39 | CODING STYLE |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
40 | ============ |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
41 | |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
42 | Coding styles are like assholes, everyone has one and no one likes |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
43 | anyone elses. This is mine and if you want me to accept a patch from |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
44 | you without getting annoyed you'll follow this coding style. :) |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
45 | |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
46 | It would probably just be easier for me to include CodingStyle from the |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
47 | linux kernel source. |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
48 | |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
49 | Tab indents. I *HATE* 2-space indents, and I strongly dislike 8-space |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
50 | indents. Use a tab character. I'm likely to refuse a patch if it has |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
51 | 2-space indents. |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
52 | |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
53 | K&R style for braces. Braces always go on the same line as the if, etc. |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
54 | that they're associated with; the only exception is functions. Braces |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
55 | for else statements should have both braces on the same line as the else |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
56 | (i.e. "} else {"). |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
57 | |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
58 | No functionOrVariableNamesLikeThis. Save it for Java. Underscores are |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
59 | your friend. "tmp" is an excellent variable name. Hungarian style will |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
60 | not be tolerated. Go back to Microsoft. |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
61 | |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
62 | I have a 105-char wide Eterm. Deal with it. |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
63 | |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
64 | NO goto. I'm very likely to refuse a patch if it makes use of goto. If |
|
6d6bb304043a
[gaim-migrate @ 2154]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1863
diff
changeset
|
65 | you feel the need to use goto, you need to rethink your design and flow. |
|
684
85f0ef25fe51
[gaim-migrate @ 694]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
639
diff
changeset
|
66 | |
| 639 | 67 | |
| 68 | PROGRAM FLOW | |
| 69 | ============ | |
| 70 | ||
|
979
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
71 | Before gaim does anything you can see, it initializes itself, which is |
|
1038
850b893e1ac9
[gaim-migrate @ 1048]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
980
diff
changeset
|
72 | mostly just reading .gaimrc (handled by the functions in gaimrc.c) and |
|
850b893e1ac9
[gaim-migrate @ 1048]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
980
diff
changeset
|
73 | parsing command-line options. It then draws the login window by calling |
|
850b893e1ac9
[gaim-migrate @ 1048]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
980
diff
changeset
|
74 | show_login, and waits for input. |
|
979
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
75 | |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
76 | At the login window, when "Accounts" is clicked, account_editor() is |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
77 | called. This then displays all of the users and various information |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
78 | about them. If the user clicks the "Signon" button instead, serv_login |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
79 | is called. |
|
979
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
80 | |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
81 | When the "Sign on/off" button is clicked, serv_login is passed the |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
82 | username and the password for the account. If the password length is |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
83 | zero (the password field is a character array rather than pointer so it |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
84 | will not be NULL) then the Signon callback will prompt for the password |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
85 | before calling serv_login. serv_login then signs in the user using the |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
86 | appropriate protocol. We'll assume TOC for the rest of this discussion; |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
87 | even the libfaim guys get scared by oscar.c, and I'll talk about the |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
88 | PRPLs later. |
|
979
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
89 | |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
90 | After you're signed in (I'll skip that discussion - I doubt many people |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
91 | are going to change the login process, since it pretty much just follows |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
92 | PROTOCOL), Gaim draws the buddy list by calling show_buddy_list, and |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
93 | waits for input from two places: the server and the user. The first |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
94 | place it gets input from after signon is usually the server, when the |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
95 | server tells Gaim which buddies are signed on. |
|
979
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
96 | |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
97 | When there is information ready to be read from the server, toc_callback |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
98 | is called (by GDK) to parse the incoming information. On an UPDATE, |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
99 | serv_got_update is called, which takes care of things like notifying |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
100 | conversation windows of the update if need be; notifying the plugins; |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
101 | and finally, calling set_buddy. |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
102 | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
103 | set_buddy is responsible for a lot of stuff, but most of it is done |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
104 | implicitly. It's responsible for the sounds (which is just a call to |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
105 | play_sound), but the biggest thing it does is call new_group_show and |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
106 | new_buddy_show if necessary. There's only one group_show per group name, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
107 | even between connections, and only one buddy_show per group_show per |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
108 | buddy name, even between connections. (If that's not confusing enough, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
109 | wait until I really start describing how the buddy list works.) |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
110 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
111 | New connections happen the exact same way as described above. Each |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
112 | aim_user can have one gaim_connection associated with it. aim_user and |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
113 | gaim_connection both have a protocol field; gaim_connection's should |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
114 | be constant once it is set. (I'll talk about the gaim_connection struct |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
115 | more later.) |
|
979
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
116 | |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
117 | When the user opens a new conversation window, new_conversation is called. |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
118 | That's easy enough. If there isn't a conversation with the person already |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
119 | open (checked by calling find_conversation), show_conv is called to |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
120 | create the new window. All sorts of neat things happen there, but it's |
|
1038
850b893e1ac9
[gaim-migrate @ 1048]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
980
diff
changeset
|
121 | mostly drawing the window. show_conv is the best place to edit the UI. |
|
979
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
122 | |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
123 | That's pretty much it for the quick tutorial. I know it wasn't much but |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
124 | it's enough to get you started. Make sure you know GTK before you get too |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
125 | involved. Most of the back-end stuff is pretty basic; most of gaim is GTK. |
| 639 | 126 | |
| 127 | ||
| 128 | SOURCE FILES | |
| 129 | ============ | |
| 130 | ||
| 131 | about.c: | |
| 132 | Not much to say here, just a few basic functions. | |
| 133 | ||
| 134 | aim.c: | |
| 135 | This is where the main() function is. It takes care of a lot of the | |
| 136 | initialization stuff, and showing the login window. It's pretty tiny | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
137 | and there's not really much to edit in it. This has some of the most |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
138 | pointless functions, like gaim_setup, which optionally turns off sounds |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
139 | on signon. A lot of this file should actually be part of other files. |
| 639 | 140 | |
|
1558
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
141 | applet.c: |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
142 | This controls most things that are related to the applet. I don't like |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
143 | looking at this file because it still has functionsLikeThis. But at |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
144 | least it doesn't have many of them anymore. Anyway, this file isn't |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
145 | very big because there's really not much difference between the panel |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
146 | version and the app version. |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
147 | |
| 639 | 148 | away.c: |
| 149 | This takes care of most of the away stuff: setting the away message | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
150 | (do_away_message); coming back (do_im_back); drawing the away window; |
|
1558
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
151 | etc. Away messages work really oddly due to multiple connections and |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
152 | multiple protocols; I think there are really only two or three people |
|
1619
8a254206f8d4
[gaim-migrate @ 1629]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1558
diff
changeset
|
153 | who know how it works and I don't think any of us know why it works |
|
1558
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
154 | that way. |
| 639 | 155 | |
| 156 | browser.c: | |
| 157 | Code for opening a browser window. Most of the code is trying to deal | |
| 158 | with Netscape. The most important function here is open_url. Have fun. | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
159 | (This file may give you problems with GTK 2.0, because it uses parts |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
160 | of GDK that it's not supposed to know about.) |
| 639 | 161 | |
| 162 | buddy.c: | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
163 | This takes care of not only nearly everything buddy-related (the |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
164 | buddy lists, the window, etc.), but also a lot of the code flow and |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
165 | util functions. Look for good things like find_buddy, set_buddy, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
166 | and signoff here. |
| 639 | 167 | |
| 168 | buddy_chat.c: | |
| 169 | This takes care of the buddy chat stuff. This used to be a lot bigger | |
| 170 | until the chat and IM windows got merged in the code. Now it mostly | |
| 171 | just takes care of chat-specific stuff, like ignoring people and | |
| 172 | keeping track of who's in the room. This is also where the chat window | |
| 173 | is created. | |
| 174 | ||
| 175 | conversation.c: | |
| 176 | This is where most of the functions dealing with the IM and chat windows | |
| 177 | are hidden. It tries to abstract things as much as possible, but doesn't | |
| 178 | do a very good job. This is also where things like "Enter sends" and | |
| 179 | "Ctrl-{B/I/U/S}" options get carried out (look for send_callback). The | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
180 | chat and IM toolbar (with the B/I/U/S buttons) are both built from |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
181 | the same function, build_conv_toolbar. |
| 639 | 182 | |
| 183 | dialogs.c: | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
184 | A massive file with a lot of little utility functions. This is where all |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
185 | of those little dialog windows are created. Things like the warn dialog |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
186 | and the add buddy dialog are here. Not all of the dialogs in gaim are in |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
187 | this file, though. But most of them are. This is also where do_import |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
188 | is housed, to import buddy lists. (The actual buddy list parsing code |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
189 | is in util.c for winaim lists and toc.c for gaim's own lists.) |
| 639 | 190 | |
| 191 | gaimrc.c: | |
| 192 | This controls everything about the .gaimrc file. There's not really much | |
| 193 | to say about it; this is probably one of the better designed and easier | |
| 194 | to follow files in gaim. The important functions are towards the bottom. | |
| 195 | ||
|
1558
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
196 | gtkimhtml.c: |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
197 | This is gaim's HTML widget. It replaced the old widget, GtkHtml (which |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
198 | was different than GNOME's GtkHTML). It's self-contained (it doesn't |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
199 | use any of gaim's code) and is actually a separate project from gaim |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
200 | (but is maintained by Eric). |
| 639 | 201 | |
|
1558
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
202 | gtkspell.c: |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
203 | This controls spell checking. It's not a widget per se but it does have |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
204 | some influence over the GtkText widget. It's a separate project from |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
205 | gaim; if you have a patch for this file send it to the author (the |
|
e40c85ff32c0
[gaim-migrate @ 1568]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1237
diff
changeset
|
206 | contact info is in the file). |
| 639 | 207 | |
| 208 | gtkticker.c: | |
| 209 | Syd, our resident GTK God, wrote a GtkWidget, GtkTicker. This is that | |
| 210 | widget. It's cool, and it's tiny. | |
| 211 | ||
| 212 | html.c: | |
| 213 | Don't ask my why this is called html.c. Most of it is just grab_url, | |
| 214 | which does like the name says; it downloads a URL to show in the | |
| 215 | GtkHTML widget. http.c would be a more appropriate name, but that's OK. | |
| 216 | ||
| 217 | idle.c: | |
|
1038
850b893e1ac9
[gaim-migrate @ 1048]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
980
diff
changeset
|
218 | This file used to be entirely #if 0'd out of existance. However, thanks |
|
850b893e1ac9
[gaim-migrate @ 1048]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
980
diff
changeset
|
219 | to some very generous people who submitted patches, this takes care of |
|
850b893e1ac9
[gaim-migrate @ 1048]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
980
diff
changeset
|
220 | reporting idle time (imagine that). It's a pretty straight-forward file. |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
221 | This also takes care of the auto-away stuff. |
| 639 | 222 | |
|
979
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
223 | multi.c: |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
224 | This is the file that tries to take care of most of the major issues |
|
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
225 | with multiple connections. The best function in here by far is the |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
226 | account_editor(). auto_login() is also in here (I'm just reading multi.h |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
227 | now...). account_editor is really the only function that the UI needs |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
228 | to be concerned with. |
|
979
f9bea7bfe1b0
[gaim-migrate @ 989]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
960
diff
changeset
|
229 | |
| 639 | 230 | oscar.c: |
| 231 | One big hack of copied code. This is supposed to be the libfaim tie-in | |
| 232 | in gaim. Most of it is just copied straight from faimtest, the small | |
| 233 | program that comes with libfaim. I'm not even sure how half of it works, | |
| 234 | if that makes you feel any better. | |
| 235 | ||
| 236 | perl.c: | |
| 237 | This was basically copied straight from X-Chat through the power of | |
| 238 | the GPL. Perl is the biggest, most confusing piece of C code I've ever | |
| 239 | seen in my life (and keep in mind I'm a gaim hacker). I have a basic | |
| 240 | idea of what's going on in it, but I couldn't tell you exactly. The | |
| 241 | top half sets up perl and tells it what's going on and the bottom half | |
| 242 | implements the AIM module. | |
| 243 | ||
| 244 | plugins.c: | |
| 245 | This is the "plugin plug", as the file states. This file is probably | |
| 246 | the only file in all of gaim that at the top has all of the functions | |
| 247 | and global and static variables named out for you. It makes reading | |
| 248 | it a little easier, but not by much. A lot of the code in here deals | |
| 249 | with the plugin window rather than the plugins themselves. | |
| 250 | ||
| 251 | prefs.c: | |
| 252 | The important function in here is build_prefs, but the most useful | |
| 253 | function is gaim_button. build_prefs draws the window, and calls | |
| 254 | gaim_button probably 30 or 40 times. (I don't really wanna run grep | |
| 255 | | wc to count.) This is where you add the toggle button for gaim | |
| 256 | preferences. It's very simple, and if you look at a couple of the | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
257 | calls to gaim_button you'll figure it out right away. The new prefs |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
258 | window uses a CList instead of a Notebook, and there's a pretty bad |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
259 | hack to get it to work. I won't tell you what though. |
| 639 | 260 | |
|
1038
850b893e1ac9
[gaim-migrate @ 1048]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
980
diff
changeset
|
261 | prpl.c: |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
262 | This file is what lets gaim dynamically load protocols, sort of. All |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
263 | of the actual dlopen(), dlsym() stuff is in plugins.c. But this |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
264 | contains all of the functions that the protocol plugin needs to call, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
265 | and manages all of the protocols. It's a pretty simple file actually. |
|
1038
850b893e1ac9
[gaim-migrate @ 1048]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
980
diff
changeset
|
266 | |
| 639 | 267 | proxy.c: |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
268 | Adam (of libfaim glory) got bored one day and rewrote this file, so |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
269 | now everything actually works. The main function is proxy_connect, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
270 | which figures out which proxy you want to use (if you want to use one |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
271 | at all) and passes off the data to the appropriate function. This file |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
272 | should be pretty straight-forward. |
| 639 | 273 | |
| 274 | rvous.c: | |
| 275 | This was originally going to be the stuff for all of the Buddy Icon | |
| 276 | and Voice Chat stuff, but I got really sick of protocol hacking really | |
| 277 | quick. Now it only houses the file transfer stuff, which only works | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
278 | for TOC. ("Works" being a very subjective statement. This file needs |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
279 | to be rewritten.) |
| 639 | 280 | |
| 281 | server.c: | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
282 | This is where all of the differentiation between the different protocols |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
283 | is done. Nearly everything that's network related goes through here |
| 639 | 284 | at one point or another. This has good things like serv_send_im and |
| 285 | serv_got_update. Most of it should be pretty self-explanatory. | |
| 286 | ||
| 287 | sound.c: | |
|
1038
850b893e1ac9
[gaim-migrate @ 1048]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
980
diff
changeset
|
288 | The main function in this file is play_sound, which plays one of 8 |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
289 | (maybe 9?) sounds based on preferences. All that the rest of the code |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
290 | should have to do is call play_sound(BUDDY_ARRIVE), for example, and |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
291 | this file will take care of determining if a sound should be played |
|
1038
850b893e1ac9
[gaim-migrate @ 1048]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
980
diff
changeset
|
292 | and which file should be played. |
| 639 | 293 | |
| 294 | ticker.c: | |
| 295 | Syd is just so cool. I really can't get over it. He let me come | |
| 296 | visit him at Netscape one day, and I got to see all of their toys | |
| 297 | (don't worry, I'm under an NDA). Anyway, this file is for the buddy | |
| 298 | ticker. This is also a damn cool file because it's got all of the | |
| 299 | functions that you'd want right up at the top. Someday I want to be | |
| 300 | as cool as Syd. | |
| 301 | ||
| 302 | toc.c: | |
| 303 | This handles everything TOC-related, including parsing gaim's buddy | |
| 304 | list. Most of this file is toc_callback, which parses the incoming | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
305 | information from the server. I really don't like TOC though. (I've spent |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
306 | waaayyyy too much time with TOC. I rewrote the signon process for this |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
307 | file at one point, so that read was only called when data was pending. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
308 | Since then the TOC server has been blocking my IP (probably my own |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
309 | stupid fault, sending bad strings or some such).) |
| 639 | 310 | |
| 311 | util.c: | |
| 312 | There's not really a lot of cohesion to this file; it's just a lot of | |
| 313 | stuff that happened to be thrown into it for no apparent reason. None | |
| 314 | of it is particularly tasty; it's all just utility functions. Just | |
| 315 | like the name says. | |
| 316 | ||
|
1653
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
317 | PRPL sources: |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
318 | ------------- |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
319 | |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
320 | plugins/yay: Yahoo |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
321 | All of the files in here were written by Eric. All of the .c and .h |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
322 | files except yay.c are part of a library that Eric wrote, libyay. |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
323 | yay.c is what interacts with gaim. |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
324 | |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
325 | plugins/icq: ICQ (UDP v5) |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
326 | All of the .c and .h files in here, with the exception of gaim_icq.c, |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
327 | are part of ICQLib, by Bill Soudan and others. gaim_icq.c is what |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
328 | interacts with gaim, and Eric wrote it. ICQLib is a fairly complete |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
329 | implementation of the ICQ protocol, so if you want to add a new |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
330 | feature you're probably going to be adding it to gaim_icq.c and not |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
331 | to ICQLib. |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
332 | |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
333 | plugins/msn: MSN |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
334 | Rob wrote msn.c, and md5.c is a standard file. MSN doesn't use its |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
335 | own library; all of the networking code is included inside of msn.c. |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
336 | |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
337 | plugins/irc.c: IRC |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
338 | Rob wrote irc.c, and since it is only one file it stands by itself. |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
339 | All of the networking code is contained inside this file, as well as |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
340 | the parts that interact with gaim. |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
341 | |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
342 | plugins/jabber: Jabber |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
343 | jabber.c was written by Adam Fritzler (the guy that wrote libfaim), |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
344 | and is maintained by Eric. The other .c and .h files belong to |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
345 | libxode and libjabber, which were written by the Jabber developers. |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
346 | |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
347 | plugins/napster.c: Napster |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
348 | Rob wrote napster.c, and since it is only one file it stands by |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
349 | itself. |
|
955d62bcec11
[gaim-migrate @ 1663]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1619
diff
changeset
|
350 | |
|
1710
4793276c2510
[gaim-migrate @ 1720]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1653
diff
changeset
|
351 | plugins/zephyr: Zephyr |
|
4793276c2510
[gaim-migrate @ 1720]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1653
diff
changeset
|
352 | zephyr.c is the only file in this directory by Eric; all the other |
|
4793276c2510
[gaim-migrate @ 1720]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1653
diff
changeset
|
353 | files are part of the Zephyr library from MIT. |
|
4793276c2510
[gaim-migrate @ 1720]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1653
diff
changeset
|
354 | |
| 639 | 355 | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
356 | HOW THE BUDDY LIST WORKS |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
357 | ======================== |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
358 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
359 | The buddy list is a pain in the ass. Let me start off by saying that. The |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
360 | most difficult part about getting gaim to do multiple connections was |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
361 | the buddy list. In its current state it's very much like the UI for |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
362 | 0.10.x and earlier, which is what I was aiming for. However, the code |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
363 | is completely different. And not much better. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
364 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
365 | All of the buddy list stuff is in buddy.c, so you'll only have to have |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
366 | that one file open (and possibly gaim.h for the struct definitions). There |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
367 | are two sets of functions: those that deal with the buddy lists, and |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
368 | those that deal with the window. (I say lists because each connection |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
369 | has their own buddy list, independent of the others, even though the UI |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
370 | merges them.) |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
371 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
372 | The buddy list functions work pretty much the same way they did before; |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
373 | except now that each buddy and group belongs to a connection, things |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
374 | like find_buddy take an additional argument, the connection you want to |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
375 | search for the buddy in. Read gaim.h for a good list of them: find_buddy, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
376 | find_group, add_buddy, remove_buddy, remove_group. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
377 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
378 | The window is a lot more fun. There's really only one function that |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
379 | does anything interesting, and that's set_buddy. (There's also things |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
380 | like build_edit_tree, but that's boring.) set_buddy is called by |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
381 | serv_got_update (and should only be called by that function) any time |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
382 | a user signs on, signs off, goes away, comes back, goes idle, etc, etc, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
383 | etc. Various things happen depending on the new state of the buddy. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
384 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
385 | struct buddy has a member, present, which is set to either 0, 1, or |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
386 | 2. You can check if the buddy is online with "if (b->present)". This |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
387 | becomes important. present is set to either 0 or 1 by serv_got_update, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
388 | or is not set at all. When the buddy is passed to set_buddy, if present |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
389 | is 1 then set_buddy plays the BUDDY_ARRIVE sound, and sets present to 2, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
390 | to indicate it has already received notification of arrival. It then |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
391 | does other signin-related stuff: setting the pixmap to the login icon; |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
392 | updating the conversation windows; etc. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
393 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
394 | The most important thing it does though, if a buddy is present, is it |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
395 | checks for the existance of the appropriate group_show and buddy_show for |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
396 | that buddy. Each buddy must belong to a group. group_shows are based on |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
397 | name; there can only be one group_show for each group name. buddy_shows |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
398 | are based both on name and on group_show; there can only be one buddy_show |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
399 | in a group_show for each name. However, there can be two buddy_shows |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
400 | with the same name as long as they have different group_shows. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
401 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
402 | Each buddy_show has a GList of connections that has registered its related |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
403 | buddy as being online. set_buddy makes sure that the connection that it's |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
404 | being passed is part of the connlist for the buddy_show associated with |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
405 | the struct buddy that it's passed (it helps to know your data structures). |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
406 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
407 | If a buddy logs off (b->present == 0), and a buddy_show exists for |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
408 | that buddy, then set_buddy will play the logoff sound, change the icon, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
409 | remove the connection from the connlist for the buddy_show, etc. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
410 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
411 | And that's how that works. For the buddy lists, connections own buddies; |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
412 | for the window, the buddies own the connections. When the buddy_show |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
413 | connlist count drops to zero it disappears from existance. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
414 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
415 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
416 | PLUGINS |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
417 | ======= |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
418 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
419 | OK, so you want to load a plugin. You go through whatever UI (you |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
420 | can read all about the UI in plugins.c or whereever). You finally get |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
421 | to load_plugin, the meat of the plugins stuff (plugins can actually |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
422 | call load_plugin themselves to load other plugins). load_plugin |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
423 | is passed the full path to the plugin you want to load |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
424 | (e.g. /usr/local/lib/gaim/irc.so). |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
425 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
426 | load_plugin does a few things with that filename. The first is to see |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
427 | if you've already loaded that plugin. If you have, load_plugin unloads |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
428 | the one that is currently loaded. You might wonder why; it's because |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
429 | the same plugin can't be loaded twice. If you call g_module_open on a |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
430 | filename twice, both times it will return the same pointer, and both times |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
431 | increment the reference count on the GModule * that it returns. This |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
432 | means you really do have the same plugin twice, which fucks up the |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
433 | callback system to no end. So it's better that you can only have it |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
434 | loaded once at any given time. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
435 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
436 | Now that we're assured that we don't have this particular plugin loaded |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
437 | yet, we better load it. g_module_open, baby. Much more portable than |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
438 | dlopen(). In fact, for Linux it actually is the equivalent of dlopen() |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
439 | (you can read the gmodule source and see for yourself). There's only one |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
440 | quirk. It always logically ORs the options you pass with RTLD_GLOBAL, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
441 | which means that plugins share symbols. I haven't figured out yet if |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
442 | this means just functions or variables too; but in either case make every |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
443 | function and variable in your plugin static except for gaim_plugin_*(), |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
444 | name(), and description(). It's good coding practice anyway. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
445 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
446 | So, assuming we didn't get NULL back from g_module_open, we then make sure |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
447 | it's a valid gaim plugin by looking for and calling gaim_plugin_init, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
448 | courtesy g_module_symbol (g_module_symbol is actually what's portable |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
449 | about gmodule as opposed to dl*; some BSD's require '_' prepended to |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
450 | symbol names and g_module_symbol guarantees we do The Right Thing). |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
451 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
452 | Assuming we've found gaim_plugin_init and it hasn't returned non-NULL |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
453 | to us, we then add it to our list of plugins and go merrily about our way. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
454 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
455 | So when do the callbacks happen?! plugin_event, baby, plugin_event. Any |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
456 | time you want to trigger a plugin event simply call plugin_even with the |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
457 | parameters to be passed to any event handlers and you're set. plugin_event |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
458 | then makes sure that any plugins waiting for the event get passed the |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
459 | arguments properly and passes it on to perl. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
460 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
461 | Speaking of perl. If you really want to know how this works, you're |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
462 | better off reading X-Chat's documentation of it, because it's better |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
463 | than what I could provide. |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
464 | |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
465 | |
|
1063
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
466 | MULTIPLE CONNECTIONS AND PRPLS |
|
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
467 | ============================== |
|
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
468 | |
|
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
469 | OK, let's start with the basics. There are users. Each user is contained |
|
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
470 | in an aim_user struct, and kept track of in the aim_users GList (GSList?). |
|
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
471 | Each aim_user has certain features: a username, a password, and user_info. |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
472 | It also has certain options, and the protocol it uses to sign on (kept |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
473 | as an int which is #define'd in prpl.h). The way the management of the |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
474 | users works is, there will (hopefully) only be one user for a given |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
475 | screenname/ protocol pair (i.e. you may have two user warmenhoven's, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
476 | but they'll both have a different protocol number). |
|
1063
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
477 | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
478 | Now then, there are protocols that gaim knows about. Each protocol is |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
479 | in a prpl struct and kept track of in the protocols GSList. The way the |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
480 | management of the protocols is, there will only ever be one prpl per |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
481 | numeric protocol. Each prpl defines a basic set of functions: login, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
482 | logout, send_im, etc. The prpl is responsible not only for handling |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
483 | these functions, but also for calling the appropriate serv_got functions |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
484 | (e.g. serv_got_update when a buddy comes online/goes offline/goes |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
485 | idle/etc). It handles each of these on a per-connection basis. |
|
1063
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
486 | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
487 | So why's it called a PRPL? It stands for PRotocol PLugin. That means |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
488 | that it's possible to dynamically add new protocols to gaim. However, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
489 | all protocols must be implemented the same way: by using a prpl struct |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
490 | and being loaded, regardless of whether they are static or dynamic. |
|
1063
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
491 | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
492 | Here's how struct gaim_connection fits into all of this. At some point |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
493 | the User (capitalized to indicate a person and not a name) will try to |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
494 | sign on one of Their users. serv_login is then called for that user. It |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
495 | searches for the prpl that is assigned to that user, and calls that prpl's |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
496 | login function, passing it the aim_user struct that is attempting to sign |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
497 | on. The prpl is then responsible for seeing that the gaim_connection |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
498 | is created (by calling new_gaim_connection), and registering it as |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
499 | being online (by calling account_online and passing it the aim_user and |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
500 | gaim_connection structs). At that point, the aim_user and gaim_connection |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
501 | structs have pointers to each other, and the gaim_connection struct has |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
502 | a pointer to the prpl struct that it is using. The gaim_connections are |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
503 | stored in the connections GSList. The way connection management works is, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
504 | there will always only be one gaim_connection per user, and the prpl that |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
505 | the gaim_connection uses will be constant for the gaim_connection's life. |
|
1063
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
506 | |
|
1237
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
507 | So at certain points the User is going to want to do certain things, |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
508 | like send a message. They must send the message on a connection. So the UI |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
509 | figures out which gaim_connection the User want to send a message on (for |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
510 | our example), and calls serv_send_im, telling it which gaim_connection to |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
511 | use, and the necessary information (who to send it to, etc). The serv_ |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
512 | function then calls the handler of the prpl of the connection for that |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
513 | event (that was way too many prepositions). OK, each prpl has a send_im |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
514 | function. Each connection has a prpl. so you call gc->prpl->send_im and |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
515 | pass it the connection and all the necessary info. And that's how things |
|
5074b5b953da
[gaim-migrate @ 1247]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1099
diff
changeset
|
516 | get done. |
|
1063
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
517 | |
|
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
518 | I hope some of that made sense. Looking back at it it makes absolutely no |
|
f766a178ee59
[gaim-migrate @ 1073]
Eric Warmenhoven <warmenhoven@yahoo.com>
parents:
1038
diff
changeset
|
519 | sense to me. Thank god I wrote the code; otherwise I'm sure I'd be lost. |