On 10/11/12 12:39, Henri Verbeet wrote:
> On 11 October 2012 12:16, Jacek Caban <ja...@codeweavers.com> wrote:
>> +struct _jsstr_t {
>> + unsigned length_flags;
>> + unsigned ref;
>> + WCHAR str[1];
>> +};
> It's probably much too late for jscript, and perhaps Wine in general,
> but strictly speaking the _t suffix is reserved by POSIX.
I consider using it a matter of taste. It's often convenient to add
suffix so that the namespace is free for things like that:
mystryct_t *mystruct;
(and struct mystruct *mystruct; is a worse solution IMHO).
As a matter of fact, I was considering dropping _t suffix for jsstr_t
type (as well as jsval_t, which I introduced lately) mostly because
those types are so core for jscript that they deserve looking a bit like
base types. I didn't know about the suffix reservation for POSIX (well,
C99 also uses _t and it's not part of POSIX), that's a good point.
That said, I don't have strong opinion here for general practices in
Wine. I will reconsider dropping the suffix for a few types.
Jacek
