You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2020/11/17 22:46:32 UTC
[GitHub] [calcite] vlsi opened a new pull request #2268: [CALCITE-4199] Add nullability annotations to the methods and fields, ensure consistency with checkerframework
vlsi opened a new pull request #2268:
URL: https://github.com/apache/calcite/pull/2268
https://issues.apache.org/jira/browse/CALCITE-4199
CheckerFramework needs an annotated JDK (which might be slow to download), so it is not activated by default
Third-party issues:
* https://github.com/gradle/gradle/issues/14332: Order classes by recent failures and modification date when feeding them to javac compiler
* https://github.com/policeman-tools/forbidden-apis/issues/173: Ability to skip class file parse errors
* https://github.com/typetools/checker-framework/issues/3616: Infer annotations for receiver of instanceof checks
* https://github.com/typetools/checker-framework/issues/3614: False alarm when using "return null" and type is known to support nulls
* https://github.com/typetools/checker-framework/issues/3612: Crash in AbstractAtmComboVisitor visitArray_Typevar .. defaultAction
Corner cases:
* https://github.com/typetools/checker-framework/issues/399 checker framework does not understand `if (queue.isEmpty()) { ... = queue.poll(); }`, and it treats `poll()` as always nullable.
* Arrays vs annotations: https://twitter.com/tagir_valeev/status/1301443075080949760, https://twitter.com/tagir_valeev/status/1302173719398436864
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [calcite] vlsi merged pull request #2268: [CALCITE-4199] Add nullability annotations to the methods and fields, ensure consistency with checkerframework
Posted by GitBox <gi...@apache.org>.
vlsi merged pull request #2268:
URL: https://github.com/apache/calcite/pull/2268
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org