<p dir="ltr">On May 16, 2015 11:18, "Per Bergqvist" <<a href="mailto:per@bst.lu">per@bst.lu</a>> wrote:<br>
><br>
> Lennart,<br>
><br>
> Thank you for all the comments. <br>
><br>
> I have changed everything except the 'No space between function name and opening “(“‘.<br>
> Cannot find anything about that in CODING_STYLE or evidence in other sources.</p>
<p dir="ltr">"Most every call in the entire source tree uses that style" should be good enough as evidence.</p>
<p dir="ltr">If you don't care about being consistent with existing code, at least be consistent with *your own* code, where you also use 'func()' most of the time.</p>
<p dir="ltr">> +        log_parse_environment();<br>
> +        log_open();<br>
> +<br>
> +        udev = udev_new();<br>
> +        if (udev == NULL)<br>
> +                return EXIT_SUCCESS;<br>
> +<br>
> +        while (1) {<br>
> +                int option;<br>
> +<br>
> +                option = getopt_long(argc, argv, "xh", options, NULL);</p>
<p dir="ltr">Here,</p>
<p dir="ltr">> +        fd = open(node, O_RDONLY|O_NONBLOCK|O_CLOEXEC);<br>
> +        if (fd < 0) {<br>
> +                log_error_errno(errno, "Unable to open '%s': %m", node);<br>
> +                return -errno;<br>
> +        }<br>
> +<br>
> +        if (nvme_identify(udev, fd, &nvme_id_ctrl, sizeof(struct nvme_id_ctrl)) == 0) {</p>
<p dir="ltr">here,</p>
<p dir="ltr">> +                memcpy (model, <a href="http://nvme_id_ctrl.mn">nvme_id_ctrl.mn</a>, sizeof(<a href="http://nvme_id_ctrl.mn">nvme_id_ctrl.mn</a>));</p>
<p dir="ltr">but for some reason not here.</p>
<p dir="ltr">This could be easily fixed up at commit time, though.</p>