diff --git a/configfile-scan.l b/configfile-scan.l
index d4da6f0f..59127b0a 100644
--- a/configfile-scan.l
+++ b/configfile-scan.l
@@ -109,7 +109,7 @@ ProcessRepmgrConfigFile(const char *config_file, const char *base_dir, ItemList
 extern bool
 ProcessPostgresConfigFile(const char *config_file, const char *base_dir, KeyValueList *contents, ItemList *error_list, ItemList *warning_list)
 {
-	printf("ProcessPostgresConfigFile(): base: %s file: %s\n", base_dir, config_file);
+	//printf("ProcessPostgresConfigFile(): base: %s file: %s\n", base_dir, config_file);
 
 	return ProcessConfigFile(base_dir, config_file, NULL, true, contents, error_list, warning_list);
 }
diff --git a/configfile.c b/configfile.c
index 9a2cc89e..cebf9c72 100644
--- a/configfile.c
+++ b/configfile.c
@@ -1486,7 +1486,7 @@ repmgr_atoi(const char *value, const char *config_item, ItemList *error_list, in
 void
 repmgr_canonicalize_path(const char *name, const char *value, char *config_item, ItemList *errors)
 {
-	// XXX check for errors?
+	/* NOTE: canonicalize_path does not produce any errors */
 	canonicalize_path(config_item);
 }
 
@@ -1711,19 +1711,6 @@ modify_auto_conf(const char *data_dir, KeyValueList *items)
 	appendPQExpBuffer(&auto_conf, "%s/%s",
 					  data_dir, PG_AUTOCONF_FILENAME);
 
-	// XXX do we need this?
-	fp = fopen(auto_conf.data, "r");
-
-	if (fp == NULL)
-	{
-		fprintf(stderr, "unable to open \"%s\": %s\n",
-				auto_conf.data,
-				strerror(errno));
-		termPQExpBuffer(&auto_conf);
-		return false;
-	}
-	fclose(fp);
-
 	success = ProcessPostgresConfigFile(auto_conf.data, NULL, &config, NULL, NULL);
 
 	if (success == false)