do not ignore return value

[mgallien]

Signed-off-by: Matthieu Gallien matthieu.gallien@nextcloud.com

[codecov]

Codecov Report

Merging #4998 (db2394f) into master (42f6a63) will decrease coverage by 0.19%.
The diff coverage is 100.00%.

❗ Current head db2394f differs from pull request most recent head 74f3519. Consider uploading reports for the commit 74f3519 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4998      +/-   ##
==========================================
- Coverage   57.16%   56.97%   -0.20%     
==========================================
  Files         138      138              
  Lines       17401    17228     -173     
==========================================
- Hits         9947     9815     -132     
+ Misses       7454     7413      -41     

Impacted Files	Coverage Δ
src/libsync/vfs/cfapi/vfs_cfapi.cpp	82.92% <100.00%> (-0.48%)	⬇️
src/libsync/abstractnetworkjob.h	27.27% <0.00%> (-18.19%)	⬇️
src/libsync/account.h	29.41% <0.00%> (-3.93%)	⬇️
src/libsync/syncfileitem.h	37.50% <0.00%> (-2.16%)	⬇️
src/libsync/abstractnetworkjob.cpp	72.61% <0.00%> (-1.99%)	⬇️
src/libsync/propagateremotemkdir.cpp	64.53% <0.00%> (-0.50%)	⬇️
src/libsync/syncfileitem.cpp	94.54% <0.00%> (-0.37%)	⬇️
src/common/syncjournaldb.cpp	77.42% <0.00%> (-0.22%)	⬇️
src/libsync/bulkpropagatorjob.cpp	73.10% <0.00%> (-0.15%)	⬇️
... and 6 more

[nextcloud-desktop-bot]

AppImage file: nextcloud-PR-4998-74f35191e0e78eb6df4527c299b28fc469bc8066-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication