Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

wxWidgets 2.6.4 Screws Up File-Type Icons!

<< < (3/4) > >>

Biplab:
I think I've figured out the issue.


--- Quote from: mandrav on March 26, 2007, 12:04:27 pm ---As you know, icons embedded inside an exe are indexed by number. My guess would be that wx2.6.4 probably includes more built-in icons in its resource file so we bring more icons inside our exe. This, in turn, screws up the icon indexes for our associations. This is just a guess but I wouldn't call it "wild" ;).

--- End quote ---

Yiannis is correct. Resource Hacker points that at id 1, standard wxWidgets Icon gets embedded, which basically screws up the rest of the icons. So all the numbers pointing to file type icons needs to be incremented by 1 to fix this issue. :D

Regards,

Biplab

Biplab:
Following patch should fix this issue.

--- Code: ------ C:/wxWidgets-2.6.4/include/wx/msw/wx.rc Mon Mar 26 18:15:59 2007
+++ C:/wxWidgets-2.6.4/include/wx/msw/wx.rc Mon Mar 26 18:17:13 2007
@@ -79,7 +79,7 @@
 
 // Low alphabetically to make it the default, but not so low as to
 // affect applications that use 'a'
-bICON                           ICON "wx/msw/std.ico"
+//bICON                           ICON "wx/msw/std.ico"
 
 //////////////////////////////////////////////////////////////////////////////
 //
--- End code ---

Though I've not tested the patch, but I'm quite sure that this is the source of problem.

I'll post the result after another full recompilation with wxWidgets-2.6.4.

Regards,

Biplab

thomas:

--- Quote from: mandrav on March 26, 2007, 12:04:27 pm ---My guess would be that wx2.6.4 probably includes more built-in icons in its resource file so we bring more icons inside our exe.
--- End quote ---
Hah! How easy :)

Well, people were asking for "skinnable" icons anyway, so I guess we could as well move the icons into a separate file, to avoid this problem in the future alltogether.

(Though I have no idea what file Windows wants then... .icl probably?)

Biplab:
FYI, that line is not present in CVS HEAD snapshot, wx-2.8.3. Only wx 2.6.4 has this issue. :)

Biplab:
Hi,

The patch I posted earlier fixes this issue. I've recompiled C::B with patched wxWidgets-2.6.4 and the issue gets resolved.

That line has been removed in later release, but unfortunately WX_26 series didn't get that update.

Regards,

Biplab

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version