[Asterisk-code-review] Fix some doxygen and curly placement. (asterisk[master])

Richard Mudgett asteriskteam at digium.com
Tue Jun 19 11:46:47 CDT 2018


Richard Mudgett has uploaded this change for review. ( https://gerrit.asterisk.org/9205


Change subject: Fix some doxygen and curly placement.
......................................................................

Fix some doxygen and curly placement.

Change-Id: I9a784a7c804120a8fa826c2a4cb9957e4b0b2fc8
---
M include/asterisk/utils.h
M main/pbx_functions.c
2 files changed, 11 insertions(+), 9 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/05/9205/1

diff --git a/include/asterisk/utils.h b/include/asterisk/utils.h
index b892cda..f1607ff 100644
--- a/include/asterisk/utils.h
+++ b/include/asterisk/utils.h
@@ -818,7 +818,7 @@
  */
 char *ast_crypt(const char *key, const char *salt);
 
-/*
+/*!
  * \brief Asterisk wrapper around crypt(3) for encrypting passwords.
  *
  * This function will generate a random salt and encrypt the given password.
@@ -831,7 +831,7 @@
  */
 char *ast_crypt_encrypt(const char *key);
 
-/*
+/*!
  * \brief Asterisk wrapper around crypt(3) for validating passwords.
  *
  * \param key User's password to validate.
@@ -841,7 +841,7 @@
  */
 int ast_crypt_validate(const char *key, const char *expected);
 
-/*
+/*!
  * \brief Test that a file exists and is readable by the effective user.
  * \since 13.7.0
  *
@@ -851,7 +851,7 @@
  */
 int ast_file_is_readable(const char *filename);
 
-/*
+/*!
  * \brief Compare 2 major.minor.patch.extra version strings.
  * \since 13.7.0
  *
@@ -864,7 +864,7 @@
  */
 int ast_compare_versions(const char *version1, const char *version2);
 
-/*
+/*!
  * \brief Test that an OS supports IPv6 Networking.
  * \since 13.14.0
  *
@@ -878,7 +878,7 @@
 	AST_FD_FLAG_CLEAR,
 };
 
-/*
+/*!
  * \brief Set flags on the given file descriptor
  * \since 13.19
  *
@@ -894,7 +894,7 @@
 #define ast_fd_set_flags(fd, flags) \
 	__ast_fd_set_flags((fd), (flags), AST_FD_FLAG_SET, __FILE__, __LINE__, __PRETTY_FUNCTION__)
 
-/*
+/*!
  * \brief Clear flags on the given file descriptor
  * \since 13.19
  *
diff --git a/main/pbx_functions.c b/main/pbx_functions.c
index 3c15e9d..bb3c663 100644
--- a/main/pbx_functions.c
+++ b/main/pbx_functions.c
@@ -300,7 +300,8 @@
  * \return True (non-zero) if reads escalate privileges.
  * \return False (zero) if reads just read.
  */
-static int read_escalates(const struct ast_custom_function *acf) {
+static int read_escalates(const struct ast_custom_function *acf)
+{
 	return acf->read_escalates;
 }
 
@@ -311,7 +312,8 @@
  * \return True (non-zero) if writes escalate privileges.
  * \return False (zero) if writes just write.
  */
-static int write_escalates(const struct ast_custom_function *acf) {
+static int write_escalates(const struct ast_custom_function *acf)
+{
 	return acf->write_escalates;
 }
 

-- 
To view, visit https://gerrit.asterisk.org/9205
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9a784a7c804120a8fa826c2a4cb9957e4b0b2fc8
Gerrit-Change-Number: 9205
Gerrit-PatchSet: 1
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180619/eb314412/attachment.html>


More information about the asterisk-code-review mailing list