> > Devaraj Das wrote: > > I was thinking that we really don't require the > mCreationTime field in > > the dfsFileInfo struct and instead have mModificationTime. For now, > > mModificationTime is really the time at which the file got created. > > We can always grow the API later, but it's hard to ever > shrink it. So I'd go with the minimum for now and only > include a single time. I'd suspect application code more > often requires a modification time, so let's call it > mModificationTime for now. Does that make sense?
Modifying a structure in an API requires recompilation of the apps that use it, so growing it is also undesirable. Having a couple extra values that are set to some default value before they're fully supported is common enough, and imposes minimal overhead. I'd add anything that's remotele relevant from 'stat' to dfsFileInfo. Directory listings however should contain then minimum required information.
