Ignore:
Timestamp:
Aug 30, 2018, 6:45:18 PM (4 years ago)
Author:
viala@…
Message:

[boot] Remove testing if a pointer is null.

It's a bad practice in a lib to hide undesired null pointers.
Using an assert is better.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/boot/tsar_mips32/boot_utils.c

    r521 r523  
    284284void boot_strcpy( char * dest, const char * src)
    285285{
    286     /* Checking if the arguments are correct. */
    287     if ((dest == NULL) || (src == NULL))
    288         return;
    289 
    290286    /* Copying the string. */
    291287    while ((*dest++ = *src++) != '\0');
     
    297293{
    298294    uint32_t res = 0;   /* Length of the string (in bytes).             */
    299 
    300     if (s != NULL)
    301     {
    302         while (*s++ != '\0')
    303             res++;
     295    while (*s++ != '\0') {
     296        res++;
    304297    }
    305298
    306299    return res;
    307 
    308300} // boot_strlen()
    309301
     
    311303int boot_strcmp( const char * s1, const char * s2 )
    312304{
    313     if ((s1 == NULL) || (s2 == NULL))
     305    if (s1 == s2)
    314306        return 0;
    315307
Note: See TracChangeset for help on using the changeset viewer.