<p>In <a href="https://github.com/RIOT-OS/RIOT/pull/3170#discussion_r31925295">cpu/sam3x8e/periph/pwm.c</a>:</p>
<pre style='color:#555'>> +     * Mode check.
> +     * Involves things that shold be in "periph_conf.h".
> +     * !Junk!Just wasting memory an a few cpu cycles for now. ;/
> +     */
> +    switch ( mode) {
> +    case PWM_LEFT:
> +        //!\todo
> +        break;
> +    case PWM_CENTER:
> +        //!\todo
> +        break;
> +    default:
> +        //retval = ERR_INIT_BWTH;
> +        // Don't like :/
> +        return ERR_INIT_BWTH;
> +    }
</pre>
<p>I tend to have a "completely useless" switch-block to (i) be prepared for further implementations and (ii) highlight the lack so one immediately sees what's missing.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/RIOT-OS/RIOT/pull/3170/files#r31925295">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/AEn7YOiMJgrJF9FPdfqyRQm8VvwGnTUyks5oRan9gaJpZM4E5xxI.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
  <div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
    <link itemprop="url" href="https://github.com/RIOT-OS/RIOT/pull/3170/files#r31925295"></link>
    <meta itemprop="name" content="View Pull Request"></meta>
  </div>
  <meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>