That's fine, although I prefer the following structure to cut down on the number of lines:

```ruby
return @windows if defined?(@windows)
@windows = (RbConfig::CONFIG['host_os'] =~ /mswin|windows|mingw/i)
```