[Libreoffice-commits] core.git: Branch 'libreoffice-5-0' - vcl/unx

Caolán McNamara caolanm at redhat.com
Wed Jan 20 10:33:26 PST 2016


 vcl/unx/generic/printer/jobdata.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 01c60a8452bf991bb5e09e9780d05a8376979255
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Wed Jan 20 15:17:09 2016 +0000

    Related: rhbz#1149116 collateEquals was set with OString::boolean
    
    but extracted with toInt32()
    
    since
    
    commit c76cd71fe9bdefaef3f33f8ca193c32e3ab112ed
    
        fdo#41524: CUPS printing: use "collate" option when PDF is available
    
    though actual reported problem works fine for me with default f23
    configuration already, this looks suspicious however
    
    Change-Id: I6fcb5df8039296c0e8b0fe931cb490396182de38
    Reviewed-on: https://gerrit.libreoffice.org/21629
    Reviewed-by: Michael Stahl <mstahl at redhat.com>
    Tested-by: Michael Stahl <mstahl at redhat.com>
    (cherry picked from commit e220ba8610f8315af473a230565daa978ca6a16c)
    Reviewed-on: https://gerrit.libreoffice.org/21631

diff --git a/vcl/unx/generic/printer/jobdata.cxx b/vcl/unx/generic/printer/jobdata.cxx
index 778961f..7e25fdc 100644
--- a/vcl/unx/generic/printer/jobdata.cxx
+++ b/vcl/unx/generic/printer/jobdata.cxx
@@ -251,7 +251,7 @@ bool JobData::constructFromStreamBuffer( void* pData, int bytes, JobData& rJobDa
         }
         else if (aLine.startsWith(collateEquals))
         {
-            rJobData.m_bCollate = aLine.copy(RTL_CONSTASCII_LENGTH(collateEquals)).toInt32();
+            rJobData.m_bCollate = aLine.copy(RTL_CONSTASCII_LENGTH(collateEquals)).toBoolean();
         }
         else if (aLine.startsWith(margindajustmentEquals))
         {


More information about the Libreoffice-commits mailing list