parser: Remove unnecessary NULL checks above cStrIdxOrMemberDeref() calls

Now that cStrIdxOrMemberDeref() checks for NULL itself (85d8362),
callers don't need to do it anymore.

Author: dkl

src/compiler/parser-expr-unary.bas

@@ -222,18 +222,12 @@ function cHighestPrecExpr _
 	'' DerefExpr
 	case FB_TK_DEREFCHAR
 		expr = cDerefExpression( )
-		if( expr = NULL ) then
-			return NULL
-		end if
 
 	'' ParentExpression
 	case CHAR_LPRNT
 		dim as integer is_opt = fbGetPrntOptional( )
 
 		expr = cParentExpression( )
-		if( expr = NULL ) then
-			return NULL
-		end if
 
 		'' if parsing a SUB, don't call StrIdxOrMemberDeref() twice
 		if( is_opt ) then
@@ -253,26 +247,17 @@ function cHighestPrecExpr _
 			lexSkipToken( )
 
 			expr = hCast( 0 )
-			if( expr = NULL ) then
-				return NULL
-			end if
 
 		'' CPTR '(' DataType ',' Expression{int|uint|ptr} ')'
 		case FB_TK_CPTR
 			'' CPTR
 			lexSkipToken( )
 
 			expr = hCast( AST_CONVOPT_PTRONLY )
-			if( expr = NULL ) then
-				return NULL
-			end if
 
 		'' OperatorNew
 		case FB_TK_NEW
 			expr = cOperatorNew( )
-			if( expr = NULL ) then
-				return NULL
-			end if
 
 		'' Atom
 		case else
@@ -705,15 +690,11 @@ function cAddrOfExpression( ) as ASTNODE ptr
 
 	end select
 
-	if( expr ) then
-		'' Allow indexing on VARPTR()/STRPTR()/etc. directly, they look
-		'' like functions so this isn't ambigious, while for @ it would
-		'' mess up the operator precedence:
-		''    @expr[i]  should be  @(expr[i]), not (@expr)[i]
-		'' but for
-		''    varptr(expr)[i], that problem doesn't exist.
-		expr = cStrIdxOrMemberDeref( expr )
-	end if
-
-	function = expr
+	'' Allow indexing on VARPTR()/STRPTR()/etc. directly, they look
+	'' like functions so this isn't ambigious, while for @ it would
+	'' mess up the operator precedence:
+	''    @expr[i]  should be  @(expr[i]), not (@expr)[i]
+	'' but for
+	''    varptr(expr)[i], that problem doesn't exist.
+	function = cStrIdxOrMemberDeref( expr )
 end function