Re: [PATCH] libdebian-installer: add support for MIPS Malta
Otavio Salvador wrote:
> Aurelien Jarno <aurelien@aurel32.net> writes:
>
> > Otavio Salvador a écrit :
> >> Aurelien Jarno <aurelien@aurel32.net> writes:
> >>
> >>> +static struct cpu system_malta_cpu[] = {
> >> system_mips_malta_cpu would be clearer, IMO.
> >>
> >
> > Well other names from the same file (which BTW has mips in its name)
> > don't contain mips, that's why I did the same. But that can be changed.
>
> Personally I think it would be easier for someone reading the code if
> we put it there (and also fixes the missing ones). What others think
> about it?
I don't believe using redundant namespaces will help understanding.
In worst case it adds more confusion about mips/mipsel. That's why
I wrote the original code that way.
Thiemo
Reply to: