<div dir="ltr">I think you should have words with these pesky developers exporting all things.<div><br></div><div>I wasn't aware of the new option and it will probably work nicely for me as I can now re-enable warn_export_all in my vim syntax checker and explicitly ignore it in test modules and such. I don't think there was a nice way to achieve this before. Swings and roundabouts I suppose.</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, 31 Oct 2017 at 14:33 Loïc Hoguin <<a href="mailto:essen@ninenines.eu">essen@ninenines.eu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
The export_all warning used to be very useful before. It was only<br>
enabled when you used an option (warn_export_all).<br>
<br>
The warning has now been enabled by default. As a result, an additional<br>
option has been added to disable it when necessary (nowarn_export_all).<br>
<br>
This made the warn_export_all warning useless. Why? Because you can<br>
disable this warning at the module level. For example:<br>
<br>
-module(abc).<br>
-compile(export_all).<br>
-compile(nowarn_export_all).<br>
<br>
A module with both of these options will not trigger a warning anymore<br>
and still export everything. This is great if you're writing a test<br>
suite with a lot of functions because you don't have to worry about the<br>
warning.<br>
<br>
It's not so great when you want to actually detect export_all uses<br>
because developers can now just use both options in a module and<br>
completely silence this warning in every module where it should trigger!<br>
The only way I can see to detect it now is to grep through source files...<br>
<br>
Flawed design.<br>
<br>
Cheers,<br>
<br>
--<br>
Loïc Hoguin<br>
<a href="https://ninenines.eu" rel="noreferrer" target="_blank">https://ninenines.eu</a><br>
_______________________________________________<br>
erlang-questions mailing list<br>
<a href="mailto:erlang-questions@erlang.org" target="_blank">erlang-questions@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-questions" rel="noreferrer" target="_blank">http://erlang.org/mailman/listinfo/erlang-questions</a><br>
</blockquote></div>