Issue 17607 - IRIX: tools/fstat.hxx should include stdarg (or varargs)
Summary: IRIX: tools/fstat.hxx should include stdarg (or varargs)
Status: CLOSED WONT_FIX
Alias: None
Product: porting
Classification: Code
Component: code (show other issues)
Version: OOo 1.1 RC
Hardware: SGI IRIX
: P3 Trivial (vote)
Target Milestone: AOO PleaseHelp
Assignee: sander_traveling
QA Contact: issues@porting
URL:
Keywords:
Depends on:
Blocks: 809
  Show dependency tree
 
Reported: 2003-07-30 07:16 UTC by nickb
Modified: 2010-03-29 20:47 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description nickb 2003-07-30 07:16:48 UTC 
Compiling tools/source/fsys/fstat.cxx fails because it includes fstat.hxx, which
references va_list, without including varargs or stdargs.
Comment 1 nickb 2003-07-30 07:53:06 UTC 
Hmmm, seems that is the wrong solution.
#include <cstdarg> is correct, but that should already be there... and 
va_list is in std namespace. So should be std::va_list

svtools/inc/itemset.hxx - has the same problem
Comment 2 nickb 2003-07-30 09:10:33 UTC 
vcl/unx/source/app/i18n_wrp.cxx - needs to include <cstdarg> and then
all the va_lists need to have a namespace attached.
Comment 3 nickb 2003-07-30 09:12:08 UTC 
vcl/unx/source/app/i18n_wrp.cxx - also needs two arguments for va_start
Comment 4 jens-heiner.rechtien 2003-07-30 16:18:50 UTC 
Hi Nick,

Some platforms won't work if we prepend std:: before va_list, even if
we include <cstdarg>. Solaris even usually needs both <stdarg.h> and
<cstdarg>. It's a mess. Does it work if you have both includes?
Comment 5 nickb 2003-07-31 00:27:04 UTC 
Yes, that works! :-)
Comment 6 nickb 2003-08-01 08:19:04 UTC 
svtools/inc/itemset.hxx - needs this fix aswell
Comment 7 Martin Hollmichel 2003-08-07 15:56:58 UTC 
set target milestone to 1.1.1
Comment 8 Martin Hollmichel 2003-10-17 08:21:42 UTC 
reassigned.
Comment 9 sander_traveling 2004-03-25 11:03:08 UTC 
1.1.2->2.0
Comment 10 caolanm 2010-03-29 20:47:34 UTC 
per issue 106845 sb removed the partial irix port, so this doesn't make sense in
isolation anymore106845
Comment 11 caolanm 2010-03-29 20:47:50 UTC 
closing