Closed
Bug 150114
Opened 23 years ago
Closed 17 years ago
RFE: Auto-detect rtti and set define
Categories
(SeaMonkey :: Build Config, enhancement)
SeaMonkey
Build Config
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: roland.mainz, Unassigned)
Details
Attachments
(3 files)
641 bytes,
patch
|
Details | Diff | Splinter Review | |
875 bytes,
text/plain
|
Details | |
810 bytes,
text/plain
|
Details |
RFE: Need CPP symbol which is set when RTTI is available
Example:
I'd like to add some code in the print modules which checks whether a object
passed as argument is from a given class (this is highly usefull to hunt-down
some crashers in print preview and printing SVG). Since RTTI is AFAIK not
available in all C++ compilers and not allowed in normal Mozilla code I need a
symbol which is set only when RTTI is available, e.g.
-- snip --
#ifdef HAS_RTTI
NS_ASSERTION(x instanceof nsRenderingContextPS, "Passed renderingcontext is not
a nsRenderingContextPS");
#endif /* HAS_RTTI */
-- snip --
Comment 1•23 years ago
|
||
Reporter | ||
Comment 2•23 years ago
|
||
cls wrote:
> Created an attachment (id=86970)
> define HAVE_CPP_RTTI iff --enable-cpp-rtti is set
Mhh, the idea was more to define the flag if the compiler supports RTTI (I
assume that some vendor compilers like Sun Workshop support this out-of-the-box)
...
Comment 3•23 years ago
|
||
That's different. You want rtti to be auto-detected. Which still wouldn't
help you as you'd have to explicitly enable it with this switch anyway.
Reporter | ||
Comment 4•23 years ago
|
||
seawood wrote:
> That's different. You want rtti to be auto-detected.
Yes, that's what I want... :)
> Which still wouldn't
> help you as you'd have to explicitly enable it with this switch anyway
Really ? Do we disable RTTI in Sun Workshop ? I thought the --enable-cpp-rtti is
for gcc only...
Reporter | ||
Comment 5•23 years ago
|
||
Comment 6•23 years ago
|
||
Well, we're supposed to turn rtti and exceptions off everywhere as it supposedly
makes a difference in generated size. --enable-cpp-rtti currently only works
for gcc though I'm sure we'd hook up other compilers if we knew the flags.
Priority: -- → P5
Summary: RFE: Need CPP symbol which is set when RTTI is available → RFE: Auto-detect rtti and set define
Target Milestone: --- → Future
Reporter | ||
Comment 7•23 years ago
|
||
cls wrote:
> --enable-cpp-rtti currently only works for gcc though I'm sure we'd hook up
> other compilers if we knew the flags.
Well, AFAIK you cannot turn it off for Sun Workshop unless you want to run in
"compatibility" mode (something we IMHO do not want... =:-) for far older
versions of Sun Workshop (4.x) ...
Reporter | ||
Comment 8•23 years ago
|
||
Small test tool, compiles on Sun Workshop 7 FCS and gcc 2.95.1 (specifying
-fno-rtti for gcc will abort the compiler with an error).
Comment 9•22 years ago
|
||
Mass reassign to new default build assignee
Assignee: seawood → mozbugs-build
Priority: P5 → --
Mass reassign of Build/Config bugs to Leaf.
Assignee: mozbugs-build → leaf
Target Milestone: Future → ---
Updated•20 years ago
|
Assignee: leaf → cmp
Updated•20 years ago
|
Product: Browser → Seamonkey
Comment 11•19 years ago
|
||
Mass reassign of open bugs for chase@mozilla.org to build@mozilla-org.bugs.
Assignee: chase → build
Comment 12•19 years ago
|
||
Mass re-assign of bugs that aren't on the build team radar, so bugs assigned to build@mozilla-org.bugs reflects reality.
If there is a bug you really think we need to be looking at, please *email* build@mozilla.org with a bug number and explanation.
Assignee: build → nobody
Updated•17 years ago
|
QA Contact: granrosebugs → build-config
Updated•17 years ago
|
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•