Created attachment 508657 [details] [diff] [review] Trim lsp annotation The LSP annotation code is taking up a large chunk of the AppNotes space, which I'd like to use for other purposes. This should trim it down a bit.
How about we just remove it from the appnotes entirely? It gets cut off before anything interesting anyways :-/
(In reply to comment #1) > How about we just remove it from the appnotes entirely? It gets cut off before > anything interesting anyways :-/ That works for me too. Do you want to take it out then?
6 years ago
Created attachment 508669 [details] [diff] [review] Patch Ted, want to rubberstamp this?
Well, it _is_ a very easy way to notice that a non-English version of Windows is being used.
If that information is particularly useful we could expose it in a much less voluminous manner.
Why not remove the note variable as well?
Yeah, I'll kill that when I land this.
I landed the reviewed version: http://hg.mozilla.org/mozilla-central/rev/c698ef73edae You can fix the dead code in a follow up.
http://hg.mozilla.org/mozilla-central/rev/347c9f93d23b And the dead code is gone.