Skip to content

caml-types.el: Fix missing format argument#992

Merged
gasche merged 1 commit intoocaml:trunkfrom
hchunhui:fix
Dec 30, 2016
Merged

caml-types.el: Fix missing format argument#992
gasche merged 1 commit intoocaml:trunkfrom
hchunhui:fix

Conversation

@hchunhui
Copy link
Contributor

Fixes: 5fa4e02 ("caml-types.el: Extract the feedback logic to a separate function.")

@gasche
Copy link
Member

gasche commented Dec 30, 2016

Indeed, sorry -- I should have reviewed the refactoring more carefully.

Given that the faulty changed is part of the 4.04.0 release, could you please write a Changes entry to let other users know that the issue is fixed? It would be extra nice if you could include in it the error message given by Emacs in the error case (assuming there is an error message).

Please include your Changes entry, respecting the format, in the Bug fixes section of the Next minor version (4.04.1)`, so that it is included in the next bugfix release -- if any.

Now the emacs mode can not show the kind of call at point. Emacs shows the
following message:

  let: Wrong number of arguments: (lambda (info format) "Displays INFO using the given FORMAT." (message (format format info)) (save-current-buffer (set-buffer caml-types-buffer) (erase-buffer) (insert info))), 1

This issue was introduced in commit 5fa4e02. It extracted the feedback logic
to function caml-types-feedback, but deleted the format argument by accident.
So let's fix it.

Fixes: 5fa4e02 ("caml-types.el: Extract the feedback logic to a separate function.")
@hchunhui
Copy link
Contributor Author

@gasche Thank you! I have changed my commit.

Now the emacs mode can not show the kind of call at point. Emacs shows the
following message:

let: Wrong number of arguments: (lambda (info format) "Displays INFO using the given FORMAT." (message (format format info)) (save-current-buffer (set-buffer caml-types-buffer) (erase-buffer) (insert info))), 1

This issue was introduced in commit 5fa4e02. It extracted the feedback logic
to function caml-types-feedback, but deleted the format argument by accident.
So let's fix it.

Fixes: 5fa4e02 ("caml-types.el: Extract the feedback logic to a separate function.")

@gasche gasche merged commit 83c744a into ocaml:trunk Dec 30, 2016
@gasche
Copy link
Member

gasche commented Dec 30, 2016

Thanks for the nice work! I merged in trunk and in the 4.04 release branch.

camlspotter pushed a commit to camlspotter/ocaml that referenced this pull request Oct 17, 2017
caml-types.el: Fix missing format argument
EmileTrotignon pushed a commit to EmileTrotignon/ocaml that referenced this pull request Jan 12, 2024
Factor out shared yaml file parsing code between the different ood data items. Simplifies the process of adding new data formats to ood.

* Result monad style
* Streamlining
* Spread
* Use Result in monad style
* Remove redefinition of Result.fold
* Formatting
* Use Result.map
* Use custom let instead of >>=
* Bring-up outreachy
* Use let* instead of >>=, cont't
* All Yaml key to be different from filename
* Refactoring
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants