From: SF Markus Elfring <elfring@users.sourceforge.net>
To: "Austin S. Hemmelgarn" <ahferroin7@gmail.com>
Cc: Dan Carpenter <dan.carpenter@oracle.com>,
Richard Weinberger <richard@nod.at>,
"linux-raid@vger.kernel.org" <linux-raid@vger.kernel.org>,
Christoph Hellwig <hch@lst.de>, Guoqing Jiang <gqjiang@suse.com>,
Jens Axboe <axboe@fb.com>, Mike Christie <mchristi@redhat.com>,
Neil Brown <neilb@suse.com>, Shaohua Li <shli@kernel.org>,
Tomasz Majchrzak <tomasz.majchrzak@intel.com>,
LKML <linux-kernel@vger.kernel.org>,
"kernel-janitors@vger.kernel.org"
<kernel-janitors@vger.kernel.org>,
Julia Lawall <julia.lawall@lip6.fr>
Subject: Re: md/raid1: Improve another size determination in setup_conf()
Date: Fri, 7 Oct 2016 17:27:05 +0200 [thread overview]
Message-ID: <a3d1b5a7-4b4e-a37d-2952-bb2e01828c39@users.sourceforge.net> (raw)
In-Reply-To: <6e2c26bc-d765-6225-af72-157832ab8785@gmail.com>
>> Why do various software developers bother about coding style specifications
>> at all then?
> Coding style is important,
Thanks that you "dare" to express also such an opinion.
> but patches that just fix coding style are a bad thing
When you find such a change opportunity so "bad", are there any circumstances
left over where you would dare to touch the corresponding source code line.
> because they break things like `git blame`
I follow your concern to some degree.
But can this argument evolve against a lot of changes generally?
> and run the risk of introducing new bugs
Did this really "happen" because of an update suggestion for this software module?
> without any net benefit to end users.
Can the proposed adjustment help to make a function like "setup_conf"
a bit more robust (together with related update steps) so that an improved
coding style compliance will hopefully influence the error probability
in positive ways?
> This goes double for code you don't actually work on regularly
> or don't completely understand.
How does such a kind of general feedback fit to the shown change
possibilities in this patch series?
Do you reject this update step?
Regards,
Markus
next prev parent reply other threads:[~2016-10-07 15:27 UTC|newest]
Thread overview: 221+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <566ABCD9.1060404@users.sourceforge.net>
2016-09-27 16:44 ` [PATCH 00/16] md/bitmap: Fine-tuning for several function implementations SF Markus Elfring
2016-09-27 16:45 ` [PATCH 01/16] md/bitmap: Use kmalloc_array() in bitmap_storage_alloc() SF Markus Elfring
2016-09-28 19:51 ` Jes Sorensen
2016-09-27 16:48 ` [PATCH 02/16] md/bitmap: Move an assignment for the variable "offset" " SF Markus Elfring
2016-09-27 16:49 ` [PATCH 03/16] md/bitmap: Delete an unnecessary variable initialisation " SF Markus Elfring
2016-09-27 16:50 ` [PATCH 04/16] md/bitmap: Improve another size determination " SF Markus Elfring
2016-09-27 16:51 ` [PATCH 05/16] md/bitmap: Return directly after a failed bitmap_storage_alloc() in bitmap_resize() SF Markus Elfring
2016-09-27 16:53 ` [PATCH 06/16] md/bitmap: Return directly after a failed kzalloc() " SF Markus Elfring
2016-09-27 16:54 ` [PATCH 07/16] md/bitmap: Replace a kzalloc() call by kcalloc() " SF Markus Elfring
2016-09-27 16:55 ` [PATCH 08/16] md/bitmap: Rename a jump label in location_store() SF Markus Elfring
2016-09-28 19:55 ` Jes Sorensen
2016-09-29 3:16 ` Guoqing Jiang
2016-09-27 16:56 ` [PATCH 09/16] md/bitmap: Rename a jump label in bitmap_copy_from_slot() SF Markus Elfring
2016-09-27 16:57 ` [PATCH 10/16] md/bitmap: Rename a jump label in bitmap_create() SF Markus Elfring
2016-09-27 16:58 ` [PATCH 11/16] md/bitmap: Rename a jump label in bitmap_init_from_disk() SF Markus Elfring
2016-09-27 16:59 ` [PATCH 12/16] md/bitmap: One check less in read_page() at the end SF Markus Elfring
2016-09-28 7:33 ` Dan Carpenter
2016-09-27 17:00 ` [PATCH 13/16] md/bitmap: Adjust checks for null pointers in 11 functions SF Markus Elfring
2016-09-27 17:02 ` [PATCH 14/16] md/bitmap: Delete unnecessary braces in bitmap_resize() SF Markus Elfring
2016-09-27 17:03 ` [PATCH 15/16] md/bitmap: Add spaces around three comparison operators SF Markus Elfring
2016-09-27 17:04 ` [PATCH 16/16] md/bitmap: Delete an unwanted space in read_sb_page() SF Markus Elfring
2016-09-28 15:34 ` [PATCH 00/10] md/dm-crypt: Fine-tuning for five function implementations SF Markus Elfring
2016-09-28 15:36 ` [PATCH 01/10] md/dm-crypt: Use kcalloc() in crypt_alloc_tfms() SF Markus Elfring
2016-09-28 15:38 ` [PATCH 00/10] md/dm-crypt: Fine-tuning for five function implementations Mike Snitzer
2016-09-28 15:38 ` [PATCH 02/10] md/dm-crypt: Reduce the scope for a variable in crypt_alloc_tfms() SF Markus Elfring
2016-09-28 15:40 ` [PATCH 03/10] md/dm-crypt: Rename a jump label in crypt_message() SF Markus Elfring
2016-09-29 12:55 ` [dm-devel] " Theodore Ts'o
2016-09-29 15:43 ` SF Markus Elfring
2016-09-30 10:06 ` Dan Carpenter
2016-09-30 11:32 ` md/dm-crypt: Rename a jump label in crypt_message() ? SF Markus Elfring
2016-09-30 11:39 ` Bjørn Mork
2016-09-30 11:53 ` SF Markus Elfring
2016-09-30 12:06 ` Bjørn Mork
2016-09-30 12:54 ` SF Markus Elfring
2016-09-30 12:02 ` Dan Carpenter
2016-09-30 12:19 ` SF Markus Elfring
2016-09-28 15:41 ` [PATCH 04/10] md/dm-crypt: Delete an unnecessary variable initialisation in crypt_message() SF Markus Elfring
2016-09-28 15:42 ` [PATCH 05/10] md/dm-crypt: Rename a jump label in crypt_set_key() SF Markus Elfring
2016-09-29 12:56 ` [dm-devel] " Theodore Ts'o
2016-09-28 15:43 ` [PATCH 06/10] md/dm-crypt: Delete an unnecessary variable initialisation " SF Markus Elfring
2016-09-28 15:44 ` [PATCH 07/10] md/dm-crypt: Rename a jump label in crypt_iv_tcw_whitening() SF Markus Elfring
2016-09-28 15:45 ` [PATCH 08/10] md/dm-crypt: Return directly after a failed crypto_alloc_ahash() in crypt_iv_essiv_ctr() SF Markus Elfring
2016-09-28 15:46 ` [PATCH 09/10] md/dm-crypt: Two checks and one function call less in crypt_iv_essiv_ctr() after error detection SF Markus Elfring
2016-09-28 15:47 ` [PATCH 10/10] md/dm-crypt: Delete unnecessary variable initialisations in crypt_iv_essiv_ctr() SF Markus Elfring
2016-09-29 9:02 ` [PATCH 00/10] dm snapshot: Fine-tuning for several function implementations SF Markus Elfring
2016-09-29 9:07 ` [PATCH 01/10] dm snapshot: Use kmalloc_array() in init_origin_hash() SF Markus Elfring
2016-09-29 9:54 ` Paul Bolle
2016-09-29 10:02 ` Joe Perches
2016-09-29 11:12 ` Paul Bolle
2016-09-29 11:45 ` Paul Bolle
2016-09-29 15:01 ` Joe Perches
2016-09-29 19:43 ` Paul Bolle
2016-09-29 20:24 ` Joe Perches
2016-09-29 20:39 ` Paul Bolle
2016-09-29 20:56 ` Joe Perches
2016-09-29 21:14 ` Paul Bolle
2016-09-29 21:21 ` Joe Perches
2016-09-29 21:42 ` Paul Bolle
2016-09-30 7:14 ` dm snapshot: Use kmalloc_array() in init_origin_hash() ? SF Markus Elfring
2016-09-29 11:45 ` dm snapshot: Use kmalloc_array() in init_origin_hash() SF Markus Elfring
2016-09-29 12:59 ` Theodore Ts'o
2016-09-29 9:10 ` [PATCH 02/10] dm snapshot: Delete two error messages for a failed memory allocation SF Markus Elfring
2016-09-29 9:11 ` [PATCH 03/10] dm snapshot: Delete an unnecessary variable initialisation in snapshot_map() SF Markus Elfring
2016-09-29 9:12 ` [PATCH 04/10] dm snapshot: Rename a jump label in pending_complete() SF Markus Elfring
2016-09-29 9:13 ` [PATCH 05/10] dm snapshot: Delete unnecessary variable initialisations " SF Markus Elfring
2016-09-29 9:14 ` [PATCH 06/10] dm snapshot: Delete an unnecessary variable initialisation in snapshot_ctr() SF Markus Elfring
2016-09-29 9:15 ` [PATCH 07/10] dm snapshot: Delete an unnecessary variable initialisation in merge_callback() SF Markus Elfring
2016-09-29 9:16 ` [PATCH 08/10] dm snapshot: Delete an unnecessary variable initialisation in remove_single_exception_chunk() SF Markus Elfring
2016-09-29 9:17 ` [PATCH 09/10] dm snapshot: Combine substrings for seven error messages SF Markus Elfring
2016-09-29 9:18 ` [PATCH 10/10] dm snapshot: Delete five unwanted spaces behind "list_for_each_entry" SF Markus Elfring
2016-10-01 14:44 ` [PATCH 00/15] md-cluster: Fine-tuning for ten function implementations SF Markus Elfring
2016-10-01 14:46 ` [PATCH 01/15] md-cluster: Use kcalloc() in lock_all_bitmaps() SF Markus Elfring
2016-10-01 14:47 ` [PATCH 02/15] md-cluster: Improve another size determination in resync_info_update() SF Markus Elfring
2016-10-01 14:48 ` [PATCH 03/15] md-cluster: Improve another size determination in join() SF Markus Elfring
2016-10-01 14:49 ` [PATCH 04/15] md-cluster: Improve another size determination in __sendmsg() SF Markus Elfring
2016-10-01 14:50 ` [PATCH 05/15] md-cluster: Improve another size determination in recv_daemon() SF Markus Elfring
2016-10-01 14:51 ` [PATCH 06/15] md-cluster: Rename a jump label " SF Markus Elfring
2016-10-01 14:52 ` [PATCH 07/15] md-cluster: Improve another size determination in process_suspend_info() SF Markus Elfring
2016-10-01 14:53 ` [PATCH 08/15] md-cluster: Improve determination of sizes in read_resync_info() SF Markus Elfring
2016-10-01 14:54 ` [PATCH 09/15] md-cluster: Improve another size determination in lockres_init() SF Markus Elfring
2016-10-01 14:55 ` [PATCH 10/15] md-cluster: Delete an unnecessary variable initialisation " SF Markus Elfring
2016-10-01 14:56 ` [PATCH 11/15] md-cluster: Delete four error messages for a failed memory allocation SF Markus Elfring
2016-10-01 14:57 ` [PATCH 12/15] md-cluster: Rename a jump label in area_resyncing() SF Markus Elfring
2016-10-01 14:58 ` [PATCH 13/15] md-cluster: Less function calls in join() after error detection SF Markus Elfring
2016-10-06 15:16 ` [PATCH v2 " SF Markus Elfring
2016-10-01 14:59 ` [PATCH 14/15] md-cluster: Less function calls in lockres_init() " SF Markus Elfring
2016-10-01 15:00 ` [PATCH 15/15] md-cluster: Delete unnecessary braces in unlock_all_bitmaps() SF Markus Elfring
2016-10-07 7:46 ` Dan Carpenter
2016-10-07 8:37 ` SF Markus Elfring
2016-10-07 13:21 ` walter harms
2016-10-02 11:54 ` [PATCH 00/13] md/multipath: Fine-tuning for several function implementations SF Markus Elfring
2016-10-02 11:56 ` [PATCH 1/13] md/multipath: Use kcalloc() in multipath_run() SF Markus Elfring
2016-10-02 11:57 ` [PATCH 2/13] md/multipath: Improve another size determination " SF Markus Elfring
2016-10-02 11:59 ` [PATCH 3/13] md/multipath: Delete four error messages for a failed memory allocation SF Markus Elfring
2016-10-02 12:00 ` [PATCH 4/13] md/multipath: Reduce indentation for four lines in multipath_run() SF Markus Elfring
2016-10-02 12:01 ` [PATCH 5/13] md/multipath: Less function calls in multipath_run() after error detection SF Markus Elfring
2016-10-02 12:02 ` [PATCH 6/13] md/multipath: Delete 13 unwanted spaces behind function names SF Markus Elfring
2016-10-02 12:03 ` [PATCH 7/13] md/multipath: Delete two unwanted spaces behind asterisks SF Markus Elfring
2016-10-02 12:04 ` [PATCH 8/13] md/multipath: Replace a seq_printf() call by seq_puts() in multipath_status() SF Markus Elfring
2018-01-13 8:55 ` [PATCH v2] md-multipath: Use seq_putc() " SF Markus Elfring
2018-02-17 21:01 ` Shaohua Li
2016-10-02 12:05 ` [PATCH 9/13] md/multipath: Adjust two function calls together with a variable assignment SF Markus Elfring
2016-10-02 12:06 ` [PATCH 10/13] md/multipath: Add some spaces for better code readability SF Markus Elfring
2016-10-02 12:08 ` [PATCH 11/13] md/multipath: Move a brace for a designated initialiser SF Markus Elfring
2016-10-02 12:09 ` [PATCH 12/13] md/multipath: Delete an unnecessary return statement in multipath_make_request() SF Markus Elfring
2016-10-02 12:10 ` [PATCH 13/13] md/multipath: Replace printk() calls by the usage of higher level interfaces SF Markus Elfring
2016-10-06 8:46 ` MD-RAID: Fine-tuning for several function implementations SF Markus Elfring
2016-10-06 8:49 ` Christoph Hellwig
2016-10-06 10:07 ` SF Markus Elfring
2016-10-06 8:51 ` [PATCH 01/54] md/raid0: Use kcalloc() in create_strip_zones() SF Markus Elfring
2016-10-06 8:53 ` [PATCH 02/54] md/raid0: Less function calls in create_strip_zones() after error detection SF Markus Elfring
2016-10-06 8:54 ` [PATCH 03/54] md/raid0: Move a variable assignment in create_strip_zones() SF Markus Elfring
2016-10-06 8:56 ` [PATCH 04/54] md/raid0: Replace printk() calls by the usage of higher level interfaces SF Markus Elfring
2016-10-06 8:57 ` [PATCH 05/54] md/raid0: Move another variable assignment in create_strip_zones() SF Markus Elfring
2016-10-06 8:59 ` [PATCH 06/54] md/raid0: Delete four unwanted spaces behind function names SF Markus Elfring
2016-10-06 9:00 ` [PATCH 07/54] md/raid0: Move two misplaced braces SF Markus Elfring
2016-10-06 9:01 ` [PATCH 08/54] md/raid0: Delete an unnecessary return statement in raid0_status() SF Markus Elfring
2016-10-06 9:03 ` [PATCH 09/54] md/raid0: Add some spaces for better code readability SF Markus Elfring
2016-10-06 9:05 ` [PATCH 10/54] md/raid1: Use kcalloc() in alloc_behind_pages() SF Markus Elfring
2016-10-06 9:06 ` [PATCH 11/54] md/raid1: Use kcalloc() in raid1_reshape() SF Markus Elfring
2016-10-06 9:07 ` [PATCH 12/54] md/raid1: Use kcalloc() in setup_conf() SF Markus Elfring
2016-10-06 9:09 ` [PATCH 13/54] md/raid1: Return directly after a failed kzalloc() " SF Markus Elfring
2016-10-06 9:10 ` [PATCH 14/54] md/raid1: Move assignments for the variable "err" " SF Markus Elfring
2016-10-06 9:11 ` [PATCH 15/54] md/raid1: Less function calls in setup_conf() after error detection SF Markus Elfring
2016-10-06 9:12 ` [PATCH 16/54] md/raid1: Delete an error message for a failed memory allocation SF Markus Elfring
2016-10-06 9:14 ` [PATCH 17/54] md/raid1: Move a brace for a designated initialiser SF Markus Elfring
2016-10-06 9:15 ` [PATCH 18/54] md/raid1: Adjust 12 checks for null pointers SF Markus Elfring
2016-10-06 9:16 ` [PATCH 19/54] md/raid1: Replace printk() calls by the usage of higher level interfaces SF Markus Elfring
2016-10-06 9:18 ` [PATCH 20/54] md/raid1: Add some spaces for better code readability SF Markus Elfring
2016-10-06 9:19 ` [PATCH 21/54] md/raid1: Delete three unwanted spaces behind asterisks SF Markus Elfring
2016-10-06 9:20 ` [PATCH 22/54] md/raid1: Delete three unwanted spaces before increment operators SF Markus Elfring
2016-10-06 9:21 ` [PATCH 23/54] md/raid1: Replace a seq_printf() call by seq_puts() in raid1_status() SF Markus Elfring
2016-10-06 9:22 ` [PATCH 24/54] md/raid1: Improve another size determination in setup_conf() SF Markus Elfring
2016-10-06 9:29 ` Richard Weinberger
2016-10-07 7:53 ` Dan Carpenter
2016-10-07 8:15 ` Richard Weinberger
2016-10-07 8:53 ` SF Markus Elfring
2016-10-07 9:06 ` Richard Weinberger
2016-10-07 10:50 ` SF Markus Elfring
2016-10-07 11:52 ` Austin S. Hemmelgarn
2016-10-07 15:27 ` SF Markus Elfring [this message]
2016-10-07 15:35 ` Jiri Kosina
2016-10-07 16:38 ` SF Markus Elfring
2016-10-10 13:11 ` Jes Sorensen
2016-10-10 13:06 ` Jes Sorensen
2016-10-10 13:20 ` SF Markus Elfring
2016-10-10 14:01 ` Jes Sorensen
2016-10-10 14:20 ` SF Markus Elfring
2016-10-10 13:09 ` [PATCH 24/54] " Jes Sorensen
2016-10-12 8:28 ` Dan Carpenter
2016-10-12 12:18 ` Jes Sorensen
2016-10-10 11:12 ` Dan Carpenter
2016-10-10 12:28 ` SF Markus Elfring
2016-10-10 13:23 ` Adam Goryachev
2016-10-10 13:57 ` [PATCH 24/54] " Bjørn Mork
2016-10-11 6:20 ` Dan Carpenter
2016-10-06 9:23 ` [PATCH 25/54] md/raid5: Use kcalloc() in three functions SF Markus Elfring
2016-10-06 9:25 ` [PATCH 26/54] md/raid5: Improve another size determination in setup_conf() SF Markus Elfring
2016-10-06 9:26 ` [PATCH 27/54] md/raid5: Return directly after a failed kzalloc() " SF Markus Elfring
2016-10-06 9:28 ` [PATCH 28/54] md/raid5: Rename a jump label " SF Markus Elfring
2016-10-06 9:29 ` [PATCH 29/54] md/raid5: Return directly after a failed kcalloc() in alloc_thread_groups() SF Markus Elfring
2016-10-06 9:30 ` [PATCH 30/54] md/raid5: Delete two error messages for a failed memory allocation SF Markus Elfring
2016-10-07 5:51 ` Hannes Reinecke
2016-10-06 9:31 ` [PATCH 31/54] md/raid5: Adjust two function calls together with a variable assignment SF Markus Elfring
2016-10-06 9:32 ` [PATCH 32/54] md/raid5: Move a brace for three designated initialisers SF Markus Elfring
2016-10-06 9:33 ` [PATCH 33/54] md/raid5: Replace printk() calls by the usage of higher level interfaces SF Markus Elfring
2016-10-06 9:34 ` [PATCH 34/54] md/raid5: Delete indentation for two jump labels SF Markus Elfring
2016-10-06 9:35 ` [PATCH 35/54] md/raid5: Adjust 13 checks for null pointers SF Markus Elfring
2016-10-06 9:36 ` [PATCH 36/54] md/raid5: Delete four unwanted spaces behind function names SF Markus Elfring
2016-10-06 9:37 ` [PATCH 37/54] md/raid5: Replace a seq_printf() call by seq_puts() in raid5_status() SF Markus Elfring
2016-10-06 16:34 ` Joe Perches
2016-10-06 17:09 ` SF Markus Elfring
2016-10-06 17:36 ` Joe Perches
2016-10-06 17:49 ` SF Markus Elfring
2016-10-06 17:51 ` Joe Perches
2016-10-06 18:22 ` SF Markus Elfring
2016-10-06 18:33 ` [PATCH 37/54] " Bernd Petrovitsch
2016-10-16 8:20 ` [PATCH] MD-RAID: Use seq_putc() in three status functions SF Markus Elfring
2016-10-16 16:58 ` Hannes Reinecke
2016-10-16 17:10 ` MD-RAID: Use seq_putc() in three status functions? SF Markus Elfring
2016-10-16 17:18 ` SF Markus Elfring
2016-10-17 5:58 ` Hannes Reinecke
2016-10-17 7:39 ` SF Markus Elfring
2016-10-17 8:09 ` Hannes Reinecke
2016-10-17 9:00 ` SF Markus Elfring
2016-10-17 9:50 ` Hannes Reinecke
2016-10-17 11:10 ` SF Markus Elfring
2016-10-17 11:32 ` Bernd Petrovitsch
2016-10-17 11:43 ` SF Markus Elfring
2016-10-17 14:01 ` Hannes Reinecke
2016-10-17 14:30 ` SF Markus Elfring
2016-10-17 15:33 ` Hannes Reinecke
2016-10-17 16:08 ` SF Markus Elfring
2016-10-17 17:18 ` Hannes Reinecke
2016-10-18 18:20 ` SF Markus Elfring
2016-10-20 12:24 ` SF Markus Elfring
2016-10-28 20:04 ` SF Markus Elfring
2016-10-17 14:00 ` Hannes Reinecke
2016-10-06 9:38 ` [PATCH 38/54] md/raid5: Move four asterisks SF Markus Elfring
2016-10-06 9:39 ` [PATCH 39/54] md/raid5: Add some spaces for better code readability SF Markus Elfring
2016-10-06 9:40 ` [PATCH 40/54] md/raid10: Use kcalloc() in two functions SF Markus Elfring
2016-10-06 9:41 ` [PATCH 41/54] md/raid10: Improve another size determination in setup_conf() SF Markus Elfring
2016-10-06 9:42 ` [PATCH 42/54] md/raid10: Delete an error message for a failed memory allocation SF Markus Elfring
2016-10-06 9:43 ` [PATCH 43/54] md/raid10: Return directly after detection of unsupported settings in setup_conf() SF Markus Elfring
2016-10-06 9:44 ` [PATCH 44/54] md/raid10: Return directly after a failed kzalloc() " SF Markus Elfring
2016-10-06 9:45 ` [PATCH 45/54] md/raid10: Move assignments for the variable "err" " SF Markus Elfring
2016-10-06 9:46 ` [PATCH 46/54] md/raid10: Less function calls in setup_conf() after error detection SF Markus Elfring
2016-10-06 9:47 ` [PATCH 47/54] md/raid10: Improve another size determination in raid10_start_reshape() SF Markus Elfring
2016-10-06 9:48 ` [PATCH 48/54] md/raid10: Move a brace for a designated initialiser SF Markus Elfring
2016-10-06 9:49 ` [PATCH 49/54] md/raid10: Replace printk() calls by the usage of higher level interfaces SF Markus Elfring
2016-10-06 16:33 ` Joe Perches
2016-10-06 17:20 ` SF Markus Elfring
2016-10-06 17:39 ` Joe Perches
2016-10-06 18:04 ` SF Markus Elfring
2016-10-06 18:18 ` Joe Perches
2016-10-06 18:32 ` SF Markus Elfring
2016-10-06 9:50 ` [PATCH 50/54] md/raid10: Delete indentation for one jump label SF Markus Elfring
2016-10-06 9:51 ` [PATCH 51/54] md/raid10: Adjust 22 checks for null pointers SF Markus Elfring
2016-10-06 9:52 ` [PATCH 52/54] md/raid10: Replace a seq_printf() call by seq_puts() in raid10_status() SF Markus Elfring
2016-10-06 9:53 ` [PATCH 53/54] md/raid10: Delete two unwanted spaces behind asterisks SF Markus Elfring
2016-10-06 9:54 ` [PATCH 54/54] md/raid10: Add some spaces for better code readability SF Markus Elfring
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=a3d1b5a7-4b4e-a37d-2952-bb2e01828c39@users.sourceforge.net \
--to=elfring@users.sourceforge.net \
--cc=ahferroin7@gmail.com \
--cc=axboe@fb.com \
--cc=dan.carpenter@oracle.com \
--cc=gqjiang@suse.com \
--cc=hch@lst.de \
--cc=julia.lawall@lip6.fr \
--cc=kernel-janitors@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-raid@vger.kernel.org \
--cc=mchristi@redhat.com \
--cc=neilb@suse.com \
--cc=richard@nod.at \
--cc=shli@kernel.org \
--cc=tomasz.majchrzak@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).