On Sun, Aug 23, 2009 at 01:45:02PM +0200, Frank van Maarseveen wrote:
> 
> The ID file CreateID promotion issue is also present in the --create-id
> option and I just got bitten by it.
> 
> When ufraw-batch is used as a preparational step for anything else
> then the image output generation not only is redundant but takes a
> disproportional amount of time. So the --create-id=only option is useful
> even for ufraw-batch but it doesn't work. Fix:
> 
> diff --git a/ufraw-batch.c b/ufraw-batch.c
> index 79f77c3..85c989c 100644
> --- a/ufraw-batch.c
> +++ b/ufraw-batch.c
> @@ -86,8 +86,6 @@ int main (int argc, char **argv)
>           continue;
>       }
>       status = ufraw_config(uf, &rc, &conf, &cmd);
> -     if (uf->conf && uf->conf->createID == only_id)
> -         uf->conf->createID = also_id;
>       if (status==UFRAW_ERROR) {
>           exitCode = 1;
>           ufraw_close(uf);
> @@ -107,10 +105,11 @@ int main (int argc, char **argv)
>           stat[0] = '\0';
>       ufraw_message(UFRAW_MESSAGE, _("Loaded %s%s"), uf->filename, stat);
>       status = ufraw_batch_saver(uf);
> -     if (status==UFRAW_SUCCESS || status==UFRAW_WARNING)
> -         ufraw_message(UFRAW_MESSAGE, _("Saved %s%s"),
> -                 uf->conf->outputFilename, stat);
> -     else
> +     if (status==UFRAW_SUCCESS || status==UFRAW_WARNING) {
> +         if (uf->conf->createID!=only_id)
> +             ufraw_message(UFRAW_MESSAGE, _("Saved %s%s"),
> +                     uf->conf->outputFilename, stat);
> +      } else
>           exitCode = 1;
>       ufraw_close(uf);
>       g_free(uf);
> diff --git a/ufraw.h b/ufraw.h
> index ce85436..dc1e287 100644
> --- a/ufraw.h
> +++ b/ufraw.h
> @@ -152,7 +152,6 @@ typedef struct {
>   * CONF|ID: curve/profile are added to the list from RC.
>   * CONF: inputFilename, outputFilename are ignored.
>   * outputPath can only be specified in CMD or guessed in interactive mode.
> - * ID: createID==only_id is switched to also_id in case of ufraw-batch.
>   * ID: chanMul[] override wb, green, temperature.
>   */
>  typedef struct {
> 
> 

Reminder...

-- 
Frank

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
ufraw-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ufraw-devel

Reply via email to