Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | min and max macro's in solar.h over-ride std::min and std::max | ||
---|---|---|---|
Product: | porting | Reporter: | issues@www <issues> |
Component: | documentation | Assignee: | jens-heiner.rechtien |
Status: | CLOSED FIXED | QA Contact: | issues@www <issues> |
Severity: | Trivial | ||
Priority: | P2 | CC: | issues, jayvdb |
Version: | 632 | ||
Target Milestone: | not determined | ||
Hardware: | SGI | ||
OS: | IRIX | ||
Issue Type: | DEFECT | Latest Confirmation in: | --- |
Developer Difficulty: | --- | ||
Issue Depends on: | |||
Issue Blocks: | 1209 | ||
Attachments: |
Description
issues@www
2001-07-13 06:30:51 UTC
Created attachment 374 [details]
tools/inc/solar.h : Patch to remove min/max macro's from c++
Created attachment 381 [details]
vcl/unx/source/gdi/xfont.cxx : Patch to use std::min
vcl is the first module which breaks on this change to solar.h
vcl is still broken and attachment 381 [details] should not be pushed in. Damn why cant
they be simple one line fixes.
Created attachment 382 [details]
vcl/unx/source/gdi/xfont.cxx : Patch to build on IRIX, needs further testing
I am not even going to pretend that 382 is correct, however it does build and pass checkdll. Further changes will be needed on other platforms which I can not build. vcl/mac/source/app/NoSharMain.cxx: applZone->moreMast = (short) min(numMasterPointersDesired, SHRT_MAX); vcl/win/source/app/salinfo.cxx: + max (pSection->SizeOfRawData, pSection->Misc.VirtualSize); There is also a good chance that there are other uses of the min/max macro's within c++ code which have not come within reach of my compiler. Sander .. I have just completed a build with this patch ... and there were no other affected files (on IRIX). Heiner, can please take care of these ? Accepted The only way to really fix this problem is to remove the min()/max() macros from solar.h and use the std::min()/std::max() templates everywhere. Wouldnt this result in min()/max() not being available in files which are not c++? fixed in SRC641 g closed |