Re: [QuantLib-svn] SF.net SVN: quantlib:[16124] trunk/QuantLib

Posted by Ferdinando Ametrano-4 on
URL: http://quantlib.414.s1.nabble.com/Re-QuantLib-svn-SF-net-SVN-quantlib-16124-trunk-QuantLib-tp12693.html

On Wed, Apr 8, 2009 at 5:50 PM,  <[hidden email]> wrote:

> Revision: 16124
> [...]
> Log Message:
> -----------
> More MakeSchedule setters instead of anonymous constructor parameters
> [...]
> +        BusinessDayConvention terminationDateConvention;
> +        // if set explicitly, we use it;
> +        if (terminationDateConvention_) {
> +            terminationDateConvention = *terminationDateConvention_;
> +        } else {
> +            // it equals the convention for all other dates.
> +            terminationDateConvention = convention;
> +        }

if not set I would prefer Schedule to default to Unadjusted for the
termination date, as per ISDA standard where it states that
termination date should always be considered unadjusted unless
explicitly specified

> +        BusinessDayConvention convention;
> +        // if a convention was set, we use it.
> +        if (convention_) {
> +            convention = *convention_;
> +        } else {
> +            if (!calendar_.empty()) {
> +                // ...if we set a calendar, we probably want it to be used;
> +                convention = Following;
> +            } else {
> +                // if not, we don't care.
> +                convention = Unadjusted;
> +            }
> +        }

even here I would prefer Unadjusted. In this case I have a weaker
argument: Schedule is most often used to calculate accrual dates
(derived payment dates will be adjusted anyway). YMMV

ciao -- Nando

------------------------------------------------------------------------------
This SF.net email is sponsored by:
High Quality Requirements in a Collaborative Environment.
Download a free trial of Rational Requirements Composer Now!
http://p.sf.net/sfu/www-ibm-com
_______________________________________________
QuantLib-dev mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/quantlib-dev