Comment 6 for bug 499733

Revision history for this message
markba (mark-baaijens) wrote :

Having looked at the patch, I have a few remarks.

1. The detection on what type of project file should be automatic. I suggest something like this (in pseudo-code):

try
  [open project file as text]
  OpenProject()
except
  try
    [open project file as binary]
    OpenProject()
  except
    [launch error message: "Could not open project file"]
  end
end

This ensures backwards compatability. This cannot be done with a global setting in preferences (as used in the patch), because one is never sure which type it is, especially when the user has switched his preference.

2. The default format for new projects should be text.

3. IMO, there's no need in putting the format as a user preference, because storing as a binary serves no real advantage, other then storage size; size is normally not a problem, considering the size of the resources (clips audio/video).