[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.022 / U:14.128 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site