Release 0.61.08
[synfig.git] / synfig-core / tags / stable / src / tool / main.cpp
index 794e0d1..d74f3cf 100644 (file)
@@ -1,20 +1,22 @@
-/* === S I N F G =========================================================== */
-/*!    \file main.cpp
-**     \brief SINFG Tool
+/* === S Y N F I G ========================================================= */
+/*!    \file tool/main.cpp
+**     \brief SYNFIG Tool
 **
-**     $Id: main.cpp,v 1.9 2005/01/23 04:41:10 darco Exp $
+**     $Id$
 **
 **     \legal
-**     Copyright (c) 2002 Robert B. Quattlebaum Jr.
+**     Copyright (c) 2002-2005 Robert B. Quattlebaum Jr., Adrian Bentley
+**     Copyright (c) 2007 Chris Moore
 **
-**     This software and associated documentation
-**     are CONFIDENTIAL and PROPRIETARY property of
-**     the above-mentioned copyright holder.
+**     This package is free software; you can redistribute it and/or
+**     modify it under the terms of the GNU General Public License as
+**     published by the Free Software Foundation; either version 2 of
+**     the License, or (at your option) any later version.
 **
-**     You may not copy, print, publish, or in any
-**     other way distribute this software without
-**     a prior written agreement with
-**     the copyright holder.
+**     This package is distributed in the hope that it will be useful,
+**     but WITHOUT ANY WARRANTY; without even the implied warranty of
+**     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+**     General Public License for more details.
 **     \endlegal
 */
 /* ========================================================================= */
 #include <ETL/clock>
 #include <algorithm>
 
-#include <sinfg/loadcanvas.h>
-#include <sinfg/savecanvas.h>
-#include <sinfg/target_scanline.h>
-#include <sinfg/module.h>
-#include <sinfg/importer.h>
-#include <sinfg/layer.h>
-#include <sinfg/canvas.h>
-#include <sinfg/target.h>
-#include <sinfg/time.h>
-#include <sinfg/string.h>
-#include <sinfg/paramdesc.h>
-#include <sinfg/main.h>
-#include <sinfg/guid.h>
+#include <synfig/loadcanvas.h>
+#include <synfig/savecanvas.h>
+#include <synfig/target_scanline.h>
+#include <synfig/module.h>
+#include <synfig/importer.h>
+#include <synfig/layer.h>
+#include <synfig/canvas.h>
+#include <synfig/target.h>
+#include <synfig/time.h>
+#include <synfig/string.h>
+#include <synfig/paramdesc.h>
+#include <synfig/main.h>
+#include <synfig/guid.h>
 #endif
 
 using namespace std;
 using namespace etl;
-using namespace sinfg;
+using namespace synfig;
 
 /* === M A C R O S ========================================================= */
 
 enum exit_code
 {
-       SINFGTOOL_OK                            =0,
-       SINFGTOOL_FILENOTFOUND          =1,
-       SINFGTOOL_BORRED                        =2,
-       SINFGTOOL_HELP                          =3,
-       SINFGTOOL_UNKNOWNARGUMENT       =4,     
-       SINFGTOOL_UNKNOWNERROR          =5,     
-       SINFGTOOL_INVALIDTARGET         =6,     
-       SINFGTOOL_RENDERFAILURE         =7,     
-       SINFGTOOL_BLANK                         =8,
-       SINFGTOOL_BADVERSION            =9
+       SYNFIGTOOL_OK                           =0,
+       SYNFIGTOOL_FILENOTFOUND         =1,
+       SYNFIGTOOL_BORRED                       =2,
+       SYNFIGTOOL_HELP                         =3,
+       SYNFIGTOOL_UNKNOWNARGUMENT      =4,
+       SYNFIGTOOL_UNKNOWNERROR         =5,
+       SYNFIGTOOL_INVALIDTARGET                =6,
+       SYNFIGTOOL_RENDERFAILURE                =7,
+       SYNFIGTOOL_BLANK                                =8,
+       SYNFIGTOOL_BADVERSION           =9,
+       SYNFIGTOOL_MISSINGARGUMENT      =10
 };
 
 #ifndef VERSION
 #define VERSION "unknown"
-#define PACKAGE "sinfg-tool"
+#define PACKAGE "synfig-tool"
 #endif
 
 #ifdef DEFAULT_QUALITY
@@ -90,14 +93,14 @@ bool print_benchmarks=false;
 
 /* === M E T H O D S ======================================================= */
 
-class Progress : public sinfg::ProgressCallback
+class Progress : public synfig::ProgressCallback
 {
        const char *program;
-       
+
 public:
-       
+
        Progress(const char *name):program(name) { }
-       
+
        virtual bool
        task(const String &task)
        {
@@ -120,25 +123,25 @@ public:
        }
 
        virtual bool
-       amount_complete(int current, int total)
+       amount_complete(int /*current*/, int /*total*/)
        {
                return true;
        }
 };
 
-class RenderProgress : public sinfg::ProgressCallback
+class RenderProgress : public synfig::ProgressCallback
 {
        string taskname;
-       
+
        etl::clock clk;
        int clk_scanline; // The scanline at which the clock was reset
        etl::clock clk2;
 
-       float last_time;        
+       float last_time;
 public:
-       
-       RenderProgress():clk_scanline(0) { }
-       
+
+       RenderProgress():clk_scanline(0), last_time(0) { }
+
        virtual bool
        task(const String &thetask)
        {
@@ -176,7 +179,7 @@ public:
                        if(clk2()<0.2)
                                return true;
                        clk2.reset();
-                               
+
                        if(scanline)
                                seconds=(int)time+1;
                        else
@@ -185,7 +188,7 @@ public:
                                clk.reset();
                                clk_scanline=scanline;
                        }
-                       
+
                        if(seconds<0)
                        {
                                clk.reset();
@@ -200,7 +203,7 @@ public:
                                days++,hours-=24;
                        while(days>=7)
                                weeks++,days-=7;
-                       
+
                        cerr<<taskname<<": "<<_("Line")<<" "<<scanline<<_(" of ")<<h<<" -- ";
                        //cerr<<time/(h-clk_scanline)<<" ";
                        /*
@@ -211,9 +214,9 @@ public:
                        {
                                //cerr<<"reset"<<endl;
                                clk.reset();
-                               clk_scanline=scanline;                          
+                               clk_scanline=scanline;
                        }
-                       
+
                        if(weeks)
                                cerr<<weeks<<"w ";
                        if(days)
@@ -224,7 +227,7 @@ public:
                                cerr<<minutes<<"m ";
                        if(seconds)
                                cerr<<seconds<<"s ";
-                       
+
                        cerr<<"           \r";
                }
                else
@@ -237,13 +240,13 @@ struct Job
 {
        String filename;
        String outfilename;
-       
+
        RendDesc desc;
 
        Canvas::Handle root;
        Canvas::Handle canvas;
        Target::Handle target;
-       
+
        int quality;
        bool sifout;
 };
@@ -256,7 +259,7 @@ void guid_test()
        cout<<"GUID Test"<<endl;
        for(int i=20;i;i--)
        {
-               cout<<sinfg::GUID().get_string()<<' '<<sinfg::GUID().get_string()<<endl;
+               cout<<synfig::GUID().get_string()<<' '<<synfig::GUID().get_string()<<endl;
        }
 }
 
@@ -293,7 +296,7 @@ void display_help(int amount)
                                cerr<<strprintf(" %s %s %s",flag, arg, spaces+strlen(arg)+strlen(flag)+1)+description<<endl;
                        else
                                cerr<<strprintf(" %s %s",flag,spaces+strlen(flag))+description<<endl;
-               
+
                }
        };
        cerr<<_("syntax: ")<<progname<<" [DEFAULT OPTIONS] ([SIF FILE] [SPECIFIC OPTIONS])..."<<endl;
@@ -346,30 +349,30 @@ void display_help(int amount)
 int process_global_flags(arg_list_t &arg_list)
 {
        arg_list_t::iterator iter, next;
-       
+
        for(next=arg_list.begin(),iter=next++;iter!=arg_list.end();iter=next++)
        {
                if(*iter == "--")
-                       return SINFGTOOL_OK;
+                       return SYNFIGTOOL_OK;
 
                if(*iter == "--signal-test")
                {
                        signal_test();
-                       return SINFGTOOL_HELP;
+                       return SYNFIGTOOL_HELP;
                }
-               
+
                if(*iter == "--guid-test")
                {
                        guid_test();
-                       return SINFGTOOL_HELP;
+                       return SYNFIGTOOL_HELP;
                }
 
-               
+
                if(*iter == "--help")
                {
                        display_help(1);
 
-                       return SINFGTOOL_HELP;
+                       return SYNFIGTOOL_HELP;
                }
 
                if(*iter == "--info")
@@ -386,28 +389,28 @@ int process_global_flags(arg_list_t &arg_list)
                        cout<<" with Borland Turbo C++ "<<(__TCPLUSPLUS__>>8)<<'.'<<((__TCPLUSPLUS__&255)>>4)<<'.'<<(__TCPLUSPLUS__&15);
 #endif
 
-                       cout<<endl<<SINFG_COPYRIGHT<<endl;
+                       cout<<endl<<SYNFIG_COPYRIGHT<<endl;
                        cout<<endl;
-                       return SINFGTOOL_HELP;
+                       return SYNFIGTOOL_HELP;
                }
 
                if(*iter == "--layers")
                {
                        Progress p(PACKAGE);
-                       sinfg::Main sinfg_main(dirname(progname),&p);
-                       sinfg::Layer::Book::iterator iter=sinfg::Layer::book().begin();
-                       for(;iter!=sinfg::Layer::book().end();iter++)
+                       synfig::Main synfig_main(dirname(progname),&p);
+                       synfig::Layer::Book::iterator iter=synfig::Layer::book().begin();
+                       for(;iter!=synfig::Layer::book().end();iter++)
                                cout<<iter->first<<endl;
 
-                       return SINFGTOOL_HELP;
+                       return SYNFIGTOOL_HELP;
                }
 
                if(*iter == "--layer-info")
                {
                        Progress p(PACKAGE);
-                       sinfg::Main sinfg_main(dirname(progname),&p);
+                       synfig::Main synfig_main(dirname(progname),&p);
                        iter=next++;
-                       Layer::Handle layer=sinfg::Layer::create(*iter);
+                       Layer::Handle layer=synfig::Layer::create(*iter);
                        cout<<"Layer Name: "<<layer->get_name()<<endl;
                        cout<<"Localized Layer Name: "<<layer->get_local_name()<<endl;
                        cout<<"Version: "<<layer->get_version()<<endl;
@@ -424,85 +427,83 @@ int process_global_flags(arg_list_t &arg_list)
                                        cout<<"\tHint: "<<vocab.front().get_hint()<<endl;
                        }
 
-                       return SINFGTOOL_HELP;
+                       return SYNFIGTOOL_HELP;
                }
 
                if(*iter == "--modules")
                {
                        Progress p(PACKAGE);
-                       sinfg::Main sinfg_main(dirname(progname),&p);
-                       sinfg::Module::Book::iterator iter=sinfg::Module::book().begin();
-                       for(;iter!=sinfg::Module::book().end();iter++)
+                       synfig::Main synfig_main(dirname(progname),&p);
+                       synfig::Module::Book::iterator iter=synfig::Module::book().begin();
+                       for(;iter!=synfig::Module::book().end();iter++)
                                cout<<iter->first<<endl;
-                       return SINFGTOOL_HELP;
+                       return SYNFIGTOOL_HELP;
                }
 
                if(*iter == "--targets")
                {
                        Progress p(PACKAGE);
-                       sinfg::Main sinfg_main(dirname(progname),&p);
-                       sinfg::Target::Book::iterator iter=sinfg::Target::book().begin();
-                       for(;iter!=sinfg::Target::book().end();iter++)
+                       synfig::Main synfig_main(dirname(progname),&p);
+                       synfig::Target::Book::iterator iter=synfig::Target::book().begin();
+                       for(;iter!=synfig::Target::book().end();iter++)
                                cout<<iter->first<<endl;
-                               
-                       return SINFGTOOL_HELP;
+
+                       return SYNFIGTOOL_HELP;
                }
 
                if(*iter == "--valuenodes")
                {
                        Progress p(PACKAGE);
-                       sinfg::Main sinfg_main(dirname(progname),&p);
-                       sinfg::LinkableValueNode::Book::iterator iter=sinfg::LinkableValueNode::book().begin();
-                       for(;iter!=sinfg::LinkableValueNode::book().end();iter++)
+                       synfig::Main synfig_main(dirname(progname),&p);
+                       synfig::LinkableValueNode::Book::iterator iter=synfig::LinkableValueNode::book().begin();
+                       for(;iter!=synfig::LinkableValueNode::book().end();iter++)
                                cout<<iter->first<<endl;
-                               
-                       return SINFGTOOL_HELP;
+
+                       return SYNFIGTOOL_HELP;
                }
 
                if(*iter == "--importers")
                {
                        Progress p(PACKAGE);
-                       sinfg::Main sinfg_main(dirname(progname),&p);
-                       sinfg::Importer::Book::iterator iter=sinfg::Importer::book().begin();
-                       for(;iter!=sinfg::Importer::book().end();iter++)
+                       synfig::Main synfig_main(dirname(progname),&p);
+                       synfig::Importer::Book::iterator iter=synfig::Importer::book().begin();
+                       for(;iter!=synfig::Importer::book().end();iter++)
                                cout<<iter->first<<endl;
-                               
-                       return SINFGTOOL_HELP;
+
+                       return SYNFIGTOOL_HELP;
                }
 
                if(*iter == "--version")
                {
                        cerr<<PACKAGE<<" "<<VERSION<<endl;
-               
+
                        arg_list.erase(iter);
-                       
-                       return SINFGTOOL_HELP;
+
+                       return SYNFIGTOOL_HELP;
                }
 
                if(*iter == "--license")
                {
                        cerr<<PACKAGE<<" "<<VERSION<<endl;
-                       cout<<SINFG_COPYRIGHT<<endl<<endl;
+                       cout<<SYNFIG_COPYRIGHT<<endl<<endl;
                        cerr<<"\
-**     This software and associated documentation\n\
-**     are CONFIDENTIAL and PROPRIETARY property of\n\
-**     the above-mentioned copyright holder.\n\
+**     This package is free software; you can redistribute it and/or\n\
+**     modify it under the terms of the GNU General Public License as\n\
+**     published by the Free Software Foundation; either version 2 of\n\
+**     the License, or (at your option) any later version.\n\
 **\n\
-**     You may not use, copy, print, publish, or in\n\
-**     any other way distribute or exploit this software\n\
-**     without a prior written agreement with the\n\
-**     copyright holder. " << endl << endl;    
+**     " << endl << endl;
                        arg_list.erase(iter);
-                       
-                       return SINFGTOOL_HELP;
+
+                       return SYNFIGTOOL_HELP;
                }
 
                if(*iter == "-v")
                {
                        verbosity++;
-                       
+
                        arg_list.erase(iter);
-                       
+
                        continue;
                }
 
@@ -523,20 +524,20 @@ int process_global_flags(arg_list_t &arg_list)
                        continue;
                }
        }
-       
-       return SINFGTOOL_OK;
+
+       return SYNFIGTOOL_OK;
 }
 
 int extract_arg_cluster(arg_list_t &arg_list,arg_list_t &cluster)
 {
        arg_list_t::iterator iter, next;
-       
+
        for(next=arg_list.begin(),iter=next++;iter!=arg_list.end();iter=next++)
        {
                if(*iter->begin() != '-')
                {
                        //cerr<<*iter->begin()<<"-----------"<<endl;
-                       return SINFGTOOL_OK;
+                       return SYNFIGTOOL_OK;
                }
 
                if(
@@ -566,13 +567,17 @@ int extract_arg_cluster(arg_list_t &arg_list,arg_list_t &cluster)
                        cluster.push_back(*iter);
                        arg_list.erase(iter);
                        iter=next++;
+                       if (iter==arg_list.end()) {
+                               error("The `%s' flag requires a value.  Use --help for a list of options.", cluster.back().c_str());
+                               return SYNFIGTOOL_MISSINGARGUMENT;
+                       }
                }
-                       
+
                cluster.push_back(*iter);
                arg_list.erase(iter);
        }
-       
-       return SINFGTOOL_OK;
+
+       return SYNFIGTOOL_OK;
 }
 
 int extract_RendDesc(arg_list_t &arg_list,RendDesc &desc)
@@ -589,14 +594,14 @@ int extract_RendDesc(arg_list_t &arg_list,RendDesc &desc)
                        w=atoi(iter->c_str());
                        arg_list.erase(iter);
                }
-               if(*iter=="-h")
+               else if(*iter=="-h")
                {
                        arg_list.erase(iter);
                        iter=next++;
                        h=atoi(iter->c_str());
                        arg_list.erase(iter);
                }
-               if(*iter=="-a")
+               else if(*iter=="-a")
                {
             int a;
                        arg_list.erase(iter);
@@ -606,7 +611,7 @@ int extract_RendDesc(arg_list_t &arg_list,RendDesc &desc)
                        VERBOSE_OUT(1)<<strprintf(_("Antialiasing set to %d, (%d samples per pixel)"),a,a*a)<<endl;
                        arg_list.erase(iter);
                }
-               if(*iter=="-s")
+               else if(*iter=="-s")
                {
                        arg_list.erase(iter);
                        iter=next++;
@@ -614,7 +619,7 @@ int extract_RendDesc(arg_list_t &arg_list,RendDesc &desc)
                        VERBOSE_OUT(1)<<strprintf(_("Span set to %d units"),span)<<endl;
                        arg_list.erase(iter);
                }
-               if(*iter=="--fps")
+               else if(*iter=="--fps")
                {
                        arg_list.erase(iter);
                        iter=next++;
@@ -623,7 +628,7 @@ int extract_RendDesc(arg_list_t &arg_list,RendDesc &desc)
                        arg_list.erase(iter);
                        VERBOSE_OUT(1)<<strprintf(_("Frame rate set to %d frames per second"),fps)<<endl;
                }
-               if(*iter=="--dpi")
+               else if(*iter=="--dpi")
                {
                        arg_list.erase(iter);
                        iter=next++;
@@ -633,7 +638,7 @@ int extract_RendDesc(arg_list_t &arg_list,RendDesc &desc)
                        arg_list.erase(iter);
                        VERBOSE_OUT(1)<<strprintf(_("Physical resolution set to %f dpi"),dpi)<<endl;
                }
-               if(*iter=="--dpi-x")
+               else if(*iter=="--dpi-x")
                {
                        arg_list.erase(iter);
                        iter=next++;
@@ -643,7 +648,7 @@ int extract_RendDesc(arg_list_t &arg_list,RendDesc &desc)
                        arg_list.erase(iter);
                        VERBOSE_OUT(1)<<strprintf(_("Physical X resolution set to %f dpi"),dpi)<<endl;
                }
-               if(*iter=="--dpi-y")
+               else if(*iter=="--dpi-y")
                {
                        arg_list.erase(iter);
                        iter=next++;
@@ -653,21 +658,21 @@ int extract_RendDesc(arg_list_t &arg_list,RendDesc &desc)
                        arg_list.erase(iter);
                        VERBOSE_OUT(1)<<strprintf(_("Physical Y resolution set to %f dpi"),dpi)<<endl;
                }
-               if(*iter=="--start-time" || *iter=="--begin-time")
+               else if(*iter=="--start-time" || *iter=="--begin-time")
                {
                        arg_list.erase(iter);
                        iter=next++;
                        desc.set_time_start(Time(*iter,desc.get_frame_rate()));
                        arg_list.erase(iter);
                }
-               if(*iter=="--end-time")
+               else if(*iter=="--end-time")
                {
                        arg_list.erase(iter);
                        iter=next++;
                        desc.set_time_end(Time(*iter,desc.get_frame_rate()));
                        arg_list.erase(iter);
                }
-               if(*iter=="--time")
+               else if(*iter=="--time")
                {
                        arg_list.erase(iter);
                        iter=next++;
@@ -675,13 +680,13 @@ int extract_RendDesc(arg_list_t &arg_list,RendDesc &desc)
                        VERBOSE_OUT(1)<<_("Rendering frame at ")<<desc.get_time_start().get_string(desc.get_frame_rate())<<endl;
                        arg_list.erase(iter);
                }
-               if(*iter=="-g")
+               else if(*iter=="-g")
                {
-                       sinfg::warning("Gamma argument is currently ignored");
-                       //arg_list.erase(iter);
-                       //iter=next++;
+                       synfig::warning("Gamma argument is currently ignored");
+                       arg_list.erase(iter);
+                       iter=next++;
                        //desc.set_gamma(Gamma(atof(iter->c_str())));
-                       //arg_list.erase(iter);
+                       arg_list.erase(iter);
                }
        }
        if(w&&h)
@@ -704,7 +709,7 @@ int extract_RendDesc(arg_list_t &arg_list,RendDesc &desc)
        }
        if(span)
                desc.set_span(span);
-       return SINFGTOOL_OK;
+       return SYNFIGTOOL_OK;
 }
 
 int extract_quality(arg_list_t &arg_list,int &quality)
@@ -721,8 +726,8 @@ int extract_quality(arg_list_t &arg_list,int &quality)
                        arg_list.erase(iter);
                }
        }
-       
-       return SINFGTOOL_OK;
+
+       return SYNFIGTOOL_OK;
 }
 
 int extract_threads(arg_list_t &arg_list,int &threads)
@@ -739,15 +744,15 @@ int extract_threads(arg_list_t &arg_list,int &threads)
                        arg_list.erase(iter);
                }
        }
-       
-       return SINFGTOOL_OK;
+
+       return SYNFIGTOOL_OK;
 }
 
 int extract_target(arg_list_t &arg_list,string &type)
 {
        arg_list_t::iterator iter, next;
        type.clear();
-       
+
        for(next=arg_list.begin(),iter=next++;iter!=arg_list.end();iter=next++)
        {
                if(*iter=="-t")
@@ -759,14 +764,14 @@ int extract_target(arg_list_t &arg_list,string &type)
                }
        }
 
-       return SINFGTOOL_OK;
+       return SYNFIGTOOL_OK;
 }
 
 int extract_append(arg_list_t &arg_list,string &filename)
 {
        arg_list_t::iterator iter, next;
        filename.clear();
-       
+
        for(next=arg_list.begin(),iter=next++;iter!=arg_list.end();iter=next++)
        {
                if(*iter=="--append")
@@ -778,15 +783,15 @@ int extract_append(arg_list_t &arg_list,string &filename)
                }
        }
 
-       return SINFGTOOL_OK;
+       return SYNFIGTOOL_OK;
 }
 
 int extract_outfile(arg_list_t &arg_list,string &outfile)
 {
        arg_list_t::iterator iter, next;
-       int ret=SINFGTOOL_FILENOTFOUND;
+       int ret=SYNFIGTOOL_FILENOTFOUND;
        outfile.clear();
-       
+
        for(next=arg_list.begin(),iter=next++;iter!=arg_list.end();iter=next++)
        {
                if(*iter=="-o")
@@ -795,7 +800,7 @@ int extract_outfile(arg_list_t &arg_list,string &outfile)
                        iter=next++;
                        outfile=*iter;
                        arg_list.erase(iter);
-                       ret=SINFGTOOL_OK;
+                       ret=SYNFIGTOOL_OK;
                }
        }
 
@@ -806,7 +811,7 @@ int extract_canvasid(arg_list_t &arg_list,string &canvasid)
 {
        arg_list_t::iterator iter, next;
        //canvasid.clear();
-       
+
        for(next=arg_list.begin(),iter=next++;iter!=arg_list.end();iter=next++)
        {
                if(*iter=="-c")
@@ -818,7 +823,7 @@ int extract_canvasid(arg_list_t &arg_list,string &canvasid)
                }
        }
 
-       return SINFGTOOL_OK;
+       return SYNFIGTOOL_OK;
 }
 
 /* === M E T H O D S ======================================================= */
@@ -830,48 +835,51 @@ int main(int argc, char *argv[])
        int i;
        arg_list_t arg_list;
        job_list_t job_list;
-       
+
        progname=argv[0];
        Progress p(argv[0]);
-       
-       if(!SINFG_CHECK_VERSION())
+
+       if(!SYNFIG_CHECK_VERSION())
        {
-               cerr<<_("FATAL: Sinfg Version Mismatch")<<endl;
-               return SINFGTOOL_BADVERSION;
+               cerr<<_("FATAL: Synfig Version Mismatch")<<endl;
+               return SYNFIGTOOL_BADVERSION;
        }
        if(argc==1)
        {
                display_help(0);
-               return SINFGTOOL_BLANK;
+               return SYNFIGTOOL_BLANK;
        }
 
        for(i=1;i<argc;i++)
                arg_list.push_back(argv[i]);
-       
+
        if((i=process_global_flags(arg_list)))
                return i;
 
        VERBOSE_OUT(1)<<_("verbosity set to ")<<verbosity<<endl;
-       sinfg::Main sinfg_main(dirname(progname),&p);
-       
+       synfig::Main synfig_main(dirname(progname),&p);
+
        {
                arg_list_t defaults, imageargs;
-               
+               int ret;
+
                // Grab the defaults before the first file
-               extract_arg_cluster(arg_list,defaults);
-               
+               if ((ret = extract_arg_cluster(arg_list,defaults)) != SYNFIGTOOL_OK)
+                 return ret;
+
                while(arg_list.size())
                {
                        string target_name;
                        job_list.push_front(Job());
                        int threads=0;
-                       
+
                        imageargs=defaults;
                        job_list.front().filename=arg_list.front();
                        arg_list.pop_front();
 
-                       extract_arg_cluster(arg_list,imageargs);                        
-                       
+                       if ((ret = extract_arg_cluster(arg_list,imageargs)) != SYNFIGTOOL_OK)
+                         return ret;
+
                        // Open the composition
                        {
                                job_list.front().root=open_canvas(job_list.front().filename);
@@ -885,7 +893,7 @@ int main(int argc, char *argv[])
                        }
 
                        job_list.front().root->set_time(0);
-                       
+
                        string canvasid;
                        extract_canvasid(imageargs,canvasid);
                        if(!canvasid.empty())
@@ -900,7 +908,7 @@ int main(int argc, char *argv[])
                                        cerr<<_("Throwing out job...")<<endl;
                                        job_list.pop_front();
                                        continue;
-                                       
+
                                }
                                catch(Exception::BadLinkName)
                                {
@@ -908,12 +916,12 @@ int main(int argc, char *argv[])
                                        cerr<<_("Throwing out job...")<<endl;
                                        job_list.pop_front();
                                        continue;
-                                       
+
                                }
                        }
                        else
                                job_list.front().canvas=job_list.front().root;
-                       
+
                        extract_RendDesc(imageargs,job_list.front().canvas->rend_desc());
                        extract_target(imageargs,target_name);
                        extract_threads(imageargs,threads);
@@ -942,25 +950,20 @@ int main(int argc, char *argv[])
                                        VERBOSE_OUT(2)<<_("Appended contents of ")<<composite_file<<endl;
                                }
                        }while(false);
-                       
+
                        VERBOSE_OUT(4)<<_("Attempting to determine target/outfile...")<<endl;
-                       
+
                        // If the target type is not yet defined,
                        // try to figure it out from the outfile.
                        if(target_name.empty() && !job_list.front().outfilename.empty())
-                       try
                        {
                                VERBOSE_OUT(3)<<_("Target name undefined, attempting to figure it out")<<endl;
-                               string ext=string(find(job_list.front().outfilename.begin(),job_list.front().outfilename.end(),'.')+1,job_list.front().outfilename.end());
+                               string ext=job_list.front().outfilename.substr(job_list.front().outfilename.rfind('.')+1);
                                if(Target::ext_book().count(ext))
                                        target_name=Target::ext_book()[ext];
                                else
                                        target_name=ext;
                        }
-                       catch(std::length_error)
-                       {
-                               sinfg::warning("Length error caught when attempting to figure out target name");
-                       }
 
                        // If the target type is STILL not yet defined, then
                        // set it to a some sort of default
@@ -986,10 +989,10 @@ int main(int argc, char *argv[])
 
                        VERBOSE_OUT(4)<<"target_name="<<target_name<<endl;
                        VERBOSE_OUT(4)<<"outfile_name="<<job_list.front().outfilename<<endl;
-                       
+
                        VERBOSE_OUT(4)<<_("Creating the target...")<<endl;
-                       job_list.front().target=sinfg::Target::create(target_name,job_list.front().outfilename);
-                       
+                       job_list.front().target=synfig::Target::create(target_name,job_list.front().outfilename);
+
                        if(target_name=="sif")
                        {
                                job_list.front().sifout=true;
@@ -1011,16 +1014,16 @@ int main(int argc, char *argv[])
                        {
                                VERBOSE_OUT(4)<<_("Setting the canvas on the target...")<<endl;
                                job_list.front().target->set_canvas(job_list.front().canvas);
-                               VERBOSE_OUT(4)<<_("Setting the quality of the target...")<<endl;                                
+                               VERBOSE_OUT(4)<<_("Setting the quality of the target...")<<endl;
                                job_list.front().target->set_quality(job_list.front().quality);
                        }
-                       
+
                        // Set the threads for the target
                        if(job_list.front().target && Target_Scanline::Handle::cast_dynamic(job_list.front().target))
                        {
                                Target_Scanline::Handle::cast_dynamic(job_list.front().target)->set_threads(threads);
                        }
-                       
+
                        if(imageargs.size())
                        {
                                cerr<<_("Unidentified arguments for ")<<job_list.front().filename<<": ";
@@ -1035,22 +1038,22 @@ int main(int argc, char *argv[])
                        //getline(cin,bleh);
                }
        }
-       
+
        if(arg_list.size())
        {
                cerr<<_("Unidentified arguments:");
                for(;arg_list.size();arg_list.pop_front())
                        cerr<<' '<<arg_list.front();
                cerr<<endl;
-               return SINFGTOOL_UNKNOWNARGUMENT;
+               return SYNFIGTOOL_UNKNOWNARGUMENT;
        }
-       
+
        if(!job_list.size())
        {
                cerr<<_("Nothing to do!")<<endl;
-               return SINFGTOOL_BORRED;
+               return SYNFIGTOOL_BORRED;
        }
-       
+
        for(;job_list.size();job_list.pop_front())
        {
                VERBOSE_OUT(3)<<job_list.front().filename<<" -- "<<endl<<'\t'<<
@@ -1067,7 +1070,7 @@ int main(int argc, char *argv[])
                        job_list.front().desc.get_br()[0],job_list.front().desc.get_br()[1],
                        job_list.front().desc.get_focus()[0],job_list.front().desc.get_focus()[1]
                        )<<endl;
-               
+
                RenderProgress p;
                p.task(job_list.front().filename+" ==> "+job_list.front().outfilename);
                if(!job_list.front().sifout)
@@ -1079,7 +1082,7 @@ int main(int argc, char *argv[])
                        if(!job_list.front().target->render(&p))
                        {
                                cerr<<"Render Failure."<<endl;
-                               return SINFGTOOL_RENDERFAILURE;
+                               return SYNFIGTOOL_RENDERFAILURE;
                        }
                        if(print_benchmarks)
                        {
@@ -1091,14 +1094,14 @@ int main(int argc, char *argv[])
                        if(!save_canvas(job_list.front().outfilename,job_list.front().canvas))
                        {
                                cerr<<"Render Failure."<<endl;
-                               return SINFGTOOL_RENDERFAILURE;
+                               return SYNFIGTOOL_RENDERFAILURE;
                        }
                }
        }
-       
+
        job_list.clear();
-       
+
        VERBOSE_OUT(1)<<_("Done.")<<endl;
-       
-       return SINFGTOOL_OK;
+
+       return SYNFIGTOOL_OK;
 }