SolidWallOfCode commented on code in PR #10724:
URL: https://github.com/apache/trafficserver/pull/10724#discussion_r1381986394
##########
cmake/AutoOptionHelpers.cmake:
##########
@@ -21,17 +21,20 @@ set(GLOBAL_AUTO_OPTION_VARS "")
function(_REGISTER_AUTO_OPTION _NAME _FEATURE_VAR _DESCRIPTION _DEFAULT)
add_custom_target(${_NAME}_target)
- set_target_properties(${_NAME}_target
- PROPERTIES
- AUTO_OPTION_FEATURE_VAR ${_FEATURE_VAR}
- )
+ set_target_properties(${_NAME}_target PROPERTIES AUTO_OPTION_FEATURE_VAR
${_FEATURE_VAR})
Review Comment:
What's with the formatting? This was changed to be on one line, while line
29 was changed to be split.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
