[lkml]   [2008]   [Jun]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] x86: coding style fixes to arch/x86/math-emu/reg_constant
> On Wed, Jun 18, 2008 at 3:09 PM, Dmitri Vorobiev
> <> wrote:
>> Paolo Ciarrocchi wrote:
>>> -static void fld_const(FPU_REG const *c, int adj, u_char tag)
>>> +static void fld_const(FPU_REG const * c, int adj, u_char tag)
>> ________________________________________^
>> The above does not seem to me like a big improvement in code quality. A
>> false positive from

It should obviously be

static void fld_const(const FPU_REG *c, int adj, u_char tag)

FPU_REG is not an appropriate type name though. Is "typedef struct
fpu__reg FPU_REG;" necessary at all? (Removing it would create some
code churn all over x86/math-emu/ though. Maybe not desirable.)
Stefan Richter
-=====-==--- -==- =--=-

 \ /
  Last update: 2008-06-18 17:01    [W:0.042 / U:3.784 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site