Skip to content

Minor updates; make 0 warnings#4

Open
greenozon wants to merge 1 commit into
onlyone-rnd:mainfrom
greenozon:main
Open

Minor updates; make 0 warnings#4
greenozon wants to merge 1 commit into
onlyone-rnd:mainfrom
greenozon:main

Conversation

@greenozon
Copy link
Copy Markdown

@greenozon greenozon commented Aug 13, 2025

hello friend!
long time no see
now I know here is your GH home :)

so some notes on the changes:

  1. <> are for system headers only, so lets ease it:

#include <MemoryDumpTo.h>
->
#include "MemoryDumpTo.h"

  1. let compiler do the dirty job:

__stosb
->
{}

This is the cleanest and most portable way. It tells the compiler to zero-initialize the entire structure or array.

  1. make 0 warnings!

before


1>MemoryDumpTo.cpp
1>C:\Dev\VS2019\VC\Tools\MSVC\14.29.30133\include\vector(1296,1): warning C4530: C++ exception handler used, but unwind semantics are not enabled. Specify /EHsc
1>C:\Dev\VS2019\VC\Tools\MSVC\14.29.30133\include\vector(1286): message : while compiling class template member function 'void std::vector<BridgeCFNodeList,std::allocator<BridgeCFNodeList>>::_Reallocate_exactly(const unsigned __int64)'
1>C:\Dev\VS2019\VC\Tools\MSVC\14.29.30133\include\vector(1363): message : see reference to function template instantiation 'void std::vector<BridgeCFNodeList,std::allocator<BridgeCFNodeList>>::_Reallocate_exactly(const unsigned __int64)' being compiled

after

Rebuild started...
1>------ Rebuild All started: Project: MemoryDumpTo, Configuration: Debug x64 ------
1>CreateTableText.cpp
1>MemoryDumpTo.cpp
1>plugin.cpp
1>Generating Code...
1>   Creating library C:\Prg\MemoryDumpTo\x64\Debug\MemoryDumpTo.lib and object C:\Prg\MemoryDumpTo\x64\Debug\MemoryDumpTo.exp
1>MemoryDumpTo.vcxproj -> C:\Prg\MemoryDumpTo\x64\Debug\MemoryDumpTo.dp64
========== Rebuild All: 1 succeeded, 0 failed, 0 skipped ==========

Note: added Exception handling support as the dependent libs are using std::vector/etc -> which are in need of exceptions support...

DllMain:
An entry point was exported from a DLL. The entry point of a DLL does not need to be exported.

->
removed export directive

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant