D21159: Replace sprintf with QString::asprintf

Nicolas Fella noreply at phabricator.kde.org
Sun May 12 17:55:39 BST 2019


nicolasfella created this revision.
Herald added a project: KDE PIM.
Herald added a subscriber: kde-pim.
nicolasfella requested review of this revision.

REVISION SUMMARY
  sprintf is obsolete and causes build fail with current Qt dev. Use asprintf

TEST PLAN
  ctest
  
  Some tests fail, but they did before:
  472 - Compat-libical3-AppleICal_1.5.ics (Failed)
  473 - Compat-libical3-Evolution_2.8.2_timezone_test.ics (Failed)
  475 - Compat-libical3-KOrganizer_3.1a.ics (Failed)
  477 - Compat-libical3-MSExchange.ics (Failed)
  478 - Compat-libical3-Mozilla_1.0.ics (Failed)

REPOSITORY
  R172 KCalendar Core

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D21159

AFFECTED FILES
  autotests/testperson.cpp
  src/person.cpp
  src/person.h
  src/sorting.cpp
  src/sorting.h

To: nicolasfella
Cc: kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20190512/95c33946/attachment.html>


More information about the kde-pim mailing list