http://bugzilla.moblin.org/show_bug.cgi?id=3604 --- Comment #14 from yongsheng zhu <yongsheng.zhu(a)intel.com&gt; 2009-09-04 00:25:54 --- Ok, done. The reason is that if the callee doesn't save the password, then the caller(PasswordConfigProperty) won't reset its password with the value '-' because I've moved this part of code to PasswordConfigProperty . For example, if no keyring support and user doesn't need keyring, then savePassword in CmdlineSyncClient::savePassword will return false. see the new code. Yes, it's the main concern of this issue. Currently I past the ConfigNode to these 2 functions because the caller(PasswordConfigProperty) has this kind of information for the configNode is pasted to it by its own caller. I think the most confusing thing to me is I don't know what kind of information will be needed except config.ini in the future. It's hard to figure it out. we know that 'checkPassword' is invoked by 'sync()' for retrieving password, however, who will invoke CmdlineSyncClient::savePassword(ConfigUserInterface&)? The answer has 2 choices: invoke this before flushing to the config files or users invoke it directly. -- Configure bugmail: http://bugzilla.moblin.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching someone on the CC list of the bug.