On 10/13/17 2:40 AM, Jan Beulich wrote: >>>> On 12.10.17 at 22:56, <andrew.coop...@citrix.com> wrote: >> On 12/10/2017 21:50, Doug Goldstein wrote: >>> From: David Esler <drumandst...@gmail.com> >>> >>> The send_chr function sends an entire C-string and not one character and >>> doesn't necessarily just send it over the serial UART anymore so rename >>> it to print_err so that its closer in name to what it does. >>> >>> Reviewed-by: Doug Goldstein <car...@cardoe.com> >>> Signed-off-by: David Esler <drumandst...@gmail.com> >> >> Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> >> >> This should also be included in 4.10 IMO. > > I'm not convinced - this is merely a cleanup style patch, the more > that the label really serves dual purpose and hence the original > name isn't all that wrong anyway. > > Jan >
I purposefully broke it out so that it could be discussed. Prior to the commit I referenced in patch 1 the function sent out a character. Now it requires the supplied data to be a C-string (NULL terminated) so I would hope that you could agree that at else the "chr" part of "send_chr" is incorrect and should likely be "str" or "err". But again this patch isn't really important its more to try to make the label match the behavior of the code below the label and if its unwanted then it can be dropped. -- Doug Goldstein _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
