OK, I convinced myself that it should actually look like this:

```ruby
if Pry::Helpers::BaseHelpers.windows? && !Pry::Helpers::BaseHelpers.windows_ansi? && 
  begin
    require 'win32console'
    # The mswin and mingw versions of pry require win32console, so this should
    # only fail on jruby (where win32console doesn't work).
    # Instead we'll recommend ansicon, which does.
  rescue LoadError
    unless Pry.config.show_ansicon_warning
      warn <<-WARNING
For a better Pry experience, please use ansicon: https://github.com/adoxa/ansicon
If you use an alternative to ansicon and don't want to see this warning, you can
add "Pry.config.show_ansicon_warning = false" to your pryrc.
      WARNING
    end
  end
end
```

I think it's confusing to have the option control whether we show the warning and also control whether we try to load win32console.