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 _______________________________________________ fpc-pascal maillist - fpc-pascal@lists.freepascal.org http://lists.freepascal.org/mailman/listinfo/fpc-pascal