commit c067179b6a402fc0c284cfae886058a6738e9b20
author Yin-Chia Yeh <yinchiayeh@google.com> Fri Mar 02 15:50:48 2018 -0800
committer Yin-Chia Yeh <yinchiayeh@google.com> Fri Mar 02 15:50:48 2018 -0800
tree 8f661848279b9ee8bb7f6a833f493c7a4123f530
parent 0fd28115e485cfe9202c1eec777a6cc17663651a

Camera: fix supported FPS string

Test: CameraTest.java
Bug: 72261912
Change-Id: I0737b7411ad52159477ba5d931607067d567128e

services/camera/libcameraservice/api1/client2/Parameters.cpp

diff --git a/services/camera/libcameraservice/api1/client2/Parameters.cpp b/services/camera/libcameraservice/api1/client2/Parameters.cpp
index b4c7e9d..8d8bcab 100644
--- a/services/camera/libcameraservice/api1/client2/Parameters.cpp
+++ b/services/camera/libcameraservice/api1/client2/Parameters.cpp
@@ -242,7 +242,9 @@
                 HAL_PIXEL_FORMAT_IMPLEMENTATION_DEFINED, availableFpsRanges.data.i32[i+1])) {
                 continue;
             }
-            if (i != 0) supportedPreviewFpsRange += ",";
+            if (supportedPreviewFpsRange.length() > 0) {
+                supportedPreviewFpsRange += ",";
+            }
             supportedPreviewFpsRange += String8::format("(%d,%d)",
                     availableFpsRanges.data.i32[i] * kFpsToApiScale,
                     availableFpsRanges.data.i32[i+1] * kFpsToApiScale);