Re: Aw: Re: [PATCH] comdlg32/tests: Accept win8 failure-code

[Dmitry Timoshkov]

"Detlef Riekenberg" <wine.dev(a)web.de> 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.