Hi Patrik,

again nitpicking :)

On 04/02/2013 12:34 PM, Patrik Flykt wrote:
+void __connmanctl_save_rl(void)
+{
+       if (interactive == false)
+               return;
+
+       save_input = !RL_ISSTATE(RL_STATE_DONE);
+
+       if (save_input) {
+               saved_point = rl_point;
+               saved_line = rl_copy_text(0, rl_end);
+               rl_save_prompt();
+               rl_replace_line("", 0);
+               rl_redisplay();
+       }
+}

I understand we are going to adopt also oFono/BlueZ style for bools etc. Should the first if read the more like

        if (!interactive)
                return;

?

cheers,
daniel

_______________________________________________
connman mailing list
[email protected]
http://lists.connman.net/listinfo/connman

Reply via email to