Hi, While reviewing the code to add new Mesh States to mesh/mesh-main.conf, I noticed the following issues: The parse_settings() function in mesh/mesh.c, the existing configs parsing function, has many duplicate lines and is not well separated. On the other hand, the parse-related functions in src/main.c are well separated in the same structure. I think this code could be reused to improve the mesh conf parsing function. Rather than duplicate implementations of similar code, I would like to ask for feedback on whether it would be better to isolate the reusable lines in the src/shared/ directory and make them available in mesh/mesh.c as well. It would not be necessary to move all the prefix parse_ functions in src/main.c, just the prefix prase_config_ that parses the data type. I would like to ask if this would break the bluez semantics. Regards, Junho Lee
