Ticket #25 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

@inherits compatibility issue - fix?

Reported by: jhood Assigned to: mgalloy
Priority: minor Milestone: IDLdoc 3.0
Component: parsing Version: 3.0
Keywords: Cc:

Description

I used the following to find and remove all lines with @inherits in them. This appears to be a safe option to deal with the resultant warnings. My understanding is that inheritance is automatically found from the IDL code now. 

# in every *.pro file find lines with @inherits and delete them
find . -type f -name '*.pro' -exec sed -i '/@inherits/ D' {} \;

Change History

01/06/08 11:03:10 changed by mgalloy

  • status changed from new to closed.
  • resolution set to fixed.
  • summary changed from @inherits compatability issue - fix? to @inherits compatibility issue - fix?.

Yes, inheritance is handled automatically now as long as automatic structure definition is used to define the class instance variables. It should be able to handle multiple class definitions in a single file as well.

The find command is handy. But be careful of the case where the @inherits is spread across multiple lines. For example, some people comment in this style: 

; @inherits
;   SomeClass1, SomeClass2

There was already a warning for @uses tag at the routine level, I added a warning at the file level in r433.

01/12/08 10:07:01 changed by anonymous

  • status changed from closed to reopened.
  • type changed from defect to enhancement.
  • resolution deleted.

Thanks for adding compatability comments ot ISSUES. I suggest that you add a comment there about @inherits (and other deprecated tags if there are any).

01/13/08 22:04:07 changed by mgalloy

  • status changed from reopened to closed.
  • resolution set to fixed.

Fixed in r448, I added an item in ISSUES for the "inherits" tag and another for the ASSISTANT and PREFORMAT keywords that are obsoleted.