On Saturday, August 21, 2010, Nigel Cunningham wrote: > Hi. > > Sorry for being slow to reply, but to my mind, having two copies of the > whole save and load routines is just plain ugly and makes > maintainability a nightmare. Well, how much code is actually duplicated by having separate routines for with and without compression? I don't think it's too much. > Every time I've thought there's been a problem with compression or > decompression, it has turned out to be a problem in my code instead. > Let's not waste time and effort duplicating code unnecessarily. Having two separate functions for image saving/restoring, depending on cartain condition, is not wrong in principle as long as they are sufficiently different. So going back to the previous question, aren't they? An alternative is to put several conditionals into a sensitive piece of code, which I'd rather prefer to avoid. Thanks, Rafael _______________________________________________ linux-pm mailing list linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/linux-pm