[Xastir] win32 shapefiles

[Curt Mills, WE7U]

On Wed, 24 Sep 2003, Bryce Wolfson wrote:

> > We look at the timestamps on the files.
> > If the map index is newer than the map files, we don't index.
>
> This seems unnecessarily fragile, especially considering that I see "Q: My
> new maps aren't showing up", "A: Try touching the timestamps, or do an index
> all if all else fails" every few weeks on this list.
>
> In order to find maps that have newer timestamps than the index, you must
> already be walking the entire tree and pulling the directory entry for every
> file; why not, when walking the directory structure on an index update,
> check for the presence of the map in the index? This alone would solve the
> 'new maps with timestamps before index build date' problem. As an even
> better improvement, store the map's timestamp in the index as well (adds 4
> bytes per map), and if a particular map _is_ already in the index (since
> you're looking anyway) then check the timestamp against the one stored in
> the index to see if the map's been updated and the index entry needs updated
> as well.
>
> I'm seeing a lot of user issues caused by the current behavior; this really
> seems like something the software should be able to cleanly handle.

I was never all that satisfied with how it was done.  I'm the one
that coded it, to solve one user's problems with switching various
map mounts in and out using NFS mounts.

I think that when we reindex all maps we're blowing away the
in-memory map directory first, but when we just check for new ones,
we're not.  The second one should be able to be modified easily to
check whether a map is in the index (we already have code that does
that) and attempt an index in that case.  I'll look into doing that.

-- 
Curt Mills, WE7U                    hacker_NO_SPAM_ at tc.fluke.com
Senior Methods Engineer/SysAdmin
"Lotto:    A tax on people who are bad at math!"
"Windows:  Microsoft's tax on computer illiterates!" -- WE7U
"The world DOES revolve around me:  I picked the coordinate system!"