Changeset 338d54a7 in mainline for uspace/srv/logger

Timestamp:

2018-03-10T22:55:07Z (8 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

850fd32

Parents:

5ef16903

Message:

Gratuitous nested block makes ccheck sad.

Location:

uspace/srv/logger

Files:

• ctl.c (modified) (3 diffs)
• writer.c (modified) (3 diffs)

uspace/srv/logger/ctl.c

@@ -65 +65 @@
 void logger_connection_handler_control(ipc_callid_t callid)
 {
+        errno_t rc;
+        int fd;
+
         async_answer_0(callid, EOK);
         logger_log("control: new client.\n");
@@ -76 +79 @@
 
                 switch (IPC_GET_IMETHOD(call)) {
-                case LOGGER_CONTROL_SET_DEFAULT_LEVEL: {
-                        errno_t rc = set_default_logging_level(IPC_GET_ARG1(call));
+                case LOGGER_CONTROL_SET_DEFAULT_LEVEL:
+                        rc = set_default_logging_level(IPC_GET_ARG1(call));
                         async_answer_0(callid, rc);
                         break;
-                }
-                case LOGGER_CONTROL_SET_LOG_LEVEL: {
-                        errno_t rc = handle_log_level_change(IPC_GET_ARG1(call));
+                case LOGGER_CONTROL_SET_LOG_LEVEL:
+                        rc = handle_log_level_change(IPC_GET_ARG1(call));
                         async_answer_0(callid, rc);
                         break;
-                }
-                case LOGGER_CONTROL_SET_ROOT: {
-                        int fd;
-                        errno_t rc = vfs_receive_handle(true, &fd);
+                case LOGGER_CONTROL_SET_ROOT:
+                        rc = vfs_receive_handle(true, &fd);
                         if (rc == EOK) {
                                 rc = vfs_root_set(fd);
@@ -95 +95 @@
                         async_answer_0(callid, rc);
                         break;
-                }
                 default:
                         async_answer_0(callid, EINVAL);

uspace/srv/logger/writer.c

@@ -96 +96 @@
 void logger_connection_handler_writer(ipc_callid_t callid)
 {
+        logger_log_t *log;
+        errno_t rc;
+
         /* Acknowledge the connection. */
         async_answer_0(callid, EOK);
@@ -112 +115 @@
 
                 switch (IPC_GET_IMETHOD(call)) {
-                case LOGGER_WRITER_CREATE_LOG: {
-                        logger_log_t *log = handle_create_log(IPC_GET_ARG1(call));
+                case LOGGER_WRITER_CREATE_LOG:
+                        log = handle_create_log(IPC_GET_ARG1(call));
                         if (log == NULL) {
                                 async_answer_0(callid, ENOMEM);
@@ -126 +129 @@
                         async_answer_1(callid, EOK, (sysarg_t) log);
                         break;
-                }
-                case LOGGER_WRITER_MESSAGE: {
-                        errno_t rc = handle_receive_message(IPC_GET_ARG1(call),
+                case LOGGER_WRITER_MESSAGE:
+                        rc = handle_receive_message(IPC_GET_ARG1(call),
                             IPC_GET_ARG2(call));
                         async_answer_0(callid, rc);
                         break;
-                }
                 default:
                         async_answer_0(callid, EINVAL);