Opened 17 months ago

Closed 17 months ago

Last modified 7 months 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 Revisions:

Description


Attachments (2)

Change History (16)

comment:1 Changed 17 months ago by bernalex

  • Status changed from new to patch

comment:2 follow-up: Changed 17 months ago by tibbe

  • Resolution set to invalid
  • Status changed from patch to closed

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 17 months ago by thoughtpolice

  • Cc ekmett added
  • Resolution invalid deleted
  • Status changed from closed to new

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 follow-up: Changed 17 months 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 17 months 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 follow-up: Changed 17 months 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 17 months 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 17 months ago by jstolarek

Ah, yes! You're right.

comment:9 in reply to: ↑ 2 Changed 17 months 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 17 months ago by bernalex

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

comment:11 Changed 17 months 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 17 months 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 17 months ago by hvr

  • Resolution set to fixed
  • Status changed from new to closed

comment:14 Changed 7 months ago by hvr

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