Issue 14580 - Warning: nTime hides Time::nTime.
Summary: Warning: nTime hides Time::nTime.
Status: CLOSED FIXED
Alias: None
Product: Build Tools
Classification: Code
Component: code (show other issues)
Version: OOo 1.1 Beta2
Hardware: Sun Solaris
: P5 (lowest) Trivial (vote)
Target Milestone: ---
Assignee: foskey
QA Contact: issues@tools
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-05-18 09:29 UTC by pavel
Modified: 2003-05-20 06:03 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 pavel 2003-05-18 09:29:55 UTC 
Hi,

inc/tools/time.hxx contains

78:        long                    nTime;
...
149:        void setTime( UINT32 nTime ) { SetTime( nTime ); }

some compilers warn about this:

"/oo/BuildDir/ooo_11beta2_src/solver/644/unxsols4.pro/inc/tools/time.hxx", line
149: Warning: nTime hides Time::nTime.

Could we change the name of the argument to prevent this warning (to e.g.
newTime)? This file is included in *many* other files and this warning is
printed really often.
Comment 1 foskey 2003-05-19 14:50:36 UTC 
This has been changed to _ntime on the beta2 tag.
Comment 2 pavel 2003-05-19 16:00:58 UTC 
reopen to reassign to Ken.
Comment 3 pavel 2003-05-19 16:02:16 UTC 
Hi Ken,

it is not fixed yet:

pavel@snowwhite:~/Tmp/ooo_11beta2_src/tools/inc> cat CVS/Tag 
Tcws_srx644_ooo11beta2
pavel@snowwhite:~/Tmp/ooo_11beta2_src/tools/inc> cvs -z9 up time.hxx 
pavel@snowwhite:~/Tmp/ooo_11beta2_src/tools/inc> grep setTime time.hxx
        void setTime( UINT32 nTime ) { SetTime( nTime ); }

You forgot to commit?
Comment 4 foskey 2003-05-20 00:05:56 UTC 
Lets try that again, now applied properly.
Comment 5 pavel 2003-05-20 06:03:34 UTC 
Thanks.