Hi Ivan and Peter,

> Looks good.  Please create a SF.net patch tracker item for this, and assign 
> it 
> to me; if there are no major objections, I'll merge this on Saturday.
> 

I have two more comments:

1) Where is the RECENT_FILES_STORE going to be stored?  

  gchar *file = g_build_filename(g_get_home_dir(), RECENT_FILES_STORE, NULL);

I would really prefer it was in ~/.gEDA/gschem-recent-files  (and not a
hidden file).  That way all the user files for gEDA are in one place.

2) Ivan, are you planning/wanting to look at the "storing window position
and sizes" task next?  If so, I take it you were going to put that 
information in a different file?  I am okay with that only because it keeps
the various files nice and simple.  Or is there some advantage to putting
the info into the above file?

This patch looks pretty darn good.  Thanks,

                                                -Ales




_______________________________________________
geda-dev mailing list
[email protected]
http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev

Reply via email to