Style: Difference between revisions

From KolibriOS wiki
Jump to navigation Jump to search
No edit summary
No edit summary
Line 8: Line 8:


== Recommendations ==
== Recommendations ==
The most important thing is to describe you subroutines:
* Variables must have either a comprehensive name or a commentary.
* purpose
* The same is advisable for labels, as well as arrange them in a separate line without indent.
* input data
* When writing a subroutine, it is very important to describe it's purpose, input and output data, spoiled registers:
* output data
* spoiled registers
<syntaxhighlight lang="asm">
<syntaxhighlight lang="asm">
; doing job
; doing job

Revision as of 00:04, 11 February 2016

SVN kernel rules

  • No tab characters allowed.
  • Code labels must be on a separate line. It is not allowed to have a label and a command on the same line. Combinations of label and data in one line is allowed.
  • Lines with commands must start with 8 spaces. A mnemonic is short if it's length is less than 8. Arguments for short mnemonics must start in the column 16. Arguments for long mnemonics must be separated from the mnemonic by exactly one space. Arguments must be separated with a comma and exactly one space after a comma. Arguments continued in the next line must start from the same position as in the first line.
  • Prefixes lock/rep[z|e|nz|ne] are considered as a logical part of the command, so they start at position 8, then after exactly one space follows the main mnemonic and arguments.
  • The special property 'svn:eol-style' must be set to 'native'.
  • All code and text files should be in UTF-8 without BOM.

Recommendations

  • Variables must have either a comprehensive name or a commentary.
  • The same is advisable for labels, as well as arrange them in a separate line without indent.
  • When writing a subroutine, it is very important to describe it's purpose, input and output data, spoiled registers:
; doing job
subroutine:
; in:  eax = some data
;      ebx -> some pointer
; out: eax = result data
; destroys ebx