|
|
(91 intermediate revisions by the same user not shown) |
Line 1: |
Line 1: |
− | == Code Formatting ==
| + | {{warning|Note: The contents of this page has been migrated to GitHub. Please see the [https://github.com/eclipse-cdt/cdt/blob/main/POLICY.md Policy Guide] for current information or page history for historical information}} |
| | | |
− | * It is recommended to use default "Eclipse" code formatting for Java for new code.
| + | [[Category:CDT]] |
− | * It is recommended to preserve formatting of old code when making patches
| + | |
− | | + | |
− | == Eclipse Java Errors/Warnings ==
| + | |
− | | + | |
− | It is strongly recommended for cdt project to override default compiler error/warning and use project specific errors/warnings.
| + | |
− | These errors should be enabled:
| + | |
− | | + | |
− | * Method with a constructor name - Error
| + | |
− | * Assignment has no effect - Error
| + | |
− | * Possible accidental boolean assignment - Error
| + | |
− | * finally does not complete normally - Error
| + | |
− | * Using a char array in string concatenation - Error
| + | |
− | * Null pointer access - Error
| + | |
− | * Potential null pointer access - Warning
| + | |
− | * Unused Import - Error
| + | |
− | | + | |
− | | + | |
− | == Copyright ==
| + | |
− | | + | |
− | Use eclipse copyright header: http://www.eclipse.org/legal/copyrightandlicensenotice.php
| + | |
− | | + | |
− | <pre>
| + | |
− | /*******************************************************************************
| + | |
− | * Copyright (c) {DATE} {INITIAL COPYRIGHT OWNER} {OTHER COPYRIGHT OWNERS}.
| + | |
− | * All rights reserved. This program and the accompanying materials
| + | |
− | * are made available under the terms of the Eclipse Public License v1.0
| + | |
− | * which accompanies this distribution, and is available at
| + | |
− | * http://www.eclipse.org/legal/epl-v10.html
| + | |
− | *
| + | |
− | * Contributors:
| + | |
− | * {INITIAL AUTHOR} - initial API and implementation
| + | |
− | *******************************************************************************/
| + | |
− | </pre>
| + | |
− | | + | |
− | == Contributing the Patch ==
| + | |
− | | + | |
− | * Comment your changes in the code
| + | |
− | * Do not re-format source code which you editing to minimize the patch
| + | |
− | * Submit patch using attachment to a bug report
| + | |
− | * Mark attachment as patch
| + | |
− | * Add a comment to which branch patch has to be applied (head by default)
| + | |
− | * Send letter to mailto:cdt-patch@eclipse.org to notify committers
| + | |
− | | + | |
− | see also [[CDT/contributing]]
| + | |
− | | + | |
− | == Applying the Patch ==
| + | |
− | | + | |
− | * Assign bug to yourself
| + | |
− | * Set target milestone field to release in which patch would be applied, If it is applied in two branches set target milestone to maintenance branch
| + | |
− | * Code inspect the patch, apply and test
| + | |
− | * Commit the patch
| + | |
− | * Add keyword "contributed" to the bug
| + | |
− | * Change bug state to fixed
| + | |
− | * Add a comment about where it was fixed (branches) and related notes
| + | |
− | | + | |
− | == Committing Code ==
| + | |
− | | + | |
− | * API changes have to be discussed in cdt-dev mailing list before commiting
| + | |
− | * When development reaches cycle where release candidates are built, letter in cdt-dev should be sent for every commit you are making
| + | |