Re: [hackers] [quark] Initial commit of quark rewrite || FRIGN

From: Hiltjo Posthuma <hiltjo_AT_codemadness.org>
Date: Fri, 2 Sep 2016 14:17:01 +0200

On Fri, Sep 02, 2016 at 12:37:00PM +0100, Nick wrote:
> Hi FRIGN,
>
> I don't have time to look at this closely now, but one little thing
> that jumped out at me:
>
> > +static const struct {
> > + char *ext;
> > + char *type;
> > +} mimes[] = {
> > ...
>
> Why are you defining the struct in config.def.h? Shouldn't it be
> defined in quark.c, and then just populated in config.def.h. That is
> what other projects do, and it makes more sense, as the code in
> quark.c depends on the struct being defined a certain way.
>
> Nick
>

I disagree, aslong as it satisfies the members: char *ext, *type it will work.
The struct name itself is not defined or used anywhere. So the current code is
fine imho.

-- 
Kind regards,
Hiltjo
Received on Fri Sep 02 2016 - 14:17:01 CEST

This archive was generated by hypermail 2.3.0 : Fri Sep 02 2016 - 14:24:15 CEST