Hi Johan, > Couple of comments: > > On Tue, Dec 16, 2008, Li, Zhigang wrote: > > main_opts.inqmode = val; > > } > > + str = g_key_file_get_string(config, "General", > > Coding style: add empty line after the closing bracket. > > > + "VID/PID/VER", &err); > > Did you agree with this "VID/PID/VER" name with Marcel? I'd have used > something like "DeviceID", "DeviceIDInfo" or "DIDInfo". I think "DeviceID" sounds reasonable. Lets use that one. Regards Marcel -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html