RIF Add null checks where appropriate

Author: cquiroz

shared/src/main/scala/org/threeten/bp/Instant.scala

@@ -447,7 +447,10 @@ final class Instant private(private val seconds: Long, private val nanos: Int) e
     * @throws DateTimeException if a value for the field cannot be obtained
     * @throws ArithmeticException if numeric overflow occurs
     */
-  def getLong(field: TemporalField): Long =
+  def getLong(field: TemporalField): Long = {
+    if (field == null) {
+      throw new NullPointerException()
+    }
     if (field.isInstanceOf[ChronoField])
       field.asInstanceOf[ChronoField] match {
         case NANO_OF_SECOND  => nanos
@@ -458,6 +461,7 @@ final class Instant private(private val seconds: Long, private val nanos: Int) e
       }
     else
       field.getFrom(this)
+  }
 
   /** Gets the number of seconds from the Java epoch of 1970-01-01T00:00:00Z.
     *
@@ -765,13 +769,17 @@ final class Instant private(private val seconds: Long, private val nanos: Int) e
     * @throws DateTimeException if unable to query (defined by the query)
     * @throws ArithmeticException if numeric overflow occurs (defined by the query)
     */
-  override def query[R >: Null](query: TemporalQuery[R]): R =
+  override def query[R >: Null](query: TemporalQuery[R]): R = {
+    if (query == null) {
+      throw new NullPointerException()
+    }
     if (query eq TemporalQueries.precision)
       NANOS.asInstanceOf[R]
     else if ((query eq TemporalQueries.localDate) || (query eq TemporalQueries.localTime) || (query eq TemporalQueries.chronology) || (query eq TemporalQueries.zoneId) || (query eq TemporalQueries.zone) || (query eq TemporalQueries.offset))
       null
     else
       query.queryFrom(this)
+  }
 
   /** Adjusts the specified temporal object to have this instant.
     *
@@ -943,6 +951,9 @@ final class Instant private(private val seconds: Long, private val nanos: Int) e
     * @throws NullPointerException if otherInstant is null
     */
   def compare(otherInstant: Instant): Int = {
+    if (otherInstant == null) {
+      throw new NullPointerException("null object in comparison")
+    }
     val cmp: Int = java.lang.Long.compare(seconds, otherInstant.seconds)
     if (cmp != 0) cmp
     else nanos - otherInstant.nanos

shared/src/main/scala/org/threeten/bp/temporal/TemporalAccessor.scala

@@ -114,12 +114,16 @@ trait TemporalAccessor {
     * @return the range of valid values for the field, not null
     * @throws DateTimeException if the range for the field cannot be obtained
     */
-  def range(field: TemporalField): ValueRange =
+  def range(field: TemporalField): ValueRange = {
+    if (field == null) {
+      throw new NullPointerException()
+    }
     if (field.isInstanceOf[ChronoField])
       if (isSupported(field)) field.range
       else throw new UnsupportedTemporalTypeException(s"Unsupported field: $field")
     else
       field.rangeRefinedBy(this)
+  }
 
   /** Gets the value of the specified field as an {@code int}.
     *