[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #3927 [Vidalia]: Vidalia torrc editor 'Save settings' checkbox is useless
#3927: Vidalia torrc editor 'Save settings' checkbox is useless
---------------------+------------------------------------------------------
Reporter: rransom | Owner: chiiph
Type: defect | Status: new
Priority: minor | Milestone:
Component: Vidalia | Version: Vidalia: 0.2.20
Keywords: easy | Parent:
Points: | Actualpoints:
---------------------+------------------------------------------------------
Changes (by onizuka):
* cc: onizuka.xxxx@â (added)
* version: => Vidalia: 0.2.20
Comment:
I agree with rransom.
A glance at the master branch's code (0.2.21) revealed that the
application has not been designed to deal with a 'temporary setting' :
SAVECONF is called inÂServiceSettings.cpp,ÂConfigDialog.cpp and
TorrcDialog.cpp.ÂConfigDialogÂshould be the only one to handle the
SAVECONF command otherwise the service editing & the torrc editing
override each other - because it iterates through a stack to save the
whole set of pages -, thus the undesired behaviours which occur when a
SAVECONF is issued after a SETCONF.
Since the feature is not really essential and seems to haveÂdisappeared
from branch alpha (0.3.3-alpha) I also suggest to remove the checkbox from
theÂTorrcDialog.ui in the current branch master.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/3927#comment:3>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs