[fpc-pascal] Makeskel generation question

[Reinier Olislagers]

On 18-11-2011 13:14, Michael Van Canneyt wrote:
> On Fri, 18 Nov 2011, Reinier Olislagers wrote:
>> I've run makeskel on ibconnection.pp.
>>
>> This declaration:
>>  protected
>> ...
>>    function GetHandle : pointer; override;
>>
>> gets output as:
>> <!-- function Visibility: protected -->
>> <element name="TIBConnection.GetHandle">
>> <short></short>
>> <descr>
>> </descr>
>> <errors>
>> </errors>
>> <seealso>
>> </seealso>
>> </element>
>>
>> <!-- function result Visibility: default -->
>> <element name="TIBConnection.GetHandle.Result">
>> <short></short>
>> </element>
>>
>> Shouldn't the TIBConnection.GetHandle.Result function result visibility
>> be protected as well?
> 
> No, why ? A function result has no visibility.
> The comment is just a hint for the documenter, after all.
> 
> If you want to make a patch to specify the parent function's visibility,
> go ahead, but I think there are more useful things that can be done...
> 
> Michael.
Filed a patch: 22639
This also improves error reporting for missing source files.

Thanks,
Reinier