On Apr 13, 2014, at 8:18 PM, Adam Coddington <notifications@github.com> wrote:

> In wrapper.py:
> 
> > +				  dest="lost_password",
> > +				  default=False,
> > +				  help="Forcibly active this passcode on the idevice",)
> > +
> > +parser.add_option("--lostmessage",
> > +				  action="store", 
> > +				  dest="lost_message",
> > +				  default="",
> > +				  help="Forcibly display this message when activating lost mode.",)
> > +
> > +
> > +
> > +(options, args) = parser.parse_args()
> > +print options
> > +
> > +if options.username == "" or options.password == "":
> Since in python an empty string evaluates as falsy, let's instead
> 
> if not options.username or not options.password:
> 
> 
I normally would do that, but I was having some issues earlier with troubleshooting some logic.  So I ended up breaking it out fully to ensure that some short circuiting wasn’t happening.

		- Benjamin