Issue 87946

Summary: to-be-DEV300_m6: toolkit's layout.hxx
Product: gsl Reporter: pavel
Component: codeAssignee: AOO issues mailing list <issues>
Status: CONFIRMED --- QA Contact:
Severity: Trivial    
Priority: P3 CC: issues, kurt.zenker, quetschke
Version: DEV300m5   
Target Milestone: OOo 3.x   
Hardware: All   
OS: Windows, all   
Issue Type: DEFECT Latest Confirmation in: ---
Developer Difficulty: ---

Description pavel 2008-04-06 09:47:12 UTC
In svx:

f:/home\pavel\BuildDir\ooo_DEV300_m6_src\solver\300\\inc\layout\layout.hxx(4) : 
error C2220: warning treated as error - no object file generated
f:/home\pavel\BuildDir\ooo_DEV300_m6_src\solver\300\\inc\layout\layout.hxx(4) : 
warning C4668: 'ENABLE_LAYOUT' is not defined as a preprocessor macro, replacing with '0' for 
: warning C4668: 'ENABLE_LAYOUT' is not defined as a preprocessor macro, replacing with '0' for 
post.hxx(4) : warning C4668: 'ENABLE_LAYOUT' is not defined as a preprocessor macro, replacing with 
'0' for '#if/#elif'
dmake:  Error code 2, while making '../../'

ERROR: Error 65280 occurred while making 
Comment 1 pavel 2008-04-06 09:49:15 UTC

Please also add copyright headers...

Comment 2 jcn 2008-04-06 12:07:19 UTC
Thanks for catching this, but I object to such a patch as it is not helpful if
code is built when ENABLE_LAYOUT==0.

I would much rather have ENABLE_LAYOUT always be defined.  I'm kind of
flabbergasted it isn't, because ./configure will define it either 0 or 1.

Is this a Windows build?  Is Windows using an alternative mechanism?
Could you help me with that, I have never built for Windows.

I do not understand how this issue can crop-up now, this code has been
integrated in m2 or m3 already?

Wrt copyright headers (and lots of other fixes, amongst which building
for Windows with --enable-layout switched on), those are in the
"layoutdialogs" CWS.
Comment 3 pavel 2008-04-06 12:14:04 UTC

yes is Windows build. It pop up now, because was broken for a few 
milestones ;-)

What about #if VARIABLE > 0?

There is a long way from configure's ENABLE_LAYOUT to C++ ENABLE_LAYOUT... 

BUT: I do not like this if in the header at all...

Comment 4 clippka 2008-04-06 12:50:18 UTC
I suggest we add the following to



In the cws layouterdialogs we will decide how to implement this gracefully.
Comment 5 clippka 2008-04-07 14:01:32 UTC
looks like sw project has no makefile.pmk

In that case the proposed patch must be added to each of the
directories that fail
Comment 6 pavel 2008-04-07 20:09:29 UTC
In the current m6, while compiling svx/source/dialog/dlgfact.cxx:

In file included from /home/oo/BuildDir/ooo_DEV300_m6_src/svx/source/dialog/zoom.hxx:40,
                 from /home/oo/BuildDir/ooo_DEV300_m6_src/svx/source/dialog/dlgfact.cxx:74:
/home/oo/BuildDir/ooo_DEV300_m6_src/solver/300/ error: 
com/sun/star/awt/XLayoutContainer.hpp: No such file or directory

and that file really is not part of the build tree...

Comment 7 pavel 2008-04-09 21:12:43 UTC
lowering prio.

Works now.

But still has to be cleaned-out as the masterfix is only workaround.

Comment 8 Martin Hollmichel 2009-04-16 09:11:47 UTC
can this issue be closed or get another target ?
Comment 9 thorsten.ziehm 2009-05-18 14:36:04 UTC
OOo 3.1 is released. Please check the issue, if it still exists in OOo 3.1. If
yes, please work on it to get it fixed in one of the next releases. Until then
the issue get the target 3.x.
Comment 10 Rob Weir 2013-07-30 02:37:07 UTC
Reset assignee on issues not touched by assignee in more than 1000 days.