Changeset a35b458 in mainline for uspace/srv/vfs/vfs_node.c


Ignore:
Timestamp:
2018-03-02T20:10:49Z (7 years ago)
Author:
Jiří Zárevúcky <zarevucky.jiri@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
f1380b7
Parents:
3061bc1
git-author:
Jiří Zárevúcky <zarevucky.jiri@…> (2018-02-28 17:38:31)
git-committer:
Jiří Zárevúcky <zarevucky.jiri@…> (2018-03-02 20:10:49)
Message:

style: Remove trailing whitespace on _all_ lines, including empty ones, for particular file types.

Command used: tools/srepl '\s\+$' '' -- *.c *.h *.py *.sh *.s *.S *.ag

Currently, whitespace on empty lines is very inconsistent.
There are two basic choices: Either remove the whitespace, or keep empty lines
indented to the level of surrounding code. The former is AFAICT more common,
and also much easier to do automatically.

Alternatively, we could write script for automatic indentation, and use that
instead. However, if such a script exists, it's possible to use the indented
style locally, by having the editor apply relevant conversions on load/save,
without affecting remote repository. IMO, it makes more sense to adopt
the simpler rule.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • uspace/srv/vfs/vfs_node.c

    r3061bc1 ra35b458  
    108108{
    109109        bool free_node = false;
    110        
    111         fibril_mutex_lock(&nodes_mutex);
    112        
     110
     111        fibril_mutex_lock(&nodes_mutex);
     112
    113113        node->refcnt--;
    114114        if (node->refcnt == 0) {
     
    117117                 * Remove it from the VFS node hash table.
    118118                 */
    119                
     119
    120120                hash_table_remove_item(&nodes, &node->nh_link);
    121121                free_node = true;
    122122        }
    123        
    124         fibril_mutex_unlock(&nodes_mutex);
    125        
     123
     124        fibril_mutex_unlock(&nodes_mutex);
     125
    126126        if (free_node) {
    127127                /*
     
    129129                 * are no more hard links.
    130130                 */
    131                
     131
    132132                async_exch_t *exch = vfs_exchange_grab(node->fs_handle);
    133133                async_msg_2(exch, VFS_OUT_DESTROY, (sysarg_t) node->service_id,
     
    239239            (node->service_id == rd->service_id))
    240240                rd->refcnt += node->refcnt;
    241        
     241
    242242        return true;
    243243}
     
    268268{
    269269        async_exch_t *exch = vfs_exchange_grab(node->fs_handle);
    270        
     270
    271271        ipc_call_t answer;
    272272        aid_t req = async_send_2(exch, VFS_OUT_OPEN_NODE,
    273273            (sysarg_t) node->service_id, (sysarg_t) node->index, &answer);
    274        
     274
    275275        vfs_exchange_release(exch);
    276276
    277277        errno_t rc;
    278278        async_wait_for(req, &rc);
    279        
     279
    280280        return rc;
    281281}
     
    311311                .index = node->index
    312312        };
    313        
     313
    314314        return tri;
    315315}
Note: See TracChangeset for help on using the changeset viewer.