rm
seeemef@mac.com
-
-
usr/src/lib/fm/topo/modules/common/fac_prov_ahci/fac_prov_ahci.c (Diff revision 1) I'm afraid I don't follow why
\nis used with all but twotopo_mod_dprintf()calls in this function but I think not used elsewhere with such calls in this patch.
11576 Want support for ahci LED entries in topo
Review Request #2235 — Created Aug. 18, 2019 and submitted
| Information | |
|---|---|
| rm | |
| illumos-gate | |
| master | |
| 11576 | |
| aeb2428... | |
| Reviewers | |
| general | |
11576 Want support for ahci LED entries in topo
Reviewed by: Rob Johnston <rob.johnston@joyent.com>
Reviewed by: Robert Mustacchi <rm@joyent.com>
-
-
usr/src/lib/fm/topo/modules/common/fac_prov_ahci/fac_prov_ahci.c (Diff revision 1) So, there's really no difference between using them or not in the code. This isn't like mdb where not having a \n is semantically different. In fact, topo_mod_dprintf will insert a final \n if you don't for you. In this case, I suspect it's just a matter of when they were written in the development cycle and/or their origin.