github/acrn-hypervisor
1
0
Fork 0
mirror of https://github.com/projectacrn/acrn-hypervisor.git synced 2025-06-21 13:08:42 +00:00
Code Issues Projects Releases Wiki Activity

doc: update coding guidelines

Browse Source 
This patch updates some rules in coding guidelines.

Tracked-On: #861
Signed-off-by: Shiqing Gao <shiqing.gao@intel.com>
This commit is contained in:
Shiqing Gao 2019-03-25 10:00:23 +08:00 committed by David Kinder
parent 868778a68b
commit 558a178841
1 changed files with 47 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

53
doc/developer-guides/coding_guidelines.rst
View File

@ -1575,6 +1575,47 @@ Compliant example::
}
ST-22: 'goto' statement shall not be used
==========================================
Compliant example::
uint32_t showcase(uint32_t param)
{
uint32_t ret;
if (param < 10U) {
ret = 10U;
} else {
ret = param;
/* do something */
}
return ret;
}
.. rst-class:: non-compliant-code
Non-compliant example::
uint32_t showcase(uint32_t param)
{
uint32_t ret;
if (param < 10U) {
ret = 10U;
goto done;
} else {
ret = param;
}
/* do something */
done:
return ret;
}
Expressions
***********
@ -3240,18 +3281,18 @@ NC-07: Function pointer shall be named with suffix 'fn'
Compliant example::
struct firmware_operations firmware_sbl_ops = {
.init = sbl_init_fn,
.get_rsdp = sbl_get_rsdp_fn,
struct firmware_operations {
void (*init_fn)(void);
void *(*get_rsdp_fn)(void);
};
.. rst-class:: non-compliant-code
Non-compliant example::
struct firmware_operations firmware_sbl_ops = {
.init = sbl_init,
.get_rsdp = sbl_get_rsdp,
struct firmware_operations {
void (*init)(void);
void *(*get_rsdp)(void);
};