On 8/2/2017 1:16 AM, Johannes Thumshirn wrote:
+       for (;;) {
+               deferfcp = list_first_entry_or_null(&queue->pending_cmd_list,
+                               struct nvmet_fc_defer_fcp_req, req_list);
+               if (!deferfcp)
+                       break;

         while ((deferfcp = list_first_entry_or_null(&queue->pending_cmd_list,
                                struct nvmet_fc_defer_fcp_req,
                                req_list)) != NULL) {
?

To me - this is harder to read. But not a big deal.

-- james



Reply via email to