which_version.c

// Version 1: tracking if the operations succeed and nesting if-else statements

bool LinkedList_Add(LinkedList_t *List, Data_t Data)
{
  bool Success = true;

  if (NULL == List)
  {
    Success = false;
  }
  else
  {
    LinkedList_Node_t *NewNode = malloc(sizeof(LinkedList_Node_t));

    if (NULL == NewNode)
    {
      Success = false;
    }
    else
    {
      NewNode->Data = Data;
      NewNode->Next = List->Head;
      List->Head = NewNode;
    }
  }

  return Success;
}

// Version 2: shortcutting when things fail

bool LinkedList_Add(LinkedList_t *List, Data_t Data)
{
  if (NULL == List)
  {
    return false;
  }

  LinkedList_Node_t *NewNode = malloc(sizeof(LinkedList_Node_t));

  if (NULL == NewNode)
  {
    return false;
  }
  
  NewNode->Data = Data;
  NewNode->Next = List->Head;
  List->Head = NewNode;

  return true;
}