12
20%
master: 20%

Ran 13 Sep 2017 05:17AM UTC

Files 8

Run time 0s

Badge

Committed 13 Sep 2017 05:17AM UTC coverage: 99.368%. First build

Job # TEST_NAME="make cross (automake)" THRIFT_CROSSTEST_CONCURRENCY=6 CONFIG="--enable-tutorial=no --without-lua --without-haxe --without-d" ALL_DEPS="yes" MAKE_TARGET="cross" ERROR_LOG="test/log/unexpected_failures.log"

Build Type

Pull #687

travis-ci

Committed by Max Spencer

Commit Message

THRIFT-2974 fix optional writeToParcel

Fix for [THRIFT-2974](https://issues.apache.org/jira/browse/THRIFT-2974?jql=text%20~%20%22optional%20writeToParcel%22) which causes NPEs whenever struct with an unset optional enum is written to an Android [`Parcel`](http://developer.android.com/reference/android/os/Parcel.html#writeInt%28int%29).

[Thrift IDL docs](https://thrift.apache.org/docs/idl#enum) specify that enum values must be greater than or equal to zero so we can write -1 to the parcel to represent an unset enum. The corresponding code for reconstructing the struct (see line 1728) uses the `TEnum.findByValue` method which will return `null` for any unused enum values, which will always include -1.

Pull Request Pull Request #687: THRIFT-2974 fix optional writeToParcel

Run Details

629 of 633 relevant lines covered (99.37%)

7.29 hits per line

Source Files on job 1937.12 (TEST_NAME="make cross (automake)" THRIFT_CROSSTEST_CONCURRENCY=6 CONFIG="--enable-tutorial=no --without-lua --without-haxe --without-d" ALL_DEPS="yes" MAKE_TARGET="cross" ERROR_LOG="test/log/unexpected_failures.log")

apache / thrift / 1937 / 12
20%
master: 20%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1937.12 (TEST_NAME="make cross (automake)" THRIFT_CROSSTEST_CONCURRENCY=6 CONFIG="--enable-tutorial=no --without-lua --without-haxe --without-d" ALL_DEPS="yes" MAKE_TARGET="cross" ERROR_LOG="test/log/unexpected_failures.log")

apache / thrift / 1937 / 12 20% master: 20%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1937.12 (TEST_NAME="make cross (automake)" THRIFT_CROSSTEST_CONCURRENCY=6 CONFIG="--enable-tutorial=no --without-lua --without-haxe --without-d" ALL_DEPS="yes" MAKE_TARGET="cross" ERROR_LOG="test/log/unexpected_failures.log")

apache / thrift / 1937 / 12
20%
master: 20%

README BADGES
x