Remove the "volatile" attribute from bpf_error(). The "volatile"

attribute is an obsolete way of saying "noreturn".  clang doesn't
recognize it and the function is already marked __dead so it is
superfluous anyway.

diff --git a/lib/libpcap/gencode.h b/lib/libpcap/gencode.h
index 1fac5a8..cb4d34c 100644 (file)
--- a/lib/libpcap/gencode.h
+++ b/lib/libpcap/gencode.h
@@ -1,4 +1,4 @@
-/*     $OpenBSD: gencode.h,v 1.16 2014/04/11 04:08:58 lteo Exp $       */
+/*     $OpenBSD: gencode.h,v 1.17 2017/04/27 15:46:14 millert Exp $    */
 
 /*
  * Copyright (c) 1990, 1991, 1992, 1993, 1994, 1995, 1996
@@ -193,7 +193,7 @@ struct block *gen_p80211_fcdir(int);
 
 void bpf_optimize(struct block **);
 __dead void bpf_error(const char *, ...)
-    __attribute__((volatile, __format__ (printf, 1, 2)));
+    __attribute__((__format__ (printf, 1, 2)));
 
 void finish_parse(struct block *);
 char *sdup(const char *);