[PATCH] Improved adding images

Dirk Hohndel dirk at hohndel.org
Sat Mar 14 19:38:53 PDT 2015


On Sat, Mar 14, 2015 at 06:03:57PM +0100, Jan Darowski wrote:
> diff --git a/qt-ui/shiftimagetimes.ui b/qt-ui/shiftimagetimes.ui
> index cce51e8..ad81923 100644
> --- a/qt-ui/shiftimagetimes.ui
> +++ b/qt-ui/shiftimagetimes.ui
> @@ -48,9 +48,9 @@
>            <hour>23</hour>
>            <minute>59</minute>
>            <second>59</second>
> -          <year>2010</year>
> -          <month>12</month>
> -          <day>31</day>
> +          <year>2000</year>
> +          <month>1</month>
> +          <day>1</day>

Why this change? (you have that more than once)

> diff --git a/qt-ui/simplewidgets.cpp b/qt-ui/simplewidgets.cpp
> index 8a0f2b0..b499095 100644
> --- a/qt-ui/simplewidgets.cpp
> +++ b/qt-ui/simplewidgets.cpp
> @@ -289,7 +289,7 @@ void ShiftImageTimesDialog::syncCameraClicked()
>  	scene->addPixmap(picture.scaled(ui.DCImage->size()));
>  	ui.DCImage->setScene(scene);
>  
> -	picture_get_timestamp(fileNames.at(0).toUtf8().data(), &timestamp);
> +	picture_get_timestamp(qstrdup(fileNames.at(0).toUtf8().data()), &timestamp);

Why qstrdup here? Why not strdup?

/D


More information about the subsurface mailing list