On 2013-10-21 7:14 AM, Aron Ahmadia wrote:
>
> In bin/make_calendar.py:
>
> > @@ -84,7 +84,7 @@ def bootcamp(self, site, timestamp, info):
> >              'URL:{0}'.format(url),
> >              'LOCATION:{0}'.format(self.escape(info['venue'])),
> >          ]
> > -        if 'latlng' in info:
> > +        if info.get('latlng', None):
>
> This seems morally equivalent to the removed line, so I'm clearly 
> missing something. Is there a reason to prefer the |get| query with a 
> default value of |None| here to the standard |in| syntax?
>
The replacement line handles two cases:
1. key missing
2. key present, but value set to None because latlng couldn't be parsed 
as two floats.