diff options
| author |   Matt Ranostay <matt.ranostay@konsulko.com> | 2019-10-31 06:59:00 -0700 |
|---|---|---|
| committer | Matt Ranostay <matt.ranostay@konsulko.com> | 2019-11-11 00:45:54 -0800 |
| commit | c54faac4464c4b2b6e6c27f62d9394f8ccc4214b (patch) | |
| tree | 42ba6459a71af094e66878f5dc64bc989cacc082 /MainApp.cpp | |
| parent | 0d6fc382050f1f2efa10ff00e34a18706a350a4e (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>
Diffstat (limited to 'MainApp.cpp')
| -rw-r--r-- | MainApp.cpp | 11 |
1 files 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); } |