Ah OK, didn't know this...
I'll wait for a courtesy update then, thanks.
;)
Tuxman, make a doubleclick on the vc_9 project file,
the messagebox tell you what you must change.
(the versionnumber on 3-8 line in the projectfile)
The visual studio project C++ file is not very special, at least the 32-bit builds works.(64-bit not tested)
I not recommend to do this, but you can. Sometimnes it's the only way if the developers add/remove files.
Or import the VC6 project file.
Peter
Unfortunately, even the VC6 project files are sometimes outdated. But the advantage is that they can easily be modified manually, as compared to the XML-like projects.
One other thing to keep in mind is that a batch project in VC6 will always build according to the order the individual projects have been added; moreover, the individual dependencies are not taken into account when batch-building. Therefore, errors may pop up about missing zlib.lib.
To fix this, one should manually edit the miranda\bin\miranda32.dsw project, change the sections so that the zlib project would be at the very top. Also, keep a copy of this file, because editing anything in the project at compilation time will ask to save the workspace and it will screw up the project sorting again.
While I'm here, I'd like to ask the devs to have a look at the following warnings and maybe fix them, as I fear some of them may cause troubles:
There are others of minor importance (IMO) and also some that I fixed myself in my local copy of the sources. Since the core devs never commented on (or applied - except for the AIM Profile Manager one, I think) the patches I've posted, I'll keep those minor changes for myself.Code:• D:\miranda\trunk\miranda\plugins\clist_nicer\clui.c(336) : warning C4133: 'function' : incompatible types - from 'struct PLUGININFOEX *' to 'struct PLUGININFO *' • D:\miranda\trunk\miranda\protocols\Gadu-Gadu\services.c(768) : warning C4047: '=' : 'void *(__cdecl *)(struct tagPROTO_INTERFACE *,void *,void *,const char *)' differs in levels of indirection from 'int (__cdecl *)(struct tagPROTO_INTERFACE *,void * ,void *,const char *)' • D:\miranda\trunk\miranda\protocols\Gadu-Gadu\services.c(789) : warning C4047: '=' : 'void *(__cdecl *)(struct tagPROTO_INTERFACE *,void *,const char *,char ** )' differs in levels of indirection from 'int (__cdecl *)(struct tagPROTO_INTERFACE *,void *,const char *,char ** )' • D:\miranda\trunk\miranda\protocols\Gadu-Gadu\services.c(796) : warning C4047: '=' : 'void *(__cdecl *)(struct tagPROTO_INTERFACE *,void *)' differs in levels of indirection from 'int (__cdecl *)(struct tagPROTO_INTERFACE *,void *)' • D:\miranda\trunk\miranda\protocols\Yahoo\libyahoo2\libyahoo2.c(6637) : warning C4090: 'function' : different 'const' qualifiers • D:\miranda\trunk\miranda\protocols\Yahoo\libyahoo2\libyahoo2.c(6637) : warning C4022: 'free' : pointer mismatch for actual parameter 1
Last edited by Drugwash; 2 Apr 2009 at 12:11 PM. Reason: added revision number to comply to my own rules :-)
For example: the Network options page is F-ed up for keyboard navigation. Our visually impaired user -pv- has asked for a long time for this to be fixed and nobody gave a damn on his request.
I had reorganized those options in a more logical manner with a better visual flow (in my opinion) and fixed the keyboard navigation. But if you say it doesn't need to be fixed...
As for libyahoo2, I believe Gennady as our main Yahoo developer is more entitled to talk with the libyahoo2 developers. I merely pointed out a couple of things that may be troublesome but can't tell if it's really worth the bother or not.
Oh and while Yahoo is the topic, I should also point out that the current code connects fine for one account, but when trying to connect a second account, it crashes the main Yahoo thread and the protocol will not respond anymore.
Yesterday I've had other crashes with an earlier revision, in same conditions:Code:Miranda Crash Report from 2009-04-02 13:47:41+0300. Crash Dumper v.0.0.4.10 Likely cause of the crash plugin: Yahoo Protocol Beta/Nightly Exception Access Violation at address 02A73867. Reading from address 2A717C3C. Stack Trace: --------------------------------------------------------------- 02A73867 (YAHOO 02A60000): D:\miranda\trunk\miranda\protocols\Yahoo\server.cpp (161): CYahooProto::server_main 004014F5 (MIRANDA32 00400000): D:\miranda\trunk\miranda\src\core\miranda.cpp (192): forkthreadex_r 78007A41 (MSVCRT 78000000): (filename not available) (0): (function-name not available) BFF88F20 (KERNEL32 BFF70000): (filename not available) (0): HeapCreate BFF869EF (KERNEL32 BFF70000): (filename not available) (0): VirtualQueryEx BFF868EC (KERNEL32 BFF70000): (filename not available) (0): VirtualQueryEx 00700465 ((module-name not available) 00000000): (filename not available) (0): (function-name not available)One more issue, with AIM this time: the usage of %miranda_avatarcache% is slightly wrong. The Profiles folder is being duplicated in the path. Mirandaboot.ini reads "ProfileDir=.\Profiles". Example:Code:Miranda Crash Report from 2009-04-01 05:27:07+0300. Crash Dumper v.0.0.4.10 Likely cause of the crash plugin: Yahoo Protocol Beta/Nightly Exception Access Violation at address 02A66BFF. Reading from address FFFFFFFF. Stack Trace: --------------------------------------------------------------- 02A66BFF (YAHOO 02A60000): D:\miranda\trunk\miranda\protocols\Yahoo\libyahoo2\libyahoo2.c (4376): yahoo_getdata 02A623AA (YAHOO 02A60000): D:\miranda\trunk\miranda\protocols\Yahoo\libyahoo2\libyahoo2.c (4823): yahoo_process_pager_connection 02A68361 (YAHOO 02A60000): D:\miranda\trunk\miranda\protocols\Yahoo\libyahoo2\libyahoo2.c (5222): yahoo_read_ready 02A77C8A (YAHOO 02A60000): D:\miranda\trunk\miranda\protocols\Yahoo\yahoo.cpp (1372): yahoo_callback 02A73898 (YAHOO 02A60000): D:\miranda\trunk\miranda\protocols\Yahoo\server.cpp (169): CYahooProto::server_main 02A7360C (YAHOO 02A60000): D:\miranda\trunk\miranda\protocols\Yahoo\server.cpp (52): CYahooProto::server_main 006DF50C ((module-name not available) 00000000): (filename not available) (0): (function-name not available) 00840050 ((module-name not available) 00000000): (filename not available) (0): (function-name not available)
Correct path:
c:\Utils\08a18\profiles\08a18\AvatarCache\AIM\
Wrong path as created by AIM:
c:\Utils\08a18\profiles\Profiles\08a18\AvatarCache\AIM\
[EDIT] The protocol also does not have support for Folders anymore, which I use for other plug-ins. I believe support for Folders should be preserved, to remain consistent with all other plug-ins (all path-related settings in one place).
Last edited by Drugwash; 2 Apr 2009 at 1:32 PM. Reason: added mention about Folders support in AIM
As far as I know Gena is not able to compile latest Yahoo yet. So no fixes possible.
We are still working on directory structure change.
I guess you fond of forums discussions, go find your profile somewhere, or no you cannot put avatars from all protocols into a single folder, I am sick and tired of it.
Where did you get this idea I want all avatars in one folder?! Read my statement again, please. All I said is I like to have all settings in one page, so I can set AIM avatars, MSN avatars, PopupPlus skins, Crash Dumper logs and whatever needs/accepts a custom path, in one place (that is, the Folders options page) not one folder.
The rest of the reports are there for reference (feedback) purposes only. I know the code is in the works and only hope the reports can help finding and fixing errors faster.
Bookmarks