Ticket #259 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

Reconciler doesn't see callout visibility modifiers

Reported by: stephan Owned by: stephan
Priority: major Milestone: OTDT_1.3.0_M4
Component: compiler Version: 1.3.0M3
Keywords: Cc:

Description

Looking at role file PackageExplorerAdaptor.ContentProvider the editor shows an error claiming that method ClassPathContainer.contains is not visible. The method is defined by a callout with protected visibility.

The error is shown only in the editor, not in the problems view.

Adding an abstract method declaration for the callout with appropriate modifiers (protected abstract static) removes the error. It seems like converting the callout back and forth loses the original modifiers.

Change History

Changed 3 years ago by stephan

Witness is in r21668.

Changed 3 years ago by stephan

  • status changed from new to closed
  • resolution set to fixed
  • milestone changed from OTDT_1.3.0 to OTDT_1.3.0_M4

Fixed in r21673.

In addition to the source-level round trip from compiler to model and back again the patch also includes handling for binary types (model), which however, isn't used currently (could, e.g., be used by the package explorer to visualize callout visibility including binaries - so far tested only manually).

Note: See TracTickets for help on using tickets.