Kay Sievers wrote: > On Wed, Mar 18, 2009 at 18:50, Stefan Richter <stefanr@xxxxxxxxxxxxxxxxx> wrote: >> Chandra Seetharaman wrote: >>> On Wed, 2009-03-18 at 16:43 +0100, Stefan Richter wrote: >>>> AFAICS you could have kept the check for .vendor != NULL instead of >>>> .vendor[0] != '\0' and write the terminator thusly: >>>> >>>> {}, >>>> >>>> Saves the space for a one byte long string in the object files. :-) >>> Tried it after your response. Got a panic :(. > > It's a static char array not a pointer to a string. Right, I missed that part of the switch from struct scsi_dh_devlist to struct scsi_dh_device_id. #-| -- Stefan Richter -=====-=-=== -=-= -==-= http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html