From c54faac4464c4b2b6e6c27f62d9394f8ccc4214b Mon Sep 17 00:00:00 2001 From: Matt Ranostay Date: Thu, 31 Oct 2019 06:59:00 -0700 Subject: poi-yelp: don't use fixed buffer size Use QByteArray over arbitrary buffer size for QNetworkReply Bug-AGL: SPEC-2880 Change-Id: I65f04603afc22ef2f8ae5cfe97f747dda85e692d Signed-off-by: Matt Ranostay --- MainApp.cpp | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/MainApp.cpp b/MainApp.cpp index cbf6d57..bb2ffc6 100644 --- a/MainApp.cpp +++ b/MainApp.cpp @@ -24,7 +24,6 @@ #define URL_AUTOCOMPLETE "https://api.yelp.com/v3/autocomplete" #define URL_SEARCH "https://api.yelp.com/v3/businesses/search" -#define BIG_BUFFER_SIZE (1024*1024) #define LEFT_OFFSET 28 #define FONT_SIZE_LINEDIT 20 #define FONT_SIZE_LIST 18 @@ -629,8 +628,7 @@ void MainApp::DisplayInformation(bool display, bool RefreshDisplay) void MainApp::networkReplySearch(QNetworkReply* reply) { - char buf[BIG_BUFFER_SIZE]; - int buflen; + QByteArray buf; mutex.lock(); @@ -647,10 +645,9 @@ void MainApp::networkReplySearch(QNetworkReply* reply) return; } - buflen = reply->read(buf, BIG_BUFFER_SIZE-1); - buf[buflen] = '\0'; + buf = reply->readAll(); - if (buflen == 0) + if (buf.isEmpty()) { mutex.unlock(); return; @@ -660,7 +657,7 @@ void MainApp::networkReplySearch(QNetworkReply* reply) currentIndex = 0; Businesses.clear(); - ParseJsonBusinessList(buf, Businesses); + ParseJsonBusinessList(buf.data(), Businesses); DisplayResultList(true); FillResultList(Businesses); } -- cgit 1.2.3-korg