Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Misc fixes #100

Merged
merged 8 commits into from Feb 28, 2014
Merged

Misc fixes #100

merged 8 commits into from Feb 28, 2014

Conversation

Tilka
Copy link
Member

@Tilka Tilka commented Feb 23, 2014

I compiled Dolphin with clang and went from there.

0 << 8, // 24 bits
2 << 8, // 53 bits
3 << 8, // 64 bits
3 << 8, // mask

This comment was marked as off-topic.

This comment was marked as off-topic.

@Tilka
Copy link
Member Author

Tilka commented Feb 23, 2014

There, I also renamed the rounding mode table.

@Tilka
Copy link
Member Author

Tilka commented Feb 23, 2014

I consider this merge-ready.

@@ -134,6 +134,7 @@ static void DSPCore_FreeMemoryPages()
FreeMemoryPages(g_dsp.iram, DSP_IRAM_BYTE_SIZE);
FreeMemoryPages(g_dsp.dram, DSP_DRAM_BYTE_SIZE);
FreeMemoryPages(g_dsp.coef, DSP_COEF_BYTE_SIZE);
g_dsp.irom = g_dsp.iram = g_dsp.dram = g_dsp.coef = NULL;

This comment was marked as off-topic.

@Tilka
Copy link
Member Author

Tilka commented Feb 23, 2014

Don't merge this yet, I need to rethink TextureConversionShader.cpp

@Tilka
Copy link
Member Author

Tilka commented Feb 24, 2014

Actually, do merge it like this. While not required, making all related functions in TextureConversionShader.cpp take a pointer reference should be more readable. I think it's worth violating our "const references only" rule here for the time being.

@Tilka
Copy link
Member Author

Tilka commented Feb 27, 2014

OK, I squashed the fix commit. Ready for merging.

@@ -23,8 +23,9 @@
#include "Core/HW/WiimoteEmu/Attachment/Turntable.h"
#include "Core/HW/WiimoteReal/WiimoteReal.h"


#ifdef _WIN32
inline double round(double x) { return (x-floor(x))>0.5 ? ceil(x) : floor(x); } //because damn MSVSC doesen't comply to C99

This comment was marked as off-topic.

- mostly remove unused variables
- rename some generic JIT identifiers
- remove unused variables
- reduce the scope where it makes sense
- correct limits (did you know that strcat()'s last parameter does not
  include the \0 that is always added?)
- set some free()'d pointers to NULL
1. It's not correct, it rounds x.5 down to x.
2. It is visible across the whole project (!).
3. VS2013 finally supports this:
   http://msdn.microsoft.com/en-us/library/dn353646.aspx
   http://msdn.microsoft.com/en-us/library/dn329049.aspx
delroth added a commit that referenced this pull request Feb 28, 2014
@delroth delroth merged commit bc844a1 into dolphin-emu:master Feb 28, 2014
@Tilka Tilka deleted the misc_fixes branch March 1, 2014 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
6 participants