Hey ive got a couple warnings that i think MamiyaOtaru can only answer because its associated with his new code. this is all build off of the CVS. below is the output of the compiler with the warned code from the patch. if you don't want to scroll up the patch is http://www.lsdev.org/bugs/view.php?id=107
1>------ Rebuild All started: Project: utility, Configuration: Debug Win32 ------
1> shellhlp.cpp
1> debug.cpp
1> Generating Code...
1> utility_vc8.vcxproj -> C:\Users\cnbm47\Documents\Litestep Build\cvs\utility\../Debug_VC8\utility.lib
2>------ Rebuild All started: Project: lsapi, Configuration: Debug Win32 ------
2> stubs.cpp
2> settingsmanager.cpp
2> SettingsIterator.cpp
2> SettingsFileParser.cpp
2> settings.cpp
2> png_support.cpp
2> picopng.cpp
2> MathValue.cpp
2> MathToken.cpp
2> MathScanner.cpp
2> MathParser.cpp
2> MathEvaluate.cpp
2> lsapiInit.cpp
2> lsapi.cpp
2> graphics.cpp
2> bangs.cpp
2> BangManager.cpp
2> BangCommand.cpp
2> aboutbox.cpp
2> Generating Code...
2> match.cpp
2> Creating library ../Debug_VC8\lsapi.lib and object ../Debug_VC8\lsapi.exp
2> lsapi_vc8.vcxproj -> C:\Users\cnbm47\Documents\Litestep Build\cvs\lsapi\../Debug_VC8\lsapi.dll
3>------ Rebuild All started: Project: litestep, Configuration: Debug Win32 ------
3> WinMain.cpp
3> TrayService.cpp
3>c:\users\cnbm47\documents\litestep build\cvs\litestep\trayservice.cpp(655): warning C4100: 'cbData' : unreferenced formal parameter
3>c:\users\cnbm47\documents\litestep build\cvs\litestep\trayservice.cpp(657): warning C4189: 'lr' : local variable is initialized but not referenced
3> TrayNotifyIcon.cpp
3> StartupRunner.cpp
3> RecoveryMenu.cpp
3> ModuleManager.cpp
3> Module.cpp
3> MessageManager.cpp
3> litestep.cpp
3> DDEWorker.cpp
3> DDEStub.cpp
3> DDEService.cpp
3> DataStore.cpp
3> Generating Code...
3>c:\users\cnbm47\documents\litestep build\cvs\litestep\trayservice.cpp(684): warning C4715: 'TrayService::TrayInfoEvent' : not all control paths return a value
3> litestep_vc8.vcxproj -> C:\Users\cnbm47\Documents\Litestep Build\cvs\litestep\../Debug_VC8\litestep.exe
========== Rebuild All: 3 succeeded, 0 failed, 0 skipped ==========
the first two are straightforward enough the DWORD cbdata never gets used even tough its required, and LRESULT lr never gets used after it gets declared.
LRESULT TrayService::TrayInfoEvent(DWORD cbData, LPVOID lpData) // size, data
{
LRESULT lr = 0;
the final warning is because not all possibility are spelled out, which based on what i could figure out isn't a bad thing, just will be hard to debug if it ever gets in an unknown state.
if (s->dwMessage == 2) {
return MAKELONG(16,16);
}
POINT p;
GetCursorPos(&p);
if (s->dwMessage == 1) {
return MAKELPARAM(p.x, p.y);
}
I would think that the first warning is something to care about but the last two can be blown off for now until it gets cleaned up.
Final binarys and changed code can be found here -> http://carbinehunters.com/litestep/
Joe
Litestep will never die.