Check the comments array boundry.

[dota17]

Currently, a Segmentation fault (core dumped) caused by the wrong use of setComment method:

root.setComment(commentPtr, Json::numberOfCommentPlacement);

we should check the comment array boundry before we assign comment value to Array to avoid this seg fault.

[dota17]

The Travis CI failed in Linux xenial gcc cmake coverage:

CMake Error at include/cmake_install.cmake:41 (file):
  file INSTALL cannot find
  "/home/travis/build/open-source-parsers/jsoncpp/build-cmake_Debug_shared_g++_make/include/json/version.h".
Call Stack (most recent call first):
  cmake_install.cmake:70 (include)

[baylesj]

Fixed the travis build issue, can you try force pushing to this branch to restart the travis checks?

[dota17]

@baylesj should we throw a error or exception when users write code like:

root.setComment(commentPtr, Json::numberOfCommentPlacement);

In this PR, I do nothing in this case, just ignore the boundry action.

[baylesj]

numberOfCommentPlacement is an awkward name I would be in favor of changing. This patch is good, though.