I don't understand the commit message. Can you elaborat? What does this
fix? Your commit message seems to imply that things were better WITHOUT
your commit...

/D

On Wed, Nov 05, 2014 at 10:54:33PM +0100, Anton Lundin wrote:
> The second buddy tag will then contain a space, due to that the internal
> splitter is the comma, and not the ,<space>.
> 
> It actually looked better in the ui with ,<space>.
> 
> Signed-off-by: Anton Lundin <[email protected]>
> ---
>  qt-ui/maintab.cpp | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/qt-ui/maintab.cpp b/qt-ui/maintab.cpp
> index 545632f..b5cde40 100644
> --- a/qt-ui/maintab.cpp
> +++ b/qt-ui/maintab.cpp
> @@ -923,7 +923,7 @@ void MainTab::on_buddy_textChanged()
>       QStringList text_list = ui.buddy->toPlainText().split(",", 
> QString::SkipEmptyParts);
>       for (int i = 0; i < text_list.size(); i++)
>               text_list[i] = text_list[i].trimmed();
> -     QString text = text_list.join(", ");
> +     QString text = text_list.join(",");
>       free(displayed_dive.buddy);
>       displayed_dive.buddy = strdup(text.toUtf8().data());
>       markChangedWidget(ui.buddy);
> @@ -936,7 +936,7 @@ void MainTab::on_divemaster_textChanged()
>       QStringList text_list = ui.divemaster->toPlainText().split(",", 
> QString::SkipEmptyParts);
>       for (int i = 0; i < text_list.size(); i++)
>               text_list[i] = text_list[i].trimmed();
> -     QString text = text_list.join(", ");
> +     QString text = text_list.join(",");
>       free(displayed_dive.divemaster);
>       displayed_dive.divemaster = strdup(text.toUtf8().data());
>       markChangedWidget(ui.divemaster);
> -- 
> 1.9.1
> 
> _______________________________________________
> subsurface mailing list
> [email protected]
> http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
_______________________________________________
subsurface mailing list
[email protected]
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to