diff options
author | Christian Geddes <crgeddes@us.ibm.com> | 2018-10-11 15:46:04 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-10-16 14:12:31 -0500 |
commit | 3e677e6cdf31494f05d6c23828c80d92ba5b2280 (patch) | |
tree | b33bacd91aeb4e07f759a6284da04296b92e01b9 /src/usr/initservice | |
parent | f07e1f6859517dba668056e3e8afd8721bbabb5e (diff) | |
download | talos-hostboot-3e677e6cdf31494f05d6c23828c80d92ba5b2280.tar.gz talos-hostboot-3e677e6cdf31494f05d6c23828c80d92ba5b2280.zip |
Wrap TS_ macros in {} to avoid strange IF statement behavior
The TS_ macros will replace the macro with multiple lines of code.
So if you put a TS_FAIL in an if statement that was not enclosed
in {} you could get strange/unexpected behavior. This addes an extra
layer of protection to ensure that the macro is enclosed in its own
execution block.
Change-Id: I226ccef81095cf8511b30e147e59d1da4904cd5c
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67373
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com>
Reviewed-by: Thi N. Tran <thi@us.ibm.com>
Reviewed-by: Matt Derksen <mderkse1@us.ibm.com>
Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/initservice')
0 files changed, 0 insertions, 0 deletions