piql / afs

Archival File System
GNU General Public License v3.0
13 stars 5 forks source link

Comment clean up. #4

Closed dlemstra closed 4 years ago

dlemstra commented 4 years ago

When going through the code I noticed that at various places there are comments like this:

// If control data pointer is NULL return NULL.
if (control_data == NULL)
{
    return NULL;
}

This is stating the obvious and I am wondering if the team would mind me going through the code to remove them? I don't think they add value and some of them are even incorrect:

// If control data pointer is NULL return DFALSE
if (control_data == NULL)
{
    return NULL;
}
hakonlarsson commented 4 years ago

You're quite right, it doesn't make sense to me either. Please do go ahead and remove them.

petterreinholdtsen commented 4 years ago

This issue should have been closed when #10 was merged.