-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129275/#review100384
-----------------------------------------------------------




autotests/jobtest.cpp (line 283)
<https://git.reviewboard.kde.org/r/129275/#comment67372>

    It's stream.flush() that you need, to tell QTextStream to do the actual 
writing to the underlying device.
    
    QTextStream only flushes automatically when being deleted or when the 
underlying device is a file and is closed.
    
    Once doing stream.flush() here, this test fails like the other one, so this 
isn't related to QXmlStreamWriter, but to passing an open QTemporaryFile as the 
device to storedPut. We need to at least rewind it to pos 0...



autotests/jobtest.cpp (line 292)
<https://git.reviewboard.kde.org/r/129275/#comment67371>

    This test only passes because it's comparing 0 with 0 :-)


- David Faure


On Oct. 29, 2016, 10:35 a.m., Elvis Angelaccio wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129275/
> -----------------------------------------------------------
> 
> (Updated Oct. 29, 2016, 10:35 a.m.)
> 
> 
> Review request for KDE Frameworks and David Faure.
> 
> 
> Repository: kio
> 
> 
> Description
> -------
> 
> https://bugs.kde.org/show_bug.cgi?id=368314 seems to be an actual bug in KIO, 
> rather than a regression in kwalletmanager. I reproduced the bug in a test 
> case: basically `StoredTransferJob` fails to copy a QTemporaryFile when the 
> file has been written by a QXmlStreamWriter (destination file is created, but 
> empty).
> 
> If we write to the temp file using a QTextStreamer instead, everything works 
> as expected...
> 
> 
> Diffs
> -----
> 
>   autotests/jobtest.h 5687b4a448d26d703d33603e5a0939bc4f8c77b4 
>   autotests/jobtest.cpp 11bac4d51e97840181aff99fdddb92cdf4db1ba2 
> 
> Diff: https://git.reviewboard.kde.org/r/129275/diff/
> 
> 
> Testing
> -------
> 
> jobtest still passes (with one expected failure).
> 
> 
> Thanks,
> 
> Elvis Angelaccio
> 
>

Reply via email to