On Mon, Sep 22, 2014 at 08:53:25PM +0400, Andrei Borzenkov wrote:
> This does not sound right ...
> 
> static grub_err_t
> recv_hook (grub_net_udp_socket_t sock __attribute__ ((unused)),
>            struct grub_net_buff *nb,
>            void *data_)
> {
> ...
>   head = (struct dns_header *) nb->data;
> ...
>   for (i = 0; i < grub_cpu_to_be16 (head->qdcount); i++)
>     {
> 
> We sure want to convert *to* cpu, not *from* cpu here? Same in all
> other places in this function.

The operations are precisely synonymous in all cases, so you should just
use whichever spelling is clearest.

-- 
Colin Watson                                       [cjwat...@ubuntu.com]

_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to