I tried proposed changes. in ps_match_dialog_state_from_body there is a loop: while(node != NULL) { .... why do we need there one more while loop? What actually happens: ..... if(strcmp(tmp_state, vstate)!=0) { // here we come only once, when matching first dialog ...
all later dialogs are iterating in the second while loop.
as for me this part of code ``` + do { + if(node->next != NULL && node->next->name != NULL + && xmlStrcmp(node->name, node->next->name) == 0) { + node = node->next; + break; + } + node = node->next; + } while(node != NULL); ```
should look like: ``` node = node->next; ``` maybe that checks for NULL are also necessary, but not while do-while.
General question - is there a way to propose changes in your custom branch?
thanks for your help!