Ignore all errors in DeriveValueInstanceTypes, errors will be detecte…

…d by `TypeValidator`

If just return CompilationProblems from this pass, we will report some errors twice,
because `TypeValidator` alro reports errors from the `TypeDetector`

shared/src/main/scala/io/kaitai/struct/precompile/DeriveValueInstanceTypes.scala

@@ -59,7 +59,7 @@ class DeriveValueInstanceTypes(specs: ClassSpecs) extends PrecompileStep {
                     Log.typeProcValue.info(() => s"${instName.name} type undecided: ${tue.getMessage}")
                     // just ignore, we're not there yet, probably we'll get it on next iteration
                   case err: ExpressionError =>
-                    throw ErrorInInput(err, vi.path ++ List("value")).toException
+                    // Ignore all errors, the validation will be performed in TypeValidator pass later
                 }
               case Some(_) =>
                 // already derived, do nothing

shared/src/main/scala/io/kaitai/struct/precompile/TypeValidator.scala

@@ -105,6 +105,10 @@ class TypeValidator(specs: ClassSpecs) extends PrecompileStep {
 
   def validateValueInstance(vis: ValueInstanceSpec): Option[CompilationProblem] = {
     try {
+      // detectType performs some additional checks that validate does not (for example,
+      // applicability of operators to types, like `Not` to numbers).
+      // TODO: probably implement those checks in validate too?
+      detector.detectType(vis.value)
       detector.validate(vis.value)
       None
     } catch {