On Wed, Dec 2, 2015 at 3:53 PM, Ignat Korchagin <ig...@cloudflare.com> wrote: > Let's add utf16 later then (if needed). I also noticed a typo in one > error string in my above patch (it was checking that command receives > one argument, but prints that two arguments expected), so here is the > updated one. >
It is much more difficult to remove anything than to add it later. Unless we have use case right now, I'd skip it. > So, finally, should I remove the "raw" parser (as per Vladimir's > request). Personally, I would keep it. It might be useful to modify > the hexdump command, that it can dump the contents of a variable. Then > those pieces will fit together: storing raw value in the variable and > examining it with hexdump cmd. > hexdump does not support showing variable if that is what you mean. By the same logic, unless we have current use case we probably should not add it initially. _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org https://lists.gnu.org/mailman/listinfo/grub-devel