See <https://builds.apache.org/job/Thrift/317/changes>

Changes:

[jfarrell] Thrift-1415: delphi: add version Info to the library
Client: delphi
Patch: Jens Geyer 

Adding versioning to delphi library

[jfarrell] Thrift-1277: Node.js serializes false booleans as null
Client: js
Patch: Hans Duedal 

When serializing boolean types, false booleans are serialized as null instead 
of false switching to  using loose compare instead

------------------------------------------
[...truncated 2183 lines...]
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:1283:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.success != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:1358:18:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.thing != null) {
[jsl:jslint]                  ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:1433:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.success != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:1495:18:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.thing != null) {
[jsl:jslint]                  ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:1548:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.success != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:1601:18:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.thing != null) {
[jsl:jslint]                  ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:1654:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.success != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:1707:18:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.hello != null) {
[jsl:jslint]                  ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:1804:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.success != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:1878:21:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.argument != null) {
[jsl:jslint]                     ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:1976:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.success != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2123:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.arg0 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2128:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.arg1 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2133:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.arg2 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2138:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.arg3 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2153:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.arg4 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2158:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.arg5 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2212:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.success != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2265:16:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.arg != null) {
[jsl:jslint]                ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2319:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.err1 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2380:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.arg0 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2385:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.arg1 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2460:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.success != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2465:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.err1 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2470:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.err2 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest.js>:2523:27:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.secondsToSleep != null) {
[jsl:jslint]                           ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:70:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.message != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:75:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.type != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:136:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.im_true != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:141:21:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.im_false != null) {
[jsl:jslint]                     ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:224:25:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.string_thing != null) {
[jsl:jslint]                         ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:229:23:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.byte_thing != null) {
[jsl:jslint]                       ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:234:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.i32_thing != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:239:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.i64_thing != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:312:23:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.byte_thing != null) {
[jsl:jslint]                       ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:317:25:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.struct_thing != null) {
[jsl:jslint]                         ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:322:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.i32_thing != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:405:25:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.string_thing != null) {
[jsl:jslint]                         ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:410:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.changed != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:415:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.i32_thing != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:420:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.i64_thing != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:517:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.userMap != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:532:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.xtructs != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:757:25:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.string_field != null) {
[jsl:jslint]                         ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:762:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.set_field != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:776:23:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.list_field != null) {
[jsl:jslint]                       ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:847:25:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.binary_field != null) {
[jsl:jslint]                         ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:909:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.errorCode != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:914:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.message != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:977:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.errorCode != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:982:25:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.struct_thing != null) {
[jsl:jslint]                         ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1064:18:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.field != null) {
[jsl:jslint]                  ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1136:26:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.begin_in_both != null) {
[jsl:jslint]                          ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1141:23:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.old_string != null) {
[jsl:jslint]                       ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1146:24:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.end_in_both != null) {
[jsl:jslint]                        ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1366:26:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.begin_in_both != null) {
[jsl:jslint]                          ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1371:19:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.newint != null) {
[jsl:jslint]                   ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1376:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.newbyte != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1381:21:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.newshort != null) {
[jsl:jslint]                     ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1386:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.newlong != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1391:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.newdouble != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1396:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.newstruct != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1401:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.newlist != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1415:19:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.newset != null) {
[jsl:jslint]                   ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1429:19:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.newmap != null) {
[jsl:jslint]                   ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1444:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.newstring != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1449:24:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.end_in_both != null) {
[jsl:jslint]                        ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1523:19:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.myints != null) {
[jsl:jslint]                   ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1537:18:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.hello != null) {
[jsl:jslint]                  ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1611:20:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.strings != null) {
[jsl:jslint]                    ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1625:18:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.hello != null) {
[jsl:jslint]                  ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1700:22:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.map_field != null) {
[jsl:jslint]                      ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1891:15:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.b1 != null) {
[jsl:jslint]               ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1896:16:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.b10 != null) {
[jsl:jslint]                ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1901:17:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.b100 != null) {
[jsl:jslint]                 ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1906:23:
 Use '!==' to compare with 'null'.
[jsl:jslint]   if (this.check_true != null) {
[jsl:jslint]                       ^
[jsl:jslint] 
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/gen-js/ThriftTest_types.js>:1906:23:
 Too many errors. (70% scanned).

BUILD FAILED
<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test/build.xml>:216: 
JSLint: 96 errors in 2 files

Total time: 6 seconds
make[3]: *** [check-local] Error 1
make[3]: Leaving directory 
`<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test'>
make[2]: *** [check-am] Error 2
make[2]: Leaving directory 
`<https://builds.apache.org/job/Thrift/ws/thrift/lib/js/test'>
make[1]: *** [check-recursive] Error 1
make[1]: Leaving directory 
`<https://builds.apache.org/job/Thrift/ws/thrift/lib'>
make: *** [check-recursive] Error 1
Build step 'Execute shell' marked build as failure
[WARNINGS] Skipping publisher since build result is FAILURE
[TASKS] Skipping publisher since build result is FAILURE
Archiving artifacts
Recording test results
Publishing Javadoc
Updating THRIFT-1277
Updating THRIFT-1415

Reply via email to