summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatt Ranostay <matt.ranostay@konsulko.com>2019-10-31 06:59:00 -0700
committerMatt Ranostay <matt.ranostay@konsulko.com>2019-11-12 22:12:03 -0800
commitbbdc6f93e73a7053616f21c270e445db50e26ad2 (patch)
treeefdfe3708495127b9ea70440aaa813719ae0ab0d
parent81583bbfa365a0c260cc67f97941ac0e623e825a (diff)
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 <matt.ranostay@konsulko.com>
-rw-r--r--MainApp.cpp11
1 files changed, 4 insertions, 7 deletions
diff --git a/MainApp.cpp b/MainApp.cpp
index 59e049c..1b05538 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
@@ -625,8 +624,7 @@ void MainApp::DisplayInformation(bool display, bool RefreshDisplay)
void MainApp::networkReplySearch(QNetworkReply* reply)
{
- char buf[BIG_BUFFER_SIZE];
- int buflen;
+ QByteArray buf;
mutex.lock();
@@ -643,10 +641,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;
@@ -656,7 +653,7 @@ void MainApp::networkReplySearch(QNetworkReply* reply)
currentIndex = 0;
Businesses.clear();
- ParseJsonBusinessList(buf, Businesses);
+ ParseJsonBusinessList(buf.data(), Businesses);
DisplayResultList(true);
FillResultList(Businesses);
}