* [Xen-devel] [PATCH for-4.13] tools: Fix local variable block
@ 2019-11-01 19:13 Andrew Cooper
2019-11-02 12:24 ` Wei Liu
2019-11-08 6:09 ` Jürgen Groß
0 siblings, 2 replies; 3+ messages in thread
From: Andrew Cooper @ 2019-11-01 19:13 UTC (permalink / raw)
To: Xen-devel; +Cc: Juergen Gross, Andrew Cooper, Wei Liu, Ian Jackson
c-indent-level isn't considered a safe variable, and "solaris" isn't a
recognised C style. Both cause prompts when opening the files.
Fix all blocks up per CODING_STYLE
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Ian Jackson <Ian.Jackson@citrix.com>
CC: Wei Liu <wl@xen.org>
CC: Juergen Gross <jgross@suse.com>
This finally irritated me to the point at which I decided to fix it. It would
be nice to get into 4.13, and has 0 risk, as it is only comment changes.
---
tools/console/daemon/io.c | 2 +-
tools/console/daemon/main.c | 2 +-
tools/include/xen-sys/SunOS/evtchn.h | 4 ++--
tools/python/xen/lowlevel/xc/xc.c | 2 +-
tools/python/xen/lowlevel/xs/xs.c | 2 +-
tools/xenstore/include/xenstore.h | 2 +-
tools/xenstore/utils.h | 2 +-
tools/xenstore/xenstored_core.c | 2 +-
tools/xenstore/xenstored_core.h | 2 +-
tools/xenstore/xenstored_domain.c | 2 +-
tools/xenstore/xenstored_transaction.c | 2 +-
tools/xenstore/xenstored_watch.c | 2 +-
tools/xenstore/xs.c | 2 +-
13 files changed, 14 insertions(+), 14 deletions(-)
diff --git a/tools/console/daemon/io.c b/tools/console/daemon/io.c
index 8dac279bbb..a43c57edad 100644
--- a/tools/console/daemon/io.c
+++ b/tools/console/daemon/io.c
@@ -1470,9 +1470,9 @@ void handle_io(void)
/*
* Local variables:
+ * mode: C
* c-file-style: "linux"
* indent-tabs-mode: t
- * c-indent-level: 8
* c-basic-offset: 8
* tab-width: 8
* End:
diff --git a/tools/console/daemon/main.c b/tools/console/daemon/main.c
index 5d9d55b430..30124a1c19 100644
--- a/tools/console/daemon/main.c
+++ b/tools/console/daemon/main.c
@@ -216,9 +216,9 @@ int main(int argc, char **argv)
/*
* Local variables:
+ * mode: C
* c-file-style: "linux"
* indent-tabs-mode: t
- * c-indent-level: 8
* c-basic-offset: 8
* tab-width: 8
* End:
diff --git a/tools/include/xen-sys/SunOS/evtchn.h b/tools/include/xen-sys/SunOS/evtchn.h
index abc8c60037..e9d08d9dfc 100644
--- a/tools/include/xen-sys/SunOS/evtchn.h
+++ b/tools/include/xen-sys/SunOS/evtchn.h
@@ -85,9 +85,9 @@ struct ioctl_evtchn_notify {
/*
* Local variables:
- * c-file-style: "solaris"
+ * mode: C
+ * c-file-style: "linux"
* indent-tabs-mode: t
- * c-indent-level: 8
* c-basic-offset: 8
* tab-width: 8
* End:
diff --git a/tools/python/xen/lowlevel/xc/xc.c b/tools/python/xen/lowlevel/xc/xc.c
index 26b9a0fb74..44d3606141 100644
--- a/tools/python/xen/lowlevel/xc/xc.c
+++ b/tools/python/xen/lowlevel/xc/xc.c
@@ -2661,7 +2661,7 @@ PyMODINIT_FUNC initxc(void)
/*
* Local variables:
- * c-indent-level: 4
+ * mode: C
* c-basic-offset: 4
* End:
*/
diff --git a/tools/python/xen/lowlevel/xs/xs.c b/tools/python/xen/lowlevel/xs/xs.c
index 621039d7a7..b7d4b6ef5d 100644
--- a/tools/python/xen/lowlevel/xs/xs.c
+++ b/tools/python/xen/lowlevel/xs/xs.c
@@ -1079,7 +1079,7 @@ PyMODINIT_FUNC initxs(void)
/*
* Local variables:
- * c-indent-level: 4
+ * mode: C
* c-basic-offset: 4
* End:
*/
diff --git a/tools/xenstore/include/xenstore.h b/tools/xenstore/include/xenstore.h
index 42b2866462..254f556a0b 100644
--- a/tools/xenstore/include/xenstore.h
+++ b/tools/xenstore/include/xenstore.h
@@ -276,9 +276,9 @@ int xs_suspend_evtchn_port(int domid);
/*
* Local variables:
+ * mode: C
* c-file-style: "linux"
* indent-tabs-mode: t
- * c-indent-level: 8
* c-basic-offset: 8
* tab-width: 8
* End:
diff --git a/tools/xenstore/utils.h b/tools/xenstore/utils.h
index dc3a0a0f8b..522c3594a2 100644
--- a/tools/xenstore/utils.h
+++ b/tools/xenstore/utils.h
@@ -52,9 +52,9 @@ static inline long IS_ERR(const void *ptr)
/*
* Local variables:
+ * mode: C
* c-file-style: "linux"
* indent-tabs-mode: t
- * c-indent-level: 8
* c-basic-offset: 8
* tab-width: 8
* End:
diff --git a/tools/xenstore/xenstored_core.c b/tools/xenstore/xenstored_core.c
index c8e423700d..97ceabf964 100644
--- a/tools/xenstore/xenstored_core.c
+++ b/tools/xenstore/xenstored_core.c
@@ -2151,9 +2151,9 @@ int main(int argc, char *argv[])
/*
* Local variables:
+ * mode: C
* c-file-style: "linux"
* indent-tabs-mode: t
- * c-indent-level: 8
* c-basic-offset: 8
* tab-width: 8
* End:
diff --git a/tools/xenstore/xenstored_core.h b/tools/xenstore/xenstored_core.h
index 3d27feb91a..56a279cfbb 100644
--- a/tools/xenstore/xenstored_core.h
+++ b/tools/xenstore/xenstored_core.h
@@ -212,9 +212,9 @@ int remember_string(struct hashtable *hash, const char *str);
/*
* Local variables:
+ * mode: C
* c-file-style: "linux"
* indent-tabs-mode: t
- * c-indent-level: 8
* c-basic-offset: 8
* tab-width: 8
* End:
diff --git a/tools/xenstore/xenstored_domain.c b/tools/xenstore/xenstored_domain.c
index c793543bb4..1a83097952 100644
--- a/tools/xenstore/xenstored_domain.c
+++ b/tools/xenstore/xenstored_domain.c
@@ -979,9 +979,9 @@ void wrl_apply_debit_trans_commit(struct connection *conn)
/*
* Local variables:
+ * mode: C
* c-file-style: "linux"
* indent-tabs-mode: t
- * c-indent-level: 8
* c-basic-offset: 8
* tab-width: 8
* End:
diff --git a/tools/xenstore/xenstored_transaction.c b/tools/xenstore/xenstored_transaction.c
index 75816dd2c7..2824f7b359 100644
--- a/tools/xenstore/xenstored_transaction.c
+++ b/tools/xenstore/xenstored_transaction.c
@@ -612,9 +612,9 @@ int check_transactions(struct hashtable *hash)
/*
* Local variables:
+ * mode: C
* c-file-style: "linux"
* indent-tabs-mode: t
- * c-indent-level: 8
* c-basic-offset: 8
* tab-width: 8
* End:
diff --git a/tools/xenstore/xenstored_watch.c b/tools/xenstore/xenstored_watch.c
index 0dc5a40b99..7dedca60df 100644
--- a/tools/xenstore/xenstored_watch.c
+++ b/tools/xenstore/xenstored_watch.c
@@ -247,9 +247,9 @@ void conn_delete_all_watches(struct connection *conn)
/*
* Local variables:
+ * mode: C
* c-file-style: "linux"
* indent-tabs-mode: t
- * c-indent-level: 8
* c-basic-offset: 8
* tab-width: 8
* End:
diff --git a/tools/xenstore/xs.c b/tools/xenstore/xs.c
index 87e43bfe87..f823abe737 100644
--- a/tools/xenstore/xs.c
+++ b/tools/xenstore/xs.c
@@ -1458,9 +1458,9 @@ void unsanitise_value(char *out, unsigned *out_len_r, const char *in)
/*
* Local variables:
+ * mode: C
* c-file-style: "linux"
* indent-tabs-mode: t
- * c-indent-level: 8
* c-basic-offset: 8
* tab-width: 8
* End:
--
2.11.0
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [Xen-devel] [PATCH for-4.13] tools: Fix local variable block
2019-11-01 19:13 [Xen-devel] [PATCH for-4.13] tools: Fix local variable block Andrew Cooper
@ 2019-11-02 12:24 ` Wei Liu
2019-11-08 6:09 ` Jürgen Groß
1 sibling, 0 replies; 3+ messages in thread
From: Wei Liu @ 2019-11-02 12:24 UTC (permalink / raw)
To: Andrew Cooper; +Cc: Juergen Gross, Xen-devel, Wei Liu, Ian Jackson
On Fri, Nov 01, 2019 at 07:13:55PM +0000, Andrew Cooper wrote:
> c-indent-level isn't considered a safe variable, and "solaris" isn't a
> recognised C style. Both cause prompts when opening the files.
>
> Fix all blocks up per CODING_STYLE
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Acked-by: Wei Liu <wl@xen.org>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Xen-devel] [PATCH for-4.13] tools: Fix local variable block
2019-11-01 19:13 [Xen-devel] [PATCH for-4.13] tools: Fix local variable block Andrew Cooper
2019-11-02 12:24 ` Wei Liu
@ 2019-11-08 6:09 ` Jürgen Groß
1 sibling, 0 replies; 3+ messages in thread
From: Jürgen Groß @ 2019-11-08 6:09 UTC (permalink / raw)
To: Andrew Cooper, Xen-devel; +Cc: Ian Jackson, Wei Liu
On 01.11.19 20:13, Andrew Cooper wrote:
> c-indent-level isn't considered a safe variable, and "solaris" isn't a
> recognised C style. Both cause prompts when opening the files.
>
> Fix all blocks up per CODING_STYLE
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Release-acked-by: Juergen Gross <jgross@suse.com>
Juergen
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-11-08 6:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-01 19:13 [Xen-devel] [PATCH for-4.13] tools: Fix local variable block Andrew Cooper
2019-11-02 12:24 ` Wei Liu
2019-11-08 6:09 ` Jürgen Groß
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.