Opened 2 years ago

Closed 2 years ago

Last modified 16 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 Rev(s):
Wiki Page:

Description


Attachments (2)

Change History (16)

comment:1 Changed 2 years ago by bernalex

  • Status changed from new to patch

comment:2 follow-up: Changed 2 years 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 2 years 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 2 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 2 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 follow-up: Changed 2 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 2 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 2 years ago by jstolarek

Ah, yes! You're right.

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

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

comment:11 Changed 2 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 2 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 2 years ago by hvr

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

comment:14 Changed 16 months ago by hvr

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