Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Typo in ml_gtk_text_buffer_get_iter_at_line_index #181

Conversation

herbelin
Copy link
Contributor

Hi, there seems to be a typo in the definition of ml_gtk_text_buffer_get_iter_at_line_index as its code refers to gtk_text_buffer_get_iter_at_line_offset (like in ml_gtk_text_buffer_get_iter_at_line_offset) while I guess gtk_text_buffer_get_iter_at_line_index is instead expected. I did not test the change, but we observed an utf8 issue in coqide which might be caused by this. Best, Hugo

@garrigue
Copy link
Owner

Indeed, this is a no-brainer. Thanks.

@herbelin
Copy link
Contributor Author

@garrigue: what are your plans/policy for releases. Is there some chance that a new release including the fix is done before Coq releases its version 8.20, planned end of June?

@garrigue
Copy link
Owner

I can do that the week of June 10th. Now that OCaml 5.2 is released is a good timing for me too.

@herbelin
Copy link
Contributor Author

That would be great! Thanks.

garrigue added a commit to garrigue/opam-repository that referenced this pull request Jun 10, 2024
CHANGES:

2024.06.10 [Jacques]
  * Prepare for release
  * Fix typos in README.md (garrigue/lablgtk#182) [Sylvain Chiron]

2024.05.20 [Jacques]
  * Typo in ml_gtk_text_buffer_get_iter_at_line_index. (garrigue/lablgtk#181) [Hugo Herbelin]

2024.05.14 [Jacques]
  * Support windows sockets in Glib.IO.channel_of_descr (garrigue/lablgtk#176)
    [with Michael Soegtrop]

2024.03.24 [Jacques]
  * fix pointer incompatibility in ml_gtk_style_context_list_classes (garrigue/lablgtk#178)
@garrigue
Copy link
Owner

@herbelin I have made a release last week. The opam package (ocaml/opam-repository#26047) is in the pipeline, but I have no idea how long it will take.

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.

None yet

2 participants