Issue 1688:  Illegal IDL in CosTime module (orb_revision)
Source:  (, )
Nature:  Uncategorized Issue
Severity: 
Summary: Summary: The IDL in module CosTime for the compare_time and
 time_to_interval operations is illegal.  Reference
 http://www.omg.org/archives/experts/msg00890.html
 for further information.
 

Resolution: 
Revised Text: //www.omg.org/archives/experts/msg00890.html
Actions taken:
July 16, 1998: revceived issue
February 23, 1999: closed issue
Discussion: 
  All references to formal/97-12-21.pdf 
End of Annotations:=====
Return-Path: <beckwb@ois.com>
Date: Thu, 16 Jul 1998 15:03:34 -0400 (EDT)
From: Bill Beckwith <bill.beckwith@ois.com>
X-Sender: beckwb@gamma
To: issues@omg.org
cc: orbos@omg.org, orb_revision@omg.org, curtis@omg.org
Subject: Illegal IDL in CosTime module

The IDL in module CosTime for the compare_time and
time_to_interval operations is illegal.  Reference
http://www.omg.org/archives/experts/msg00890.html
for further information.

    module CosTime {
	interface TIO; // forward declaration
	interface UTO {
	    readonly attribute TimeBase::TimeT time;
	    readonly attribute TimeBase::InaccuracyTinaccuracy;
	    readonly attribute TimeBase::TdfT tdf;
	    readonly attribute TimeBase::UtcT utc_time;
	    UTO absolute_time();
	    TimeComparison compare_time(
		in ComparisonType comparison_type,
		in UTO uto
		^^^^^^^^^^
	    );
	    TIO time_to_interval(
		in UTO uto
		^^^^^^^^^^
	    );
	    TIO interval();
	};
    };

I suggest the following modified IDL to repair the problem:

    module CosTime {
	interface TIO; // forward declaration
	interface UTO {
	    readonly attribute TimeBase::TimeT time;
	    readonly attribute TimeBase::InaccuracyTinaccuracy;
	    readonly attribute TimeBase::TdfT tdf;
	    readonly attribute TimeBase::UtcT utc_time;
	    UTO absolute_time();
	    TimeComparison compare_time(
		in ComparisonType comparison_type,
		in UTO other_uto
		       ^^^^^^   
	    );
	    TIO time_to_interval(
		in UTO other_uto
		       ^^^^^^   
	    );
	    TIO interval();
	};
    };

-- Bill


Return-Path: <jis@fpk.hp.com>
Sender: jis@fpk.hp.com
Date: Thu, 16 Jul 1998 16:59:15 -0400
From: Jishnu Mukerji <jis@fpk.hp.com>
Organization: Hewlett-Packard New Jersey Laboratories
To: Bill Beckwith <bill.beckwith@ois.com>
Cc: orb_revision@omg.org, curtis@omg.org
Subject: Re: Illegal IDL in CosTime module
References: <Pine.SUN.3.91.980716143358.405g-100000@gamma>

Bill Beckwith wrote:

> The IDL in module CosTime for the compare_time and
> time_to_interval operations is illegal.  Reference
> http://www.omg.org/archives/experts/msg00890.html
> for further information.
>
>     module CosTime {
>         interface TIO; // forward declaration
>         interface UTO {
>             readonly attribute TimeBase::TimeT time;
>             readonly attribute TimeBase::InaccuracyTinaccuracy;
>             readonly attribute TimeBase::TdfT tdf;
>             readonly attribute TimeBase::UtcT utc_time;
>             UTO absolute_time();
>             TimeComparison compare_time(
>                 in ComparisonType comparison_type,
>                 in UTO uto
>                 ^^^^^^^^^^
>             );
>             TIO time_to_interval(
>                 in UTO uto
>                 ^^^^^^^^^^
>             );
>             TIO interval();
>         };
>     };
>
> I suggest the following modified IDL to repair the problem:
>
>     module CosTime {
>         interface TIO; // forward declaration
>         interface UTO {
>             readonly attribute TimeBase::TimeT time;
>             readonly attribute TimeBase::InaccuracyTinaccuracy;
>             readonly attribute TimeBase::TdfT tdf;
>             readonly attribute TimeBase::UtcT utc_time;
>             UTO absolute_time();
>             TimeComparison compare_time(
>                 in ComparisonType comparison_type,
>                 in UTO other_uto
>                        ^^^^^^
>             );
>             TIO time_to_interval(
>                 in UTO other_uto
>                        ^^^^^^
>             );
>             TIO interval();
>         };
>     };
>
> -- Bill


A better way to fix it is to change the type of the offending
parameters
to CosTime::UTO. I just received this issue in the Core Revision TF
(since there is no RTF in action for Time Service I presume). I will
happily propose this resolution as shown in attached to the Core
Revision voters to incorporate in the Time Service spec. It is such a
no
brainer and obvious that it should not be a problem. Actually I have
tried to have this fixed in the past as an editorial fix, but to no
avail, so this time we will try to run it through the system.:-)

Thanks,

Jishnu.



         
Issue 1688: Illegal IDL in CosTime module (orb_revision)
Click </issues/issue1688.txt>here for this issue's archive. 
Source: Objective Interface Systems (Mr. Bill Beckwith,
<mailto:>bill.beckwith@ois.com) 
Nature: Uncategorized Issue 
Severity: 
Summary: The IDL in module CosTime for the compare_time and
time_to_interval operations is illegal. Reference
http://www.omg.org/archives/experts/msg00890.html for further
information. 
Resolution: Fix the problem by changing the offending parameter names
from "uto" to "other_uto". 
Revised Text: All references to formal/97-12-21.pdf 
1. Change the second line from the top of page 14-9 from: 

    "in UTO        uto" 

to: 

    "in CosTime::UTO        uto" 

2. Change the 5th line on the same page from: 

    "in UTO        uto" 

to: 

    "in CosTime::UTO        uto" 

3.  On page 14-22 in the IDL declaration for interface UTO make the
following changes: 

    i. In declaration of operation compare_time change the type of the
last parameter from "UTO" to "CosTime::UTO". 

    ii. In declaration of the operation time_to_interval change the
name of the parameter from "UTO" to "CosTime::UTO". 

Actions to be taken: Incorporate changes in Time Service chapter and
close issue. 
July 16, 1998: revceived issue