Question

I see in some of the code on Github

if($something_funky_happens){
  throw new \LogicException(...);
  return;
}

Is this required, or it is not important to return?
because I know that after you throw exceptions the script stops, so further code does not run

Answer 1

unreachable code

I tend to denote it an error in their code. The statement is not reachable. Every static code anaylsis tool will complain about this unreachable statement. Even when it has no harm in this case you will get a load of warnings when checking your code. If you turn these warning type off you will perhaps miss other logic errors within your code. It’s a real bad smell and it needs to be fixed.