Re: (CTRL-Z) Re: commit - Win32 Warning fixes

From: Andrew Dunbar (hippietrail@yahoo.com)
Date: Thu Sep 12 2002 - 21:20:44 EDT

  • Next message: Mark Gilbert: "Re: plugin abicommand broke build of rpm"

     --- "Michael D. Pritchett" <mpritchett@attglobal.net>
    wrote:
    >
    > On 12 Sep 2002, Mark Gilbert wrote:
    >
    > > > Modified Files:
    > > [...]
    > > > src/other/spell/xp/ispell_checker.cpp
    > >
    > > Sorry dude, unix build errors take precedence over
    > > win32 build warnings. On this side of the tracks
    > > we declare our variables.
    > >
    > > -MG
    > > --
    >
    > MG -
    >
    > I agree. Errors take precendence over warnings.
    > But the only change that I made to
    > ispell_checker.cpp was removal of a UT_sint32 ret;
    > line in loadDictionaryForLanuage() function. See
    > BONSAI. This variable was decleared but never used.
    > I don't see how it broke any build. Can you
    clarify?
    >
    > Additionally, I don't see your commit in CVS. Where
    > did it go?

    Careful guys. Make sure you're both working and
    committing on the same tree. I was working with this
    very file in HEAD at the time and the result was that
    I now have the variable "ret" being used but not
    being declared...

    Andrew Dunbar.

    > Cheers!
    >
    >
    > Michael D. Pritchett
    > (getting paranoid about fixes to his commits that
    > never seem to make it
    > into CVS)
    >

    =====
    http://linguaphile.sourceforge.net/cgi-bin/translator.pl http://www.abisource.com

    __________________________________________________
    Do You Yahoo!?
    Everything you'll ever need on one web page
    from News and Sport to Email and Music Charts
    http://uk.my.yahoo.com



    This archive was generated by hypermail 2.1.4 : Thu Sep 12 2002 - 21:24:19 EDT