Changeset b7fd2a0 in mainline for uspace/srv/fs/exfat/exfat_bitmap.c


Ignore:
Timestamp:
2018-01-13T03:10:29Z (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:
a53ed3a
Parents:
36f0738
Message:

Use errno_t in all uspace and kernel code.

Change type of every variable, parameter and return value that holds an
<errno.h> constant to either errno_t (the usual case), or sys_errno_t
(some places in kernel). This is for the purpose of self-documentation,
as well as for type-checking with a bit of type definition hackery.

Although this is a massive commit, it is a simple text replacement, and thus
is very easy to verify. Simply do the following:

`
git checkout <this commit's hash>
git reset HEAD
git add .
tools/srepl '\berrno_t\b' int
git add .
tools/srepl '\bsys_errno_t\b' sysarg_t
git reset
git diff
`

While this doesn't ensure that the replacements are correct, it does ensure
that the commit doesn't do anything except those replacements. Since errno_t
is typedef'd to int in the usual case (and sys_errno_t to sysarg_t), even if
incorrect, this commit cannot change behavior.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • uspace/srv/fs/exfat/exfat_bitmap.c

    r36f0738 rb7fd2a0  
    4848
    4949
    50 int exfat_bitmap_is_free(exfat_bs_t *bs, service_id_t service_id,
     50errno_t exfat_bitmap_is_free(exfat_bs_t *bs, service_id_t service_id,
    5151    exfat_cluster_t clst)
    5252{
     
    5555        exfat_node_t *bitmapp;
    5656        uint8_t *bitmap;
    57         int rc;
     57        errno_t rc;
    5858        bool alloc;
    5959
     
    8989}
    9090
    91 int exfat_bitmap_set_cluster(exfat_bs_t *bs, service_id_t service_id,
     91errno_t exfat_bitmap_set_cluster(exfat_bs_t *bs, service_id_t service_id,
    9292    exfat_cluster_t clst)
    9393{
     
    9696        exfat_node_t *bitmapp;
    9797        uint8_t *bitmap;
    98         int rc;
     98        errno_t rc;
    9999
    100100        clst -= EXFAT_CLST_FIRST;
     
    124124}
    125125
    126 int exfat_bitmap_clear_cluster(exfat_bs_t *bs, service_id_t service_id,
     126errno_t exfat_bitmap_clear_cluster(exfat_bs_t *bs, service_id_t service_id,
    127127    exfat_cluster_t clst)
    128128{
     
    131131        exfat_node_t *bitmapp;
    132132        uint8_t *bitmap;
    133         int rc;
     133        errno_t rc;
    134134
    135135        clst -= EXFAT_CLST_FIRST;
     
    160160}
    161161
    162 int exfat_bitmap_set_clusters(exfat_bs_t *bs, service_id_t service_id,
     162errno_t exfat_bitmap_set_clusters(exfat_bs_t *bs, service_id_t service_id,
    163163    exfat_cluster_t firstc, exfat_cluster_t count)
    164164{
    165         int rc;
     165        errno_t rc;
    166166        exfat_cluster_t clst;
    167167        clst = firstc;
     
    180180}
    181181
    182 int exfat_bitmap_clear_clusters(exfat_bs_t *bs, service_id_t service_id,
     182errno_t exfat_bitmap_clear_clusters(exfat_bs_t *bs, service_id_t service_id,
    183183    exfat_cluster_t firstc, exfat_cluster_t count)
    184184{
    185         int rc;
     185        errno_t rc;
    186186        exfat_cluster_t clst;
    187187        clst = firstc;
     
    196196}
    197197
    198 int exfat_bitmap_alloc_clusters(exfat_bs_t *bs, service_id_t service_id,
     198errno_t exfat_bitmap_alloc_clusters(exfat_bs_t *bs, service_id_t service_id,
    199199    exfat_cluster_t *firstc, exfat_cluster_t count)
    200200{
     
    217217
    218218
    219 int exfat_bitmap_append_clusters(exfat_bs_t *bs, exfat_node_t *nodep,
     219errno_t exfat_bitmap_append_clusters(exfat_bs_t *bs, exfat_node_t *nodep,
    220220    exfat_cluster_t count)
    221221{
     
    240240
    241241
    242 int exfat_bitmap_free_clusters(exfat_bs_t *bs, exfat_node_t *nodep,
     242errno_t exfat_bitmap_free_clusters(exfat_bs_t *bs, exfat_node_t *nodep,
    243243    exfat_cluster_t count)
    244244{
     
    251251
    252252
    253 int exfat_bitmap_replicate_clusters(exfat_bs_t *bs, exfat_node_t *nodep)
    254 {
    255         int rc;
     253errno_t exfat_bitmap_replicate_clusters(exfat_bs_t *bs, exfat_node_t *nodep)
     254{
     255        errno_t rc;
    256256        exfat_cluster_t lastc, clst;
    257257        service_id_t service_id = nodep->idx->service_id;
Note: See TracChangeset for help on using the changeset viewer.