Mercurial > pidgin
comparison HACKING @ 2166:dbd74f49dabb
[gaim-migrate @ 2176]
updated hacking and todo slightly. sebfrance sent in an updated french translation. thanks :)
committer: Tailor Script <tailor@pidgin.im>
| author | Eric Warmenhoven <eric@warmenhoven.org> |
|---|---|
| date | Sun, 26 Aug 2001 19:18:39 +0000 |
| parents | eb2cadb18479 |
| children | 1c1501df2633 |
comparison
equal
deleted
inserted
replaced
| 2165:34ce165132de | 2166:dbd74f49dabb |
|---|---|
| 31 (don't bother with two trees, or even two copies of the same file). Then | 31 (don't bother with two trees, or even two copies of the same file). Then |
| 32 when you're ready to make your patch, simply run 'cvs diff -u >my.patch' | 32 when you're ready to make your patch, simply run 'cvs diff -u >my.patch' |
| 33 and send it off. | 33 and send it off. |
| 34 | 34 |
| 35 This file was last modified by $Author: warmenhoven $ on | 35 This file was last modified by $Author: warmenhoven $ on |
| 36 $Date: 2001-08-20 16:39:29 -0400 (Mon, 20 Aug 2001) $. | 36 $Date: 2001-08-26 15:18:39 -0400 (Sun, 26 Aug 2001) $. |
| 37 | 37 |
| 38 | 38 |
| 39 CODING STYLE | 39 CODING STYLE |
| 40 ============ | 40 ============ |
| 41 | 41 |
| 184 A massive file with a lot of little utility functions. This is where all | 184 A massive file with a lot of little utility functions. This is where all |
| 185 of those little dialog windows are created. Things like the warn dialog | 185 of those little dialog windows are created. Things like the warn dialog |
| 186 and the add buddy dialog are here. Not all of the dialogs in gaim are in | 186 and the add buddy dialog are here. Not all of the dialogs in gaim are in |
| 187 this file, though. But most of them are. This is also where do_import | 187 this file, though. But most of them are. This is also where do_import |
| 188 is housed, to import buddy lists. (The actual buddy list parsing code | 188 is housed, to import buddy lists. (The actual buddy list parsing code |
| 189 is in util.c for winaim lists and toc.c for gaim's own lists.) | 189 is in util.c for winaim lists and buddy.c for gaim's own lists.) |
| 190 | 190 |
| 191 gaimrc.c: | 191 gaimrc.c: |
| 192 This controls everything about the .gaimrc file. There's not really much | 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 | 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. | 194 to follow files in gaim. The important functions are towards the bottom. |
| 518 pass it the connection and all the necessary info. And that's how things | 518 pass it the connection and all the necessary info. And that's how things |
| 519 get done. | 519 get done. |
| 520 | 520 |
| 521 I hope some of that made sense. Looking back at it it makes absolutely no | 521 I hope some of that made sense. Looking back at it it makes absolutely no |
| 522 sense to me. Thank god I wrote the code; otherwise I'm sure I'd be lost. | 522 sense to me. Thank god I wrote the code; otherwise I'm sure I'd be lost. |
| 523 | |
| 524 There should be absolutely *ZERO* GTK in the PRPLs. PRPLs should *NEVER* | |
| 525 say what the UI *looks* like, only what information needs to be there. | |
| 526 There's currently an effort to get the GTK that is contained in the PRPLs | |
| 527 directory out of there. If you submit a patch that adds GTK to those | |
| 528 directories it's very likely to be refused, unless if I'm in a good mood | |
| 529 and decide to relocate things for you. That's not likely. |
