Recursive call to cecho?

[gamcall]

Hi - I’m wondering if you meant to call cechon() here instead of where you recursively call cecho()?

function cecho()
{
  if [ -z "${1-}" ]; then
    cecho "No message passed.\n" "${2-}"
    return $?
  fi
  cechon "$1"$'\n' "${2-}"
  return $?
}

[BMDan]

This triggers only if $1 is empty or unset. Since the recursive call sets $1 in the recursion (to No message passed.\n), it won't recurse infinitely.

You are right, however, that there's an inexplicable use of a literal \n there. That's confusing; it should either be eliminated, or the line should be changed to cechon and \n replaced with $'\n'. Of those, I chose the option that only requires a single change.