Developer forums (C::B DEVELOPMENT STRICTLY!) > Plugins development

unknown exception

<< < (6/17) > >>

280Z28:

--- Quote from: thomas on December 16, 2005, 05:28:42 pm ---Why do you keep telling that... I have been looking both under "bugs" and under "patches". It is not there.

--- End quote ---

Ohhhhh I was talking about the positioning patch. It's the only one I've submitted because it's going to very hard to isolate the others while the diff shows 50 files were changed. I'm waiting for that one and then I have 5 or so more patches in a row I'll put of SF. ;)

thomas:
Well you know... bluntly speaking, the positioning patch is really not important. Those are cosmetics.

Grv575 cannot use Code::Blocks because of the bug in the CRC calculation - this is a showstopper.
It effectively prevents him from using Code::Blocks alltogether (and not him alone, many people who use Unicode).

So if you have really solved that problem, then it would maybe a good idea if you made this patch a number 1 priority. Or you could at least describe what is wrong, then we can patch it by hand. You know, reading "I have fixed it long ago" over and over is quite annoying if it does not work for you personally...  put yourself into that position.

grv575:
The wxChar* doesn't compile (it's expecting wxCharBuffer).  Anyway the following does:


--- Code: ---unsigned long wxCrc32::FromString(const wxString& text)
{
    static unsigned long *crc_table = NULL;
    unsigned long crc = 0;
    ///const char* p = text.mb_str(wxConvUTF8);
    int i = 0;
    ///const wxCharBuffer wxcb = text.mb_str(wxConvUTF8);

    if (text)
    {
        // Get the crc table, on first call, generate, otherwise do nothing
        crc_table = GetCRC32Table( crc_table ) ;

        // Do we have a crc table?
        if ( crc_table )
        {
            // Calculate the checksum
            crc = 0xFFFFFFFFUL;
            while (text[i])
                { crc = (crc>>8) ^ crc_table[ (crc^(text[i++])) & 0xFF ]; }

            crc ^= 0xFFFFFFFFUL ;
        }
    }

    // If we have a crc table, delete it from memory
    if ( crc_table ) { delete[] crc_table; }

    // Set it to a null pointer, the have it (re)created on next calls to this
    // function
    crc_table = NULL;

    // Return the checksum result
    return( crc ) ;
}

--- End code ---

But that's not the whole cause.  Set a breakpoint (pending bps are $$$ btw) and trace to:
   if ( crc_table )
and it crashes.  Must be something in the crc table funcion above (and I must admit, I'm not wx unicode expert).

thomas:
I am not getting Code::Blocks to link with Unicode. It does not find various XRC related stuff :(

So I cannot work on this, sorry.

280Z28:

--- Quote from: thomas on December 16, 2005, 06:14:59 pm ---Well you know... bluntly speaking, the positioning patch is really not important. Those are cosmetics.

Grv575 cannot use Code::Blocks because of the bug in the CRC calculation - this is a showstopper.
It effectively prevents him from using Code::Blocks alltogether (and not him alone, many people who use Unicode).

So if you have really solved that problem, then it would maybe a good idea if you made this patch a number 1 priority. Or you could at least describe what is wrong, then we can patch it by hand. You know, reading "I have fixed it long ago" over and over is quite annoying if it does not work for you personally...  put yourself into that position.

--- End quote ---

Please, all I ask is you patch the one single patch I posted already so I can get a diff of this one. :( I'm not trying to be a ***** about it. Seriously, it's just that there are a lot of files that show up in "svn status" and then I have to do a diff on every single one to find the interspersed other patches. With the dialog patch committed, "svn status" will return a much smaller number of files and I'll be able to easily post patches for the other issues. I posted another patch just now actually... a 2-liner. I'm at work and don't have my SF password so it says I'm a nobody lol. It's the auto-complete spacing patch on SF. But yes the one I'm stuck on for svn status is the dialog one.

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version