I would prefer that too, but when it's written on a single line the defined check always evaluates to true. 

Hunt & pecked on my iPhone... Sorry if it's brief!

> On Nov 7, 2013, at 6:49 PM, Nathan Weizenbaum <notifications@github.com> wrote:
> 
> In lib/sass/util.rb:
> 
> > @@ -512,28 +524,44 @@ def av_template_class(name)
> >      #
> >      # @return [Boolean]
> >      def windows?
> > -      RbConfig::CONFIG['host_os'] =~ /mswin|windows|mingw/i
> > +      if defined?(@windows)
> That's fine, although I prefer the following structure to cut down on the number of lines:
> 
> return @windows if defined?(@windows)
> @windows = (RbConfig::CONFIG['host_os'] =~ /mswin|windows|mingw/i)
> —
> Reply to this email directly or view it on GitHub.