Re: Bidi B patch

[Alexandre Julliard]

Shachar Shemesh  writes:
> The reason I did was to reduce confusion. The usual includes are
> standard includes, and can be included with either <> or "". The new
> include (gdibidi.h) is a local include, and can only be included with
> "". To differentiate the two, I changed those who could afford a <>.

Local includes can be included with <> too, there's no reason to make
a distinction. And some of the headers in include/ are actually
internal, like gdi.h (actually I would argue that bidi definitions
should go there). The fact is that all our source files use "" for
both internal and exported headers, and we are not going to change all
of them. Changing only a few here and there creates a lot more
confusion than it solves.