Home » U++ Library support » U++ Core » FIX: UPP::Scan returns int64 for INT_V
Re: FIX: UPP::Scan returns int64 for INT_V [message #32467 is a reply to message #32460] |
Thu, 19 May 2011 09:35  |
 |
kohait00
Messages: 939 Registered: July 2009 Location: Germany
|
Experienced Contributor |
|
|
you're right as usual 
the int64 return needs to be addressed as well.
in this case i'd vote for separating the ConvertInt::Scan and ConvertInt64::Scan, so they both deal respectively with their proper minval maxval, so we have the INT_MAX/INT_MIN issue only in constructors. i think that's be more clean...
so i'd be
Value ConvertInt::Scan() { /***/ return int(v); }
Value ConvertInt64::Scan() { /***/ return v; }
i also thought of swapping derive order, to have ConvertInt->ConvertInt64. this would make possible to have a short ConvertInt::Scan like
Value ConvertInt::Scan()
{
Value v = ConvertInt64::Scan();
if(!IsErrorValue(v)) return int(m);
return v;
}
maybe that's an even cleaner option..
BTW: ConvertInt64 daysichain stuff returns ConvertInt& instead ConvertInt64&
[Updated on: Thu, 19 May 2011 09:36] Report message to a moderator
|
|
|
 |
|
FIX: UPP::Scan returns int64 for INT_V
By: kohait00 on Wed, 13 April 2011 23:01
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
By: kohait00 on Sun, 17 April 2011 14:57
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
By: mirek on Sun, 17 April 2011 21:23
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
By: kohait00 on Mon, 18 April 2011 10:46
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
By: mirek on Tue, 19 April 2011 07:28
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
By: kohait00 on Wed, 27 April 2011 14:07
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
By: mirek on Fri, 29 April 2011 20:17
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
By: mirek on Fri, 06 May 2011 09:58
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
By: mirek on Fri, 13 May 2011 10:52
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
By: mirek on Tue, 17 May 2011 18:51
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
By: mirek on Wed, 18 May 2011 22:57
|
 |
|
Re: FIX: UPP::Scan returns int64 for INT_V
|
Goto Forum:
Current Time: Tue Apr 29 23:22:00 CEST 2025
Total time taken to generate the page: 0.00618 seconds
|