Fix deprecated use of 0/NULL in velox/tpch/gen/dbgen/bm_utils.cpp + 1 (facebookincubator#12612)

Summary:
Pull Request resolved: facebookincubator#12612

`nullptr` is typesafe. `0` and `NULL` are not. In the future, only `nullptr` will be allowed.

This diff helps us embrace the future _now_ in service of enabling `-Wzero-as-null-pointer-constant`.

Reviewed By: dtolnay

Differential Revision: D70939272

fbshipit-source-id: 063f2275ba217ec2dc7438ef80f603a86ea4c5ba

Author: r-barnes

velox/tpch/gen/dbgen/bm_utils.cpp

@@ -116,7 +116,7 @@ void permute_dist(distribution* d, seed_t* seed);
 const char* tpch_env_config(const char* var, const char* dflt) {
   static char* evar;
 
-  if ((evar = getenv(var)) != NULL)
+  if ((evar = getenv(var)) != nullptr)
     return (evar);
   else
     return (dflt);
@@ -252,17 +252,17 @@ void read_dist(const char* path, const char* name, distribution* target) {
   long weight, count = 0, name_set = 0;
 
   while (read_line_into_buffer(line, sizeof(line), &src)) {
-    if ((c = strchr(line, '\n')) != NULL)
+    if ((c = strchr(line, '\n')) != nullptr)
       *c = '\0';
-    if ((c = strchr(line, '#')) != NULL)
+    if ((c = strchr(line, '#')) != nullptr)
       *c = '\0';
     if (*line == '\0')
       continue;
 
     if (!name_set) {
       if (dsscasecmp(strtok(line, "\n\t "), "BEGIN"))
         continue;
-      if (dsscasecmp(strtok(NULL, "\n\t "), name))
+      if (dsscasecmp(strtok(nullptr, "\n\t "), name))
         continue;
       name_set = 1;
       continue;