[Q] Cleaning string whitespace trimming. Does it make any sense?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dan,

If you grep "isspace" through the code you will find lots of places
using, most of them
for removing leading and/or whitespace from a string.

Examples:

* sound/usb/card.c -> has a function of its own
http://tomoyo.sourceforge.jp/cgi-bin/lxr/ident?i=remove_trailing_spaces
* sound/pci/hda/hda_hwdep.c -> also! has a function of its own
http://tomoyo.sourceforge.jp/cgi-bin/lxr/ident?i=remove_trail_spaces
* drivers/char/ipmi/ipmi_si_intf.c -> embeds a custom algorithm ->
http://tomoyo.sourceforge.jp/cgi-bin/lxr/source/drivers/char/ipmi/ipmi_si_intf.c#L1690

As we have a special function for this (or could create in case there
isn't one already)
I wonder if it makes any sense to do some patches for this, since it
won't fix anything
and it's just to improve readability.

What do you think?

Thanks,
Ezequiel.

PD: Am I too pretty-code paranoid?
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux