Opened 4 years ago

Closed 4 years ago

Last modified 3 years ago

#9008 closed feature request (fixed)

Data.Function: Add reverse application operator

Reported by: bernalex Owned by:
Priority: normal Milestone: 7.10.1
Component: libraries/base Version: 7.8.2
Keywords: report-impact Cc: hvr, ekmett
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Test Case:
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:

Description


Attachments (2)

Change History (16)

comment:1 Changed 4 years ago by bernalex

Status: newpatch

comment:2 Changed 4 years ago by tibbe

Resolution: invalid
Status: patchclosed

Adding functions to base libraries need to be discussed and agreed on on the libraries@ mailing list. After it's approved, please provide a link to the mailing list discussion together with the patch. For this particular issue (adding a reverse function application operator) I believe there have been about 1010 discussion and every single one have ended with "no", which is why we don't have a reverse function application operator.

comment:3 Changed 4 years ago by thoughtpolice

Cc: ekmett added
Resolution: invalid
Status: closednew

This was already approved by the core libraries committee including Edward (I've CC'd him on the ticket for reference), and the date of approval is in the commit. I guess Alex merely forgot to put a reference in the actual ticket.

comment:4 Changed 4 years ago by jstolarek

please provide a link to the mailing list discussion

Forgive the off-topic, but does library mailing list finally have a web archive so that it can be linked to?

comment:5 in reply to:  4 Changed 4 years ago by bernalex

Replying to jstolarek:

Forgive the off-topic, but does library mailing list finally have a web archive so that it can be linked to?

Yes, <http://www.haskell.org/pipermail/libraries/>.

comment:6 Changed 4 years ago by jstolarek

Interesting. Looks like the archive was always there. I recall several months ago no one on #ghc channel was able to point to it and Google also didn't find anything. Everyone was convinced that it does not exist and considered it a major pain. Well, that must've been a misunderstanding. Thanks for clearing that up.

comment:7 in reply to:  6 Changed 4 years ago by hvr

Replying to jstolarek:

Interesting. Looks like the archive was always there. I recall several months ago no one on #ghc channel was able to point to it and Google also didn't find anything. Everyone was convinced that it does not exist and considered it a major pain. Well, that must've been a misunderstanding. Thanks for clearing that up.

Btw, that could have been about the internal core-library-committee list, whose archives were/are not publicly available.

comment:8 Changed 4 years ago by jstolarek

Ah, yes! You're right.

comment:9 in reply to:  2 Changed 4 years ago by bernalex

Replying to tibbe:

please provide a link to the mailing list discussion together with the patch

I can't find a link. Sorry. I guess the announcement was on the internal closed list. It was likely following this[0] discussion. Edward refers to it in [1]. Hopefully this is satisfactory evidence for now. I leave it to Edward or others to provide more information.

[0] <http://www.haskell.org/pipermail/libraries/2013-October/021423.html> [1] <http://www.haskell.org/pipermail/haskell-cafe/2014-April/113748.html>

comment:10 Changed 4 years ago by bernalex

I forgot to add the bug number to the commit message. Please use the latter patch.

comment:11 Changed 4 years ago by ekmett

Actually I was talking about a different committee with non-publicly available mailing list. =)

With the core-libraries-committee we're setting up a bug tracker so you'll be able to link from issues like this to appropriate resolutions.

When this last arose I spoke to hvr about us using the github issue tracker for base, but that was put on hold due to the decision to fold base into the ghc repository.

I suppose it is time to just give in and use a trac.

comment:12 Changed 4 years ago by Herbert Valerio Riedel <hvr@…>

In 1bf6c0e482cfe4b9dfa0b5ed18a5741ba44fc226/ghc:

Add reverse application operator Data.Function.(&)

Add `&` as the reverse application operator with `infixl 1`, which allows
it to be nested in `$` (re #9008).

Approved by the core libraries committee on 2013-10-14.

This also bumps the `base` version number to 4.7.1.0

comment:13 Changed 4 years ago by hvr

Resolution: fixed
Status: newclosed

comment:14 Changed 3 years ago by hvr

Keywords: report-impact added
Note: See TracTickets for help on using tickets.