minor cleanups

src/shard_cfg.c

 /*-
- * Copyright 2016 UPLEX - Nils Goroll Systemoptimierung
+ * Copyright 2009-2016 UPLEX - Nils Goroll Systemoptimierung
  * All rights reserved.
  *
- * Author: Nils Goroll <nils.goroll@uplex.de>
+ * Authors: Nils Goroll <nils.goroll@uplex.de>
+ *	    Geoffrey Simmons <geoff@uplex.de>
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -50,7 +51,7 @@ enum shard_change_task_e {
 
 struct shard_change_task {
 	unsigned				magic;
-#define SHARD_CHANGE_TASK_MAGIC		0x1e1168af
+#define SHARD_CHANGE_TASK_MAGIC			0x1e1168af
 	enum shard_change_task_e		task;
 	void					*priv;
 	VSTAILQ_ENTRY(shard_change_task)	list;

src/shard_dir.c

@@ -278,8 +278,12 @@ init_state(struct shard_state *state,
  *   which probably is, for all practical purposes, only relevant when the
  *   saintmode vmod is used
  *
- * on the other hand, if we wanted to postpone pick_be until backend conncetion
- * time, we'd need to keep per-request per-director state
+ * if we wanted to offer delayed resolution, we'd need something like
+ * per-request per-director state or we'd need to return a dynamically created
+ * director object. That should be straight forward once we got director
+ * refcounting #2072. Until then, we could create it on the workspace, but then
+ * we'd need to keep other directors from storing any references to our dynamic
+ * object for longer than the current task
  *
  */
 VCL_BACKEND

src/shard_dir.h

@@ -64,7 +64,7 @@ struct sharddir {
 
 	VCL_DURATION				rampup_duration;
 	VCL_REAL				warmup;
-	VCL_INT				replicas;
+	VCL_INT					replicas;
 };
 
 static inline VCL_BACKEND

src/shard_hash.c

@@ -67,9 +67,9 @@ shard_hash_sha256(VCL_STRING s)
 	SHA256_Final(sha256_digest.digest, &sha256);
 
 	/*
-	* use low 32 bits only
-	* XXX: Are these the best bits to pick?
-	*/
+	 * use low 32 bits only
+	 * XXX: Are these the best bits to pick?
+	 */
 	return (sha256_digest.uint32_digest[7]);
 }