|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [PATCH] xen/arch/arm: Correcting comment typos across xen/arch/arm
On 09-Nov-26 12:49, Andrew Precious wrote: > Thanks for response > >> Scanning Xen using codespell for typo matches I have found: >> 1. Xen native project sources: 3303 typos > Is this hypervisor only or entire Xen repo? > > Entire Xen project repo - 3303 typos > Hypervisor code only has - 351 typos That's good and hypervisor code is what I would start with... > > However the scanner can be tweaked to ignore/include areas > such as code pulled from linux source trees and 3rd party stubdoms. ... having this in mind i.e. we don't want to modify files for which we first require a submission to the relevant project and then to backport the patch to Xen. ~Michal > > On Mon, Nov 9, 2026 at 10:28 AM Orzel, Michal <michal.orzel@xxxxxxx > <mailto:michal.orzel@xxxxxxx>> wrote: > > > > On 05-Nov-26 12:26, Andrew Mbugua wrote: > > No functional change. > > > The below text should be put after "---", given that this is not the > information > that should end up in a committed patch. > > > I have been making typo corrections across a variety of files > > > > Scanning Xen using codespell for typo matches I have found: > > 1. Xen native project sources: 3303 typos > Is this hypervisor only or entire Xen repo? > > > 2. Qemu Tree - 4908 typos > > > > Reasons for corrections: > > 1. Keeping the codebase clean,concise for future documentation > generation > > 2. Just because typos seem minor doesnt mean that corrections should be > ignored, > > while correcting these errors may not be pedantic I think its essential > in > > keeping things claan and more readable. > I don't mind patches like this one, so thanks for the contribution: > Reviewed-by: Michal Orzel <michal.orzel@xxxxxxx > <mailto:michal.orzel@xxxxxxx>> > > I'll do the commit msg adjustment when committing. > > ~Michal >
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |