pinkshirt / firebreath

Automatically exported from code.google.com/p/firebreath
0 stars 0 forks source link

Visual Studio Express 2008 not compiling in 1.3.2 #120

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
. Using Firebreath 1.3.2 with the amended axutil.* files, a project will still 
not compile with Visual Studio Express 2008 + Windows Driver Kit 7.1

What is the expected output? What do you see instead?
. The code fails to compile because, in #ifdefs, in some places the compiler 
version is used, and in others the ATL version is used.
. The code fails to link because implementations of two inline ATL functions 
are not defined.

What version of FireBreath are you using? On what operating system and browsers?
1.3.2, Windows XP, Visual Studio 2008, Windows Driver Kit 7.1

Please provide any additional information below.
All #ifdef lines need to be changed to use ATL version numbers.
Also, the implementations for __AllocStdCallThunk and __FreeStdCallThunk need 
to be defined.

I have attached my amended copies of axutil.h and axutil.cpp, which compile and 
run without any problems on my configuration.

Original issue reported on code.google.com by Steven.K...@gmail.com on 16 Dec 2010 at 8:59

Attachments:

GoogleCodeExporter commented 9 years ago
wow...I *still* missed a few spots, huh?  I could have sworn that I changed all 
of those... maybe it's in another fork somewhere and didn't quite make it to 
the release :-(

I might have to put out a 1.3.3

I am concerned about adding the __AllocStdCallThunk() function; others have 
been able to get it working without that.  I'll have to install vs express 2008 
and test with that.  it'll probably be next week, however.

Original comment by richarda...@gmail.com on 16 Dec 2010 at 4:22

GoogleCodeExporter commented 9 years ago
The allocstdcallthunk it turns out is an additional link library to add.   If 
you pull down the 1.4 beta (the trunk, master branch) it should all work now 
with visual studio express 2008 and 2010 without even needing to update 
directories in visual studio, given that WinDDK is installed in one of the 
standard locations (default is C:\winddk).

That should take care of all link errors as well.

Original comment by richarda...@gmail.com on 27 Dec 2010 at 9:38