Ticket #25 (closed enhancement: fixed)

Opened 4 years ago

Last modified 4 years ago

@inherits compatibility issue - fix?

Reported by: jhood Owned by: 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

Changed 4 years ago 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.

Changed 4 years ago by anonymous

  • status changed from closed to reopened
  • type changed from defect to enhancement
  • resolution fixed 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).

Changed 4 years ago 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.

Note: See TracTickets for help on using tickets.