Fixes wxWindows build breaks.

http://www.digitalmars.com/compiler.htm

"Walter" <walter@digitalmars.com> wrote in news:c16pbh$2d3a$1@digitaldaemon.com:

> Fixes wxWindows build breaks.

Thanks for all of your support.

> http://www.digitalmars.com/compiler.htm

MAPI_LONG_MSGID definition in win32/mapi.h cause warning:

/* #define MAPI_LONG_MSGID  0x00004000  /* allow 512 char returned ID   */
                                       ^
h:\dmc\BIN\..\include\win32\mapi.h(161) : Warning 3: comments do not nest


ABX

In article <c16pbh$2d3a$1@digitaldaemon.com>, Walter says...
>
>Fixes wxWindows build breaks.
>
>http://www.digitalmars.com/compiler.htm
>
>
>

Dear Walter,

Thanks for a new version of compiler!

Please check following code, it looks like a compiler bug:

//--- "nested_friend.cpp":

class C
{
public:
C() {};
};

class A: public C
{
private:
class B: public C
{
protected:
friend class A;
A* m_pThis;

public:
B( A & a ): m_pThis(&a) {}
int IncCnt() { return ++( m_pThis->m_Counter ); }
int DecCnt() { return --( m_pThis->m_Counter ); }
};

public:
A(): m_Counter(0) {}

protected:
friend class B;
int m_Counter;

private:

};

//--- eof "nested_friend.cpp"

Compilation result:

dmc -v -c nested_friend.cpp
scppn -Aa- -6 -v nested_friend.cpp
Digital Mars C/C++ Compiler Version 8.40.2n
Copyright (C) Digital Mars 2000-2004.  All Rights Reserved.
Written by Walter Bright
www.digitalmars.com
'nested_friend.cpp'
class __nt_context
class __eh_cv
class C
C::C
class A
class A::B
friend class B;
^
nested_friend.cpp(27) : Error: cannot change access of member A::B
A::B::B
A::B::IncCnt
A::B::DecCnt
A::A

--- errorlevel 1

Regards...

Sergey Gutnikov

In article <c16pbh$2d3a$1@digitaldaemon.com>, Walter says...
>
>Fixes wxWindows build breaks.
>
>http://www.digitalmars.com/compiler.htm
>
>
>

Dear Walter,

Please check following example, it causes run-time error in debug version when linked under DMC 8.40 with SNND.lib. I have prepared it as console application, but similar error appear in MFC GUI application (which works fine with version 8.37).

//--- "dbg_heap.cpp":
#include <stdio.h>
#define _CRTDBG_MAP_ALLOC 1
#include <crtdbg.h>

template<class TYPE>
class TArray
{
TYPE    * m_pBuf;

unsigned long   m_Size;
unsigned long   m_PgSize;

public:

TArray( unsigned long nInitSize = 0, unsigned long nPgSize = 16 );
~TArray();

void  Allocate( unsigned long nSize );
void  Free();
};

template <class TYPE>
void TArray<TYPE>::Free()
{
if ( m_Size > 0 )
{
delete [] m_pBuf;
//^^^^^^^^^^^^^ *** Error here!
m_Size = 0;
}
};

template <class TYPE>
void TArray<TYPE>::Allocate( unsigned long nSize )
{
if ( nSize > 0 )
{
m_pBuf = new TYPE [nSize];
m_Size = nSize;
}
};

template <class TYPE>
TArray<TYPE>::TArray( unsigned long nInitSize,
unsigned long nPgSize )
{
if ( nPgSize < 16 )
{
nPgSize = 16;
}
m_PgSize = nPgSize;
if ( nInitSize == 0 )
{
m_pBuf = NULL;
m_Size = 0;
}
else
{
Allocate( nInitSize );
}
};

template <class TYPE>
TArray<TYPE>::~TArray()
{
Free();
};


typedef struct tree_item
{
void * ti;

tree_item();

} tree_item_t;

tree_item::tree_item() : ti(NULL)
{
}

typedef TArray<tree_item_t> TTreeItemArray_t;



void test( void )
{
TTreeItemArray_t hChildItems( 64, 64 );

}

int main()
{
_CrtSetReportMode( _CRT_WARN, _CRTDBG_MODE_FILE );
_CrtSetReportFile( _CRT_WARN, _CRTDBG_FILE_STDOUT );
_CrtSetReportMode( _CRT_ERROR, _CRTDBG_MODE_FILE );
_CrtSetReportFile( _CRT_ERROR, _CRTDBG_FILE_STDOUT );
_CrtSetReportMode( _CRT_ASSERT, _CRTDBG_MODE_FILE );
_CrtSetReportFile( _CRT_ASSERT, _CRTDBG_FILE_STDOUT );

printf( "test()\n" );
test();
printf( "done...\n" );

_CrtDumpMemoryLeaks();
}

//--- eof "dbg_heap.cpp"

The example has been compiled with following flags:

dmc -cpp -Ae -r -mn -C -WA -S -a8 -gh -D_DEBUG=1 -NLsnnd dbg_heap.cpp


Thanks in advance.

Regards...

Sergey Gutnikov

In article <c16pbh$2d3a$1@digitaldaemon.com>, Walter says...
>
>Fixes wxWindows build breaks.
>
>http://www.digitalmars.com/compiler.htm
>
>
>

Dear Walter,

(Sorry, it's me again...)

Possibly a bug:

The DMC version 8.40 does not embed to .obj-file includelib statements from precompiled headers.

Thanks in advance,

Sergey Gutnikov

Walter,  you must be working 24 hours a day..........



"Walter" <walter@digitalmars.com> wrote in message news:c16pbh$2d3a$1@digitaldaemon.com...
> Fixes wxWindows build breaks.
>
> http://www.digitalmars.com/compiler.htm
>
>
>

"W³odzimierz Skiba" <abx@abx.art.pl> wrote in news:c1cuno$1c2n$2@digitaldaemon.com:
> > http://www.digitalmars.com/compiler.htm
> 
> MAPI_LONG_MSGID definition in win32/mapi.h cause warning:
> 
> /* #define MAPI_LONG_MSGID  0x00004000  /* allow 512 char returned ID
>  */
>                                        ^
> h:\dmc\BIN\..\include\win32\mapi.h(161) : Warning 3: comments do not
> nest

Just a reminder that another DMC beta did not fixed it.

Regards, ABX

W³odzimierz Skiba wrote:
> "W³odzimierz Skiba" <abx@abx.art.pl> wrote in
> news:c1cuno$1c2n$2@digitaldaemon.com: 
> 
>>>http://www.digitalmars.com/compiler.htm
>>
>>MAPI_LONG_MSGID definition in win32/mapi.h cause warning:
>>
>>/* #define MAPI_LONG_MSGID  0x00004000  /* allow 512 char returned ID 
>> */                                       ^
>>h:\dmc\BIN\..\include\win32\mapi.h(161) : Warning 3: comments do not
>>nest 
> 
> 
> Just a reminder that another DMC beta did not fixed it.
> 
> Regards, ABX
Hey  it is a WARNING...
/* ... */ comments do not nest!
So just change the M$ include file (mapi.h) and solve the problem that way if it is that important to you.

Jan


-- 
ManiaC++
Jan Knepper

But as for me and my household, we shall use Mozilla...
www.mozilla.org

Jan Knepper <jan@smartsoft.us> wrote in news:cd4cnp$jp9$1@digitaldaemon.com:
> > Just a reminder that another DMC beta did not fixed it.
> > 
> > Regards, ABX
>
> Hey  it is a WARNING...
> /* ... */ comments do not nest!

I know.

> So just change the M$ include file (mapi.h) and solve the problem that way if it is that important to you.

Being wxWidgets developer responsible for delivering wxWidgets in warning free form under all free compilers I care about quality delivered to the users. I can't change mapi.h in user's instalations. I do not care for such a warning for myself.

ABX

W³odzimierz Skiba wrote:
> Jan Knepper <jan@smartsoft.us> wrote in news:cd4cnp$jp9$1@digitaldaemon.com:
> 
>>>Just a reminder that another DMC beta did not fixed it.
>>>
>>>Regards, ABX
>>
>>Hey  it is a WARNING...
>>/* ... */ comments do not nest!
> 
> 
> I know.
> 
> 
>>So just change the M$ include file (mapi.h) and solve the problem that way if it is that important to you.
> 
> 
> Being wxWidgets developer responsible for delivering wxWidgets in warning free form under all free compilers I care about quality delivered to the users. I can't change mapi.h in user's instalations. I do not care for such a warning for myself.
> 
> ABX
I can understand that, but that warning is not something we control.
Just add a release note that if the users want this warning to disappear to change this particular line in the M$ header file (mapi.h).
You also might want to report it to M$.

Jan


-- 
ManiaC++
Jan Knepper

But as for me and my household, we shall use Mozilla...
www.mozilla.org