Changeset 7c8267b in mainline for uspace/lib/socket/packet/packet_client.c

Timestamp:

2010-09-19T12:55:05Z (14 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

2fa0ad9, dc94cb2

Parents:

553f430

Message:

Coding style fixes.

This changeset converts the entire TCP code into HelenOS cstyle and improves
its general legibility by adding vertical spacing and removing unnecessary
clutter. It also attempts to suppress too deep nesting of blocks by
functional-equivalent code restructuralization.

This changeset brings no change in functionality (hopefully), but it should make
it easier to understand and eventually fix TCP code.

File:

• uspace/lib/socket/packet/packet_client.c (modified) (1 diff)

uspace/lib/socket/packet/packet_client.c

@@ -122 +122 @@
 }
 
-int packet_set_addr(packet_t packet, const uint8_t * src, const uint8_t * dest, size_t addr_len){
+int
+packet_set_addr(packet_t packet, const uint8_t * src, const uint8_t * dest,
+    size_t addr_len)
+{
         size_t padding;
         size_t allocated;
 
-        if(! packet_is_valid(packet)){
+        if (!packet_is_valid(packet))
                 return EINVAL;
-        }
+
         allocated = PACKET_MAX_ADDRESS_LENGTH(packet);
-        if(allocated < addr_len){
+        if (allocated < addr_len)
                 return ENOMEM;
-        }
+
         padding = allocated - addr_len;
         packet->addr_len = addr_len;
-        if(src){
+
+        if (src) {
                 memcpy((void *) packet + packet->src_addr, src, addr_len);
-                if(padding){
-                        bzero((void *) packet + packet->src_addr + addr_len, padding);
-                }
-        }else{
+                if (padding)
+                        bzero((void *) packet + packet->src_addr + addr_len,
+                            padding);
+        } else {
                 bzero((void *) packet + packet->src_addr, allocated);
         }
-        if(dest){
+
+        if (dest) {
                 memcpy((void *) packet + packet->dest_addr, dest, addr_len);
-                if(padding){
-                        bzero((void *) packet + packet->dest_addr + addr_len, padding);
-                }
-        }else{
+                if (padding)
+                        bzero((void *) packet + packet->dest_addr + addr_len,
+                            padding);
+        } else {
                 bzero((void *) packet + packet->dest_addr, allocated);
         }
+
         return EOK;
 }