1 Patch guidelines
This section lists some guidelines for writing a good patch which is
more likely to be accepted.
Any new features or large scale work should first be discussed on the
evolution-hackers list first. This will ensure the idea fits in the
direction we wish to take Evolution, and also that the effort is not
duplicated. See section 3 for details on the mailing lists.
1.1 Patch basics
o The patch should apply cleanly at the time it is made.
o It must compile once applied.
o It must not generate any more compile time warnings than were
already there. This may be platform dependent so simply do your
best.
o It must conform to C89/C90 (ANSI/ISO C), and build with gcc using
the default compile flags.
The primary trap is that in C99 you may define variables anywhere in
the code, in C89 they must be declared in a declaration block which
follows any block start '{'.
If you wish to ensure the code is C89, try the following.
From the gcc manual page:
"To select
this standard in GCC, use one of the options `-ansi', `-std=c89' or
`-std=iso9899:1990'; to obtain all the diagnostics required by the
standard, you should also specify `-pedantic'" ...
You may actually have to use '-std=gnu89' if libraries have taken
advantage of gcc extensions and where not compiled similarly, as the
above options will disable all gnu extensions.
[FIXME: Add the same option for Forte here]
o It should not add any extra debug printing by default, unless the
patch is specifically to add extra debug printing.
o It should not use any gcc extensions, except where they are properly
checked for and not used with other compilers. glib provides some
of these features as portable macros and should be used when they
cover the required functionality.
o It must include ChangeLog entries in the appropriate ChangeLog for
the file modified. Use emacs, C-4-a will start a properly formatted
ChangeLog entry in the correct ChangeLog file automatically.
o If it is from a bug report, it must reference the bug number, and if
it isn't in the gnome bugzilla, it must reference the bug system from
whence it came.
1.1 GUI changes
If the change requires non-trivial user interface changes, then they
will have to be discussed and approved on the evolution-hackers list
first. This is highly recommended before embarking on any UI work, or
large scale work in general. The Gnome HIG document is the place to
start on any UI changes or additions.
1.2 Translated string changes
Any changes to translated strings in a stable release must be
discussed on the hackers list (see section 3), and/or as part of the
patch submission. There must be very good reasons for changing the
strings in this case.
1.3 Coding style
Generally the coding style employed matches the "Linux Kernel" style,
that is, basically K&R style indenting with 8 space tabs. Tabs should
be used rather than space characters. Reformatting of otherwise
unchanged code is not acceptable. Editors should have any automatic
reformatting features disabled.
K&R style indenting puts braces on the same line. The opening
parenthesis of a function call or conditional statement should be on
the same line as the function. "else" "} else" and "} else {" must
always occur on lines by themselves.
A single blank line should follow {} blocks (if not immediately
followed by the close of another block), and conditional statements,
and be used to separate logical groups of statements in the same
block.
A single blank line only should separate functions, and other
structures at the top level of the file (i.e. outside functions). The
same rule applies to variable declarations at the start of a block.
An example of the most-developer-preferred formatting:
TheType
the_function (int frank)
{
int a = 1;
if (a == frank) {
a = foo (a);
} else {
do {
a = bob (frank) + a;
} until (a == frank);
frank = a;
}
return (TheType) a;
}
Where there are slight stylistic differences, the style in the
surrounding code should be followed.
1.3.1 Object casts
You can either use C style casts, or Gtk style casts. Note that Gtk
style casts can add significant execution overhead, which is not
adding any extra checking. e.g. if arguments have already been
type-checked by preconditions. Putting a space between a cast and a
variable is optional, but preferred by most of the developers.
1.3.2 Preconditions
External api entry points should have preconditions (g_return_if_fail,
etc), although their use varies from case to case. Internal entry
points and/or when you are guaranteed the type has already been
checked, are unecessary. Object initialisation and other virtual
method invocations are considered internal entry points.
1.3.3 Line lengths
Do not expend effort and resort to unreadable formatting merely to fit
any long lines into 80 column widths. We use 8 space tabs, and
because of the lack of namespacing other than extending the function
name, many of the function and type names are too long for this to be
practical. We now all uses high resolution displays, and not
circa-80's VT100 terminals!
On the other hand, lines should generally not exceed 100 characters,
and absolutely not exceed 160 characters. If your tab nesting is too
deep you probably have a poor design that needs rethinking.
1.4 Design
This is a tricky issue to document, but the design of new code should
`fit' with the existing design of the relevent module. It should at
the very least, be no worse.
Code should not cross existing abstraction boundaries or attempt
to remove or work around them, if required the existing design may
need adjustment.
Type and method names should follow the existing practice in the
surrounding code. Method arguments should follow the same order as
related methods, and should use the same names for matching
parameters.
Per file, static class globals are ok, true globals may be ok, but
should be used sparingly. Use 'i' for a loop variable, if that's all
it is, don't use 'the_current_index'. etc.
If in doubt, ask on the lists.
2. Patch submission guidelines
This section outlines procedures that should be followed when
submitting patches for Evolution.
The patch must simply be attached to an appropriate, open bug on
bugzilla.gnome.org.
For discussion of the patch, or to expediate processing of the patch,
an email may be sent to the evolution-patches list. See the mailing
lists section for more information. You may attach patches when
sending to this list for discussion.
Any non-trival patches (patches of more than 1 or 2 changed lines in
more than 5 isolated locations) also require copyright assignment.
See http://developer.ximian.com/projects/evolution/copyright.html for
details.
If you follow the guidelines listed here, you should generally expect
a response within 2 working days. If you re-send the same patch
repeatedly, you will more likely receive less attention. Do not
re-send the same patch repeatedly.
2.1 Subject Lines
If the patch addresses a specific bug in bugzilla.gnome.org, then the
bug number must be included in the subject line, preferably near the
beginning of the subject line. A concise summary of the bug(s) being
addressed, should be the remainder of the subject.
It is unnecessary to add "[PATCH]", "patch" or similar to the subject
line, unless it is being cross-posted to other non-patch lists.
It is absolutely unnecessary to add "please consider", "please review",
or "seeking review", or similar, to the subject line. Please do not do
this.
Where the patch does not address a specific bug number, then the subject
line should simply be a concise summary of the problem/feature it
addresses.
In all cases the subject line should include the module(s) to which the
patch applies, and would generally match the component on the bug or
the top-level module directory (e.g. camel, mail, addressbook, use 'all'
for more than 3 or 4 modules).
2.2 Message Body
Patches should be attached as attachments, preferably as a single
diff, when possible, and the changes are related. The diff must be in
unified diff format, "-up" is a suitable argument to give to "cvs
diff" (-p may be dropped if not supported by your diff). If you have
added files, then -N should also be used, but if you are using cvs,
"cvs add" is needed, and requires write access to the repository.
If the patch does not address a specific bug, then the patch email
should describe which feature or problem it addresses. If it does
address a specific bug, then further explanation beyond the bug
commentary is optional, although often convenient.
It would also be helpful to summarise the module to which it applies
in the message body.
In all cases you should include which branch, or branches, the patch
is intended to apply to. If this is not given it will be assumed to
be the trunk (HEAD), and such patches will and must not be applied to
any stable branch without further approval.
2.3 Stable branches
Generally, any patch to the stable branch from non-core developers
must address a specific bug in bugzilla.gnome.org. The patch should
also be attached to the bug in question. The patch must not be
applied until reviewed.
3 Mailing lists
3.1 Evolution Hackers
If you wish to discuss patches before they are submitted, or ideas
before you start to work on them, do it on the evolution-hackers list,
which may be subscribed and viewed at
`http://lists.ximian.com/mailman/listinfo/evolution-hackers'.
This is a low-volume list (5-10 posts per day on average).
Some patches may be discussed here to get a wider audience, although
once a patch has been made it should generally be discussed on
evolution-patches. Large posts are blocked, so they should be sent to
the patches list intsead, or reference resources elsewhere.
Feature requests, bug reports, and other user related discussions,
without the intention to write code to address them, will be ignored.
3.2 Evolution Patches
The patch discussion list evolution-patches may be subscribed and
viewed at
`http://lists.ximian.com/mailman/listinfo/evolution-patches'. Once a
patch has been written, it may be submitted here for discussion, as
well as final approval.
Patches may be sent to this list as attachments for discussion.
Any non-patch related postings to this list will be ignored.
6| -40/+236 |
| * | Revert my last patch. | Srinivasa Ragavan | 2008-01-20 | 2 | -1/+5 |
| * | Take off the cursor/focus. Now we have a nice cairo rendered message list | Srinivasa Ragavan | 2008-01-20 | 2 | -1/+6 |
| * | ** Fix for bug #507564 | Djihed Afifi | 2008-01-15 | 2 | -18/+64 |
| * | Patch from David Turner <cillian64@googlemail.com> ** Partial fix for bug #4... | Suman Manjunath | 2008-01-14 | 2 | -2/+9 |
| * | ** Fix for bug #488213 | Milan Crha | 2008-01-12 | 2 | -1/+8 |
| * | ** Fix for bug #448441 | Milan Crha | 2008-01-10 | 3 | -7/+71 |
| * | ** Fix for bug #339813 | Milan Crha | 2008-01-07 | 2 | -0/+8 |
| * | Patch from Michael Monreal <michael.monreal@gmx.net>: Fix for bug #492188 (U... | Suman Manjunath | 2008-01-07 | 2 | -1/+10 |
| * | Patch from Christian Krause <chkr@plauener.de>: Fix for bug #506772 (Not-NUL... | Suman Manjunath | 2008-01-07 | 2 | -1/+8 |
| * | ** Fixes bug 474651 | Tobias Mueller | 2007-12-14 | 1 | -1/+1 |
| * | ** Fix for bug #501969 Passwords should not be forgotten for all errors | Sankar P | 2007-12-11 | 2 | -1/+11 |
| * | ** Fixes bug 474651 | Tobias Mueller | 2007-12-10 | 1 | -0/+1 |
| * | Patch from David Turner <cillian64@googlemail.com>: Fix for bug #468277 (mis... | Suman Manjunath | 2007-12-07 | 2 | -1/+8 |
| * | Corrected misspelled instances of vCard in: | Suman Manjunath | 2007-12-06 | 6 | -11/+25 |
| * | ** Fixes bug #489652 | Matthew Barnes | 2007-11-29 | 9 | -1638/+1720 |
| * | ** Fix for bug #495711 | Milan Crha | 2007-11-23 | 3 | -5/+23 |
| * | ** Fix for bug #495875 | Gilles Dartiguelongue | 2007-11-18 | 2 | -5/+14 |
| * | ** Remove trailing whitespace from source code. | Matthew Barnes | 2007-11-15 | 62 | -814/+814 |
| * | ** Remove dead files from source control. The dates below indicate when | Matthew Barnes | 2007-11-03 | 5 | -372/+11 |
| * | Remove debugging output. | Priit Laes | 2007-11-01 | 1 | -2/+0 |
| * | ** Fix for bug #484064 | Michael Monreal | 2007-10-31 | 3 | -4/+13 |
| * | Remove unneeded include. | Kjartan Maraas | 2007-10-31 | 2 | -1/+5 |
| * | Zero out the GnomePilotRecord struct to silence a compiler warning. | Matthew Barnes | 2007-10-26 | 2 | -0/+7 |
| * | ** Fixes bug #489661 | Matthew Barnes | 2007-10-25 | 10 | -1196/+17 |
| * | Warning fixes: - mixing code and declarations - NULL vs 0 vs FALSE/TRUE - | Kjartan Maraas | 2007-10-24 | 17 | -47/+83 |
| * | Patch by Michael Monreal <michael.monreal@gmx.net> | Tobias Mueller | 2007-10-23 | 2 | -1/+9 |
| * | ** Fixes part of bug #417999 | Matthew Barnes | 2007-10-23 | 6 | -53/+83 |
| * | Patch by Michael Monreal <michael.monreal@gmx.net> | Tobias Mueller | 2007-10-19 | 2 | -2/+10 |
| * | ** Fixes bug #437579 | Matthew Barnes | 2007-10-10 | 5 | -11/+26 |
| * | ** Fixes bug #477045 | Matthew Barnes | 2007-10-04 | 2 | -1/+8 |
| * | 2007-10-04 mcrha Fix for bug #331421 | Milan Crha | 2007-10-04 | 2 | -0/+12 |
| * | ** Fixes bug #469657 | Matthew Barnes | 2007-10-02 | 2 | -8/+11 |
| * | 2007-10-01 mcrha Fix for bug #337539 | Milan Crha | 2007-10-01 | 2 | -2/+12 |
| * | ** Fixes part of bug #474000 | Matthew Barnes | 2007-09-28 | 2 | -92/+15 |
| * | Fix for bug #461195 from Hiroyuki Ikezoe. | Matthew Barnes | 2007-09-28 | 11 | -36/+35 |
| * | ** Fixes bug #477045 | Matthew Barnes | 2007-09-28 | 8 | -44/+55 |
| * | 2007-09-21 mcrha Fix for bug #411125 | Milan Crha | 2007-09-21 | 3 | -38/+47 |
| * | Remove the --enable-file-chooser configure option. | Matthew Barnes | 2007-09-14 | 3 | -83/+8 |
| * | 2007-09-07 mcrha Fix for bug #473903 | Milan Crha | 2007-09-07 | 2 | -1/+8 |
| * | Fix for a build break. | Srinivasa Ragavan | 2007-09-03 | 2 | -1/+6 |
| * | ** Fix for bug #471791 (Move away from assert to g_ret) | Srinivasa Ragavan | 2007-09-03 | 8 | -15/+27 |
| * | Update FSF address in header comments (#469886). Patch from Tobias | Matthew Barnes | 2007-09-02 | 71 | -125/+125 |
| * | Committed on behalf of Ushveen Kaur <kushveen@novell.com> | Johnny Jacob | 2007-08-27 | 3 | -3/+15 |
| * | 2007-08-24 Suman Manjunath Fix for bug #440328 | Milan Crha | 2007-08-24 | 3 | -2/+11 |
| * | ** Fix for bug #353462 | Anand V M | 2007-08-23 | 2 | -2/+13 |
| * | 2007-08-16 mcrha Fix for bug #412732 | Milan Crha | 2007-08-16 | 2 | -0/+10 |
| * | 2007-08-06 mcrha Partial fix for bug #313221 | Milan Crha | 2007-08-06 | 2 | -6/+25 |
| * | 2007-08-06 mcrha Partial fix for bug #427469 | Milan Crha | 2007-08-06 | 2 | -3/+10 |
| * | ** Fix for bug #455799 | Hiroyuki Ikezoe | 2007-08-04 | 12 | -107/+0 |
| * | ** Fix for bug #462007 | Hiroyuki Ikezoe | 2007-08-02 | 2 | -2/+13 |
| * | ** Fix for bug #462002 | Hiroyuki Ikezoe | 2007-07-31 | 2 | -1/+14 |
| * | Use g_strdup() instead of g_strdup_printf() for duplication of a string | Hiroyuki Ikezoe | 2007-07-27 | 2 | -3/+8 |
| * | Plugged memory leaks. | Hiroyuki Ikezoe | 2007-07-26 | 4 | -0/+11 |
| * | Plugged memory leak. | Hiroyuki Ikezoe | 2007-07-26 | 2 | -0/+5 |
| * | Minor typos in strings. | Claude Paroz | 2007-07-21 | 2 | -1/+5 |
| * | Fix the date. See comments in bug #353752. | Hiroyuki Ikezoe | 2007-07-12 | 1 | -1/+1 |
| * | Fix for bug #353752 | Srinivasa Ragavan | 2007-07-09 | 2 | -1/+8 |
| * | ** Fix for bug #448568 | Gilles Dartiguelongue | 2007-07-07 | 3 | -1428/+1030 |
| * | ** Fix for bug #337787 | Hiroyuki Ikezoe | 2007-07-06 | 2 | -2/+35 |
| * | make the path to perl configurable and add addressbook/tools/csv2vcard to | Laszlo (Laca) Peter | 2007-06-20 | 4 | -2/+13 |
| * | ** Fix for bug #448223 from Gilles Dartiguelongue | Srinivasa Ragavan | 2007-06-18 | 2 | -5/+8 |
| * | Fix for bug #448223 from Gilles Dartiguelongue | Srinivasa Ragavan | 2007-06-18 | 2 | -27/+8 |
| * | ** Fix for bug #445812 from Gilles Dartiguelongue | Srinivasa Ragavan | 2007-06-18 | 2 | -2/+8 |
| * | ** Fix for bug #325882 | Srinivasa Ragavan | 2007-06-18 | 5 | -4/+14 |
| * | Fixes #444248. | Wang Xin | 2007-06-12 | 2 | -1/+12 |
| * | Reverted automake 1.6 breakage. Fixes bug 444747. Patch by Patch by Daniel | Nickolay V. Shmyrev | 2007-06-06 | 2 | -3/+10 |
| * | ** Fix for bug #386503 from Matthew Barnes | Srinivasa Ragavan | 2007-06-03 | 2 | -6/+11 |
| * | Fix for version removal from installed files. | Srinivasa Ragavan | 2007-06-03 | 4 | -16/+22 |
| * | Removal of version information from installed files | Srinivasa Ragavan | 2007-06-03 | 1 | -0/+0 |
| * | Fix compiler warnings in addressbook folder. (#440272) | Matthew Barnes | 2007-06-03 | 15 | -101/+140 |
| * | Fix for bug #333858 | Srinivasa Ragavan | 2007-05-30 | 3 | -2/+21 |
| * | Don't assume the relative URI is non-NULL (#427232). | Matthew Barnes | 2007-05-26 | 3 | -2/+18 |
| * | Fix for bug #417797 and #424055 | Srinivasa Ragavan | 2007-05-14 | 3 | -4/+21 |
| * | Fix for bug #404239 #358250 | Srinivasa Ragavan | 2007-05-14 | 4 | -3/+17 |
| * | ** Fixes part of bug #337616 | Matthew Barnes | 2007-05-13 | 2 | -0/+8 |
| * | Add Tango icons to preference window. | Srinivasa Ragavan | 2007-05-11 | 2 | -2/+7 |
| * | *tools/evolution-addressbook-export-list-cards.c: Fixes 394932. | Wang Xin | 2007-05-08 | 2 | -2/+9 |
| * | Don't include e-contact-print-envelope.h. | Matthew Barnes | 2007-04-24 | 4 | -3/+7 |
| * | Clean up printing in Evolution (bug #426816) | Matthew Barnes | 2007-04-20 | 11 | -911/+496 |
| * | Massive code cleanup (bug #429422) | Matthew Barnes | 2007-04-20 | 11 | -78/+147 |
| * | Committing Contact Merging patch from Ebby Wiselyn | Srinivasa Ragavan | 2007-04-18 | 3 | -6/+309 |
| * | ** Fix for bug #426743. Corrected typo "asychronous". | Elizabeth Greene | 2007-04-10 | 2 | -1/+7 |
| * | change "E-Mail" to "Email". Fixes bug #404233. | Andre Klapper | 2007-04-10 | 2 | -2/+7 |
| * | ** Fix for bug #426487 | simon.zheng | 2007-04-09 | 2 | -7/+15 |
| * | ** Fix for bug #426829. | simon.zheng | 2007-04-09 | 3 | -3/+13 |
| * | Call gettext() on the EConfigItem labels. (#406933) Patch from Jeff Cai. | Matthew Barnes | 2007-04-05 | 2 | -1/+10 |
| * | Don't mix declarations and code (#405495). | Matthew Barnes | 2007-03-27 | 8 | -45/+51 |
| * | ** Fixes bug #419524 | Matthew Barnes | 2007-03-21 | 24 | -23/+29 |
| * | ** Fixes bug #360240 | Matthew Barnes | 2007-03-17 | 3 | -9/+10 |
| * | Bypass the body of these functions on non-G_OS_UNIX for now as they use | Tor Lillqvist | 2007-03-01 | 3 | -0/+16 |
| * | ** Fix for bug #232065 | Srinivasa Ragavan | 2007-02-26 | 2 | -1/+50 |
| * | Fix critical warnings from typos in the glade file. Closes bug #397893. | Kjartan Maraas | 2007-02-12 | 2 | -3/+9 |
| * | Print migration updates from Ebby Wiselyn. | Srinivasa Ragavan | 2007-02-12 | 4 | -40/+84 |
| * | ** Fix for bug #332908 | Srinivasa Ragavan | 2007-02-09 | 2 | -19/+30 |
| * | ** Fixes bug #400726 | Matthew Barnes | 2007-02-09 | 2 | -0/+7 |
| * | Removed the obsolete function e_contact_print_preview. | Chenthill Palanisamy | 2007-02-08 | 2 | -2/+0 |
| * | ** Committing Ebby Wiselyn's patch for GtkPrint API migration. 2006-01-22 | Srinivasa Ragavan | 2007-01-23 | 5 | -415/+348 |
| * | Fix for 389664 - Adding pilot-link 0.12 compatibility. | Veerapuram Varadhan | 2007-01-08 | 2 | -6/+102 |
| * | Fixes bug #357970 | Matthew Barnes | 2006-12-04 | 4 | -3/+12 |
| * | Fixes #358249 Fixes #358249 | Francisco Javier F. Serrador | 2006-11-11 | 3 | -2/+7 |
| * | Changed the parameter passed to gal_view_minicard_attach. | Devashish Sharma | 2006-10-16 | 4 | -5/+27 |
| * | ** Fixes bug#332908 | Ushveen Kaur | 2006-10-16 | 3 | -9/+20 |
| * | addressbook/importers/evolution-ldif-importer.c | Francisco Javier F. Serrador | 2006-10-15 | 3 | -3/+3 |
| * | Remove duplicate include. Fix some compiler warnings. Cast away a compiler | Kjartan Maraas | 2006-10-01 | 8 | -16/+34 |
| * | ** Fixes bug #261062 | Srinivasa Ragavan | 2006-09-29 | 2 | -0/+10 |
| * | ** Fix for bug #357811 | Srinivasa Ragavan | 2006-09-28 | 2 | -2/+8 |
| * | Fixes bug #351332 | Matthew Barnes | 2006-09-19 | 2 | -0/+8 |
| * | ** Fixes bug #352353 | Ushveen Kaur | 2006-09-12 | 2 | -0/+8 |
| * | Reverting last commit, head and stable are not branched so cannot commit stri... | Devashish Sharma | 2006-09-07 | 1 | -1/+1 |
| * | Changed the comment "Do you want to overwrite it" to "Do you want to overwrit... | Devashish Sharma | 2006-09-07 | 1 | -1/+1 |
| * | Get rid of this to lose this old directory | Kjartan Maraas | 2006-08-24 | 1 | -7/+0 |
| * | Fixes a lot of search issues. | Srinivasa Ragavan | 2006-08-21 | 2 | -16/+90 |
| * | Clean out this one | Kjartan Maraas | 2006-08-18 | 1 | -15/+0 |
| * | Plug a leak. | Kjartan Maraas | 2006-08-17 | 2 | -0/+6 |
| * | Fixes bug #261239 | Srinivasa Ragavan | 2006-08-17 | 2 | -924/+1638 |
| * | Added a callback to handle contact deletion error and corresponding error | Sushma Rai | 2006-08-07 | 3 | -4/+37 |
| * | Assign proper search item ids: Fix from Johnny | Srinivasa Ragavan | 2006-08-02 | 1 | -0/+5 |
| * | Assign proper search item ids in addressbook search. | Devashish Sharma | 2006-07-27 | 1 | -5/+4 |
| * | Commented out reference to undefined function eab_search_dialog_new(). | Sushma Rai | 2006-07-24 | 2 | -1/+6 |
| * | Added support for gadu gadu im in contact entry. | Devashish Sharma | 2006-07-24 | 4 | -0/+12 |
| * | Assign credits to Mikhail Zabulev. | Harish Krishnaswamy | 2006-07-22 | 1 | -1/+1 |
| * | Reviewed by Harish Krishnaswamy <kharish@novell.com> | Yuri Pankov | 2006-07-22 | 2 | -0/+9 |
| * | ** Fixes bug #347381 | Boby Wang | 2006-07-22 | 2 | -0/+11 |
| * | Autocompletion in contact list email entry. | Devashish Sharma | 2006-07-21 | 2 | -11/+83 |
| * | Fix for Bug 343816 | Devashish Sharma | 2006-07-19 | 3 | -16/+182 |
| * | [addressbook] | Ross Burton | 2006-07-19 | 4 | -15/+27 |
| * | remove "*" from translation. Fixes bug #331147. | Andre Klapper | 2006-07-19 | 6 | -58/+67 |
| * | Fix for bug 332909 | Srinivasa Ragavan | 2006-07-14 | 3 | -48/+66 |
| * | Fixing memory leaks. | Hiroyuki Ikezoe | 2006-07-10 |