[vlmc-devel] [PATCH 14/14] Main: Apply the latest Project and ConsoleRenderer

Hugo Beauzée-Luyssen hugo at beauzee.fr
Tue Apr 12 14:18:24 CEST 2016


On 04/11/2016 08:09 AM, Yikai Lu wrote:
> ---
>   src/Main/main.cpp | 7 +++----
>   1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/src/Main/main.cpp b/src/Main/main.cpp
> index 77ba8b5..ec1e37d 100644
> --- a/src/Main/main.cpp
> +++ b/src/Main/main.cpp
> @@ -193,11 +193,10 @@ VLMCCoremain( int argc, char **argv )
>
>   #ifndef WITH_GUI
>       ConsoleRenderer renderer;
> -    ProjectManager  *pm = Core::instance()->currentProject();
> +    Project  *p = Core::instance()->project();
>
> -    //FIXME: only signal projectLoaded( const QString& ) in ProjectManager
> -    QCoreApplication::connect( pm, SIGNAL( projectLoaded() ), &renderer, SLOT( startRender() ) );
> -    pm->loadProject( app.arguments()[1] );
> +    QCoreApplication::connect( p, &Project::projectLoaded, &renderer, &ConsoleRenderer::startRender );
> +    p->load( app.arguments()[1] );
>   #endif
>       return app.exec();
>   }
>
LGTM, but the commit message isn't clear IMHO


More information about the Vlmc-devel mailing list