>>>>> "Angus" == Angus Leeming <[EMAIL PROTECTED]> writes:

Angus> This, too, seems fair. I see that *only* the XForms file dialog
Angus> uses FileInfo::typeIndicator. However, the directory indicator,
Angus> '/', is certainly a useful visual aid, so I don't think we
Angus> should remove FileInfo::typeIndicator entirely.

Or we can use FileInfo::isDir for this.

The FormFileDialog code is scary.

Angus> Nonetheless, I basically agree. The dialog already displays the
Angus> "rwxr-xr-x" status of the file, so why don't I change the patch
Angus> to:

That's a reasonable simple way to do it.

Note that this kind of basic information is available directly from xforms.

JMarc

Reply via email to