"Detlef Riekenberg" <[email protected]> wrote:

> >> + /* win8: E_NOTIMPL */
> >> + ok(hr == E_FAIL || hr == E_NOTIMPL, "got 0x%08x.\n", hr);
> 
> >A usual form for such comparisons is
> >ok(hr == E_FAIL || hr == E_NOTIMPL /* win8 */, "got 0x%08x.\n", hr);
> I disagree!
> 
> Comments at the end of a line are ok,
> but a comment in the middle /* broken */ of a line decrease /* ugly by design 
> */ the readability
> of the code /* not compiler related */ for humans/developers a lot and should 
> be allowed
> only for backwards compatibility.

You are confusing a sentence and a set of statements.

-- 
Dmitry.


Reply via email to