intel/compute-runtime
1
0
Fork 0
mirror of https://github.com/intel/compute-runtime.git synced 2025-12-18 22:08:53 +08:00
Code Issues Packages Projects Releases Wiki Activity

GUIDELINES: add an explicit-this rule

Browse Source 
Related-To: NEO-6789
Signed-off-by: Maciej Bielski <maciej.bielski@intel.com>
This commit is contained in:
Maciej Bielski
2022-03-15 15:53:55 +01:00
committed by Compute-Runtime-Automation
parent cd95572443
commit b7f516982b
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
GUIDELINES.md
View File

@@ -1,6 +1,6 @@
<!---
Copyright (C) 2018-2021 Intel Corporation
Copyright (C) 2018-2022 Intel Corporation
SPDX-License-Identifier: MIT
@@ -18,6 +18,7 @@ File to cover guidelines for NEO project.
* avoid includes in headers unless absolutely necessary
* use of exceptions in driver code needs strong justification
* prefer static create methods returning std::unique_ptr instead of throwing from constructor
* inside methods, use an explicit `this->` pointer for refering to non-static class members
# Naming conventions
@@ -62,4 +63,4 @@ a general rule test shouldn't be longer then 1ms in Debug driver.
* Do not use `TODO`s in the code
* Use `UNRECOVERABLE_IF` and `DEBUG_BREAK_IF` instead of `asserts`:
* Use `UNRECOVERABLE_IF` when a failure is found and driver cannot proceed with normal execution. `UNRECOVERABLE_IF` is implemented in Release and Debug builds.
* Use `DEBUG_BREAK_IF` when a failure can be handled gracefully by the driver and it can continue with normal execution. `DEBUG_BREAK_IF` is only implemented in Debug builds.
* Use `DEBUG_BREAK_IF` when a failure can be handled gracefully by the driver and it can continue with normal execution. `DEBUG_BREAK_IF` is only implemented in Debug builds.