mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Charlie Carson" <>
Subject Review Request 17568: Change stout JSON::Value default to be JSON::Null
Date Fri, 31 Jan 2014 00:58:09 GMT

This is an automatically generated e-mail. To reply, visit:

Review request for mesos, Benjamin Hindman and Jeff Currier.


Repository: mesos-git


    Change stout JSON::Value default to be JSON::Null

    The first type specified for the variant which is JSON::Value
    is the default.  Currently that is JSON::String which in turn
    defaults to empty string.  JSON::Null would be a more correct
    default value.

    This change moves JSON::Null to the front of the declaration
    so that it becomes the default and adds a unit test.


  3rdparty/libprocess/3rdparty/stout/include/stout/json.hpp 3148a7873397f6b0b2ebdbff3b640535ccd12318

  3rdparty/libprocess/3rdparty/stout/tests/json_tests.cpp 29ada8a4a0396f889eb2583c3b3ff622050125af



make check
added unit test


Charlie Carson

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message