<html><body><div style="font-family: times new roman, new york, times, serif; font-size: 12pt; color: #000000"><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><div><blockquote class=""><div class="">On 6 Sep 2016, at 15:07 PM, Frediano Ziglio <<a href="mailto:fziglio@redhat.com" class="" target="_blank">fziglio@redhat.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class=""><div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;" class=""><blockquote style="border-left-width: 2px; border-left-style: solid; border-left-color: rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica, Arial, sans-serif; font-size: 12pt;" class=""><div dir="ltr" class=""><div class="gmail_extra">On Mon, Sep 5, 2016 at 9:26 PM, Frediano Ziglio <span dir="ltr" class=""><<a href="mailto:fziglio@redhat.com" target="_blank" class="">fziglio@redhat.com</a>></span> wrote:<br class=""><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Why the rationale was removed?<br class=""><br class=""></blockquote><div class="">By mistake, i'll add it back in the next version. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">By the way this was a workaround for a VS bug, get Visual Studio 2015 Update 1 (actually we are at update 3)<br class=""></blockquote><div class="">True, however microsoft recommends using visual c++ compiler profile as it is different from the c compiler profile.</div></div></div></div></blockquote><div class=""><br class=""></div><div class="">Yes, especially if you don't know what you are doing and you don't have a acceptable test suite.<br class=""></div><div class="">I hope that they all don't apply to us.<br class=""></div></div></div></div></blockquote><div><br class=""></div><div><br class=""></div><div>Hi Frediano,</div><div><br class=""></div><div>Actually using Microsoft C++ compiler for C files is recommended because it performs</div><div>more excessive static checks and produces warnings regarding more potential problematic cases.</div><div>And this is definitely a good thing.</div><div><br class=""></div><div>~Dmitry</div></div></blockquote><div><div>Yes... C++ does more strictly checks. This does not means that all C programmers are moving to a C++ compiler!</div><div><br></div><div>Frediano</div><div><br></div></div><blockquote style="border-left:2px solid #1010FF;margin-left:5px;padding-left:5px;color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;"><div><blockquote class=""><div class=""><div class=""><div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;" class=""><div class=""><br class=""></div><div class="">Frediano<br class=""></div><div class=""><br class=""></div><blockquote style="border-left-width: 2px; border-left-style: solid; border-left-color: rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica, Arial, sans-serif; font-size: 12pt;" class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex" class="gmail_quote"><br class=""> Nack<br class=""><br class=""> Frediano<br class=""><br class=""> ><br class=""> > Signed-off-by: Sameeh Jubran <<a href="mailto:sameeh@daynix.com" target="_blank" class="">sameeh@daynix.com</a>><br class=""> > ---<br class=""> >  qxldod/mspace.c               | 2437<br class=""> >  -----------------------------------------<br class=""> >  qxldod/mspace.cpp             | 2437<br class=""> >  +++++++++++++++++++++++++++++++++++++++++<br class=""> >  qxldod/qxldod.vcxproj         |    2 +-<br class=""> >  qxldod/qxldod.vcxproj.filters |    2 +-<br class=""> >  4 files changed, 2439 insertions(+), 2439 deletions(-)<br class=""> >  delete mode 100755 qxldod/mspace.c<br class=""> >  create mode 100755 qxldod/mspace.cpp<br class=""> ><br class=""> > diff --git a/qxldod/mspace.c b/qxldod/mspace.c<br class=""> > deleted file mode 100755<br class=""> > index d0ba123..0000000<br class=""> > --- a/qxldod/mspace.c<br class=""> > +++ /dev/null<br class=""> > @@ -1,2437 +0,0 @@<br class=""> > -// based on dlmalloc from Doug Lea<br class=""> > -<br class=""> > -<br class=""> > -// quote from the Doug Lea original file<br class=""> > -    /*<br class=""> > -      This is a version (aka dlmalloc) of malloc/free/realloc written by<br class=""> > -      Doug Lea and released to the public domain, as explained at<br class=""> > -      <a href="http://creativecommons.org/licenses/publicdomain" rel="noreferrer" target="_blank" class="">http://creativecommons.org/licenses/publicdomain</a>.  Send questions,<br class=""> > -      comments, complaints, performance data, etc to <a href="mailto:dl@cs.oswego.edu" target="_blank" class="">dl@cs.oswego.edu</a><br class=""> > -<br class=""> > -    * Version 2.8.3 Thu Sep 22 11:16:15 2005  Doug Lea  (dl at gee)<br class=""> > -<br class=""> > -       Note: There may be an updated version of this malloc obtainable at<br class=""> > -               <a href="ftp://gee.cs.oswego.edu/pub/misc/malloc.c" rel="noreferrer" target="_blank" class="">ftp://gee.cs.oswego.edu/pub/misc/malloc.c</a><br class=""> > -             Check before installing!<br class=""> > -    */<br class=""> > -<br class=""> > -<br class=""> > -#include <ntddk.h><br class=""> > -<br class=""> > -#include "mspace.h"<br class=""> > -<br class=""> > -#pragma warning( disable : 4146 ) /* no "unsigned" warnings */<br class=""> > -<br class=""> > -#define MALLOC_ALIGNMENT ((size_t)8U)<br class=""> > -#define USE_LOCKS 0<br class=""> > -#define malloc_getpagesize ((size_t)4096U)<br class=""> > -#define DEFAULT_GRANULARITY malloc_getpagesize<br class=""> > -#define MAX_SIZE_T (~(size_t)0)<br class=""> > -#define MALLOC_FAILURE_ACTION<br class=""> > -#define MALLINFO_FIELD_TYPE size_t<br class=""> > -#define FOOTERS 0<br class=""> > -#define INSECURE 0<br class=""> > -#define PROCEED_ON_ERROR 0<br class=""> > -#define DEBUG 0<br class=""> > -#define ABORT_ON_ASSERT_FAILURE 1<br class=""> > -#define ABORT(user_data) abort_func(user_data)<br class=""> > -#define USE_BUILTIN_FFS 0<br class=""> > -#define USE_DEV_RANDOM 0<br class=""> > -#define PRINT(params) print_func params<br class=""> > -<br class=""> > -<br class=""> > -#define MEMCPY(dest, src, n) RtlCopyMemory(dest, src, n)<br class=""> > -#define MEMCLEAR(dest, n) RtlZeroMemory(dest, n)<br class=""> > -<br class=""> > -<br class=""> > -#define M_GRANULARITY        (-1)<br class=""> > -<br class=""> > -void default_abort_func(void *user_data)<br class=""> > -{<br class=""> > -    for (;;);<br class=""> > -}<br class=""> > -<br class=""> > -void default_print_func(void *user_data, char *format, ...)<br class=""> > -{<br class=""> > -}<br class=""> > -<br class=""> > -static mspace_abort_t abort_func = default_abort_func;<br class=""> > -static mspace_print_t print_func = default_print_func;<br class=""> > -</blockquote></div></div></div></blockquote></div></div></div></blockquote></div></blockquote></div></body></html>