[PATCH-INTEGRATED] Cleanup, optimize, etc

avindra · (This post was last modified: 07-19-2010, 07:19 AM by avindra.)

Nothing too fancy or overly exciting here. I worked on killing warnings from the compiler, there's still a lot but this takes care of many of them.

Changes:
-mod 2 > & 1
-remove redundant code, header
-converted block of code into switch statement
-use else to avoid unnecessary evaluation
-lots of little code "compression"

DX11:
-Remove calls which were being made twice (through Renderer::Shutdown())
-Call the more destructive functions to ensure no leakage.
-Reorder shutdown procedure to resemble DX9's.

Lectrode · (This post was last modified: 07-18-2010, 07:30 PM by Lectrode.)

Code:

===================================================================

--- Source/Plugins/Plugin_VideoDX11/Plugin_VideoDX11.vcproj    (revision 5897)

+++ Source/Plugins/Plugin_VideoDX11/Plugin_VideoDX11.vcproj    (working copy)

@@ -836,6 +836,18 @@

                 >

             </File>

         </Filter>

+        <Filter

+            Name="IntelliSense"

+            >

+            <File

+                RelativePath="..\..\..\..\..\..\..\Program Files (x86)\Microsoft DirectX SDK (June 2010)\Include\D3D11.h"

+                >

+            </File>

+            <File

+                RelativePath="..\..\..\..\..\..\..\Program Files (x86)\Microsoft DirectX SDK (June 2010)\Include\D3DX11.h"

+                >

+            </File>

+        </Filter>

         <File

             RelativePath=".\Src\Globals.h"

             >

Is that supposed to be like that? Not everyone has "Program Files (x86)"

Nice patch btw

avindra · 07-19-2010, 12:21 AM #3

"Not everyone has "Program Files (x86)""

Yeah I was thinking about that, but then I couldn't figure out how to do the pathing that works for both.

Like, if you use %PROGRAMFILES%, it gives C:\Program Files, and not the x86 version. :/

**neobrain** · 07-19-2010, 03:15 AM #4

uhm, what is that vcproj change supposed to do anyway?

avindra · 07-19-2010, 04:22 AM #5

It should show all the IntelliSense information for DirectX. Am I the only one who's not getting the IntelliSense working?

**neobrain** · 07-19-2010, 04:35 AM #6

Yes you are... ^^
At least it's working fine for me.

blackJade · 07-19-2010, 04:45 AM #7

How about using %DXSDK_DIR%\Include\D3DX11.h?