After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 744836 - Documentation comment about grapheme in pango-layout.c is wrong?
Documentation comment about grapheme in pango-layout.c is wrong?
Status: RESOLVED FIXED
Product: pango
Classification: Platform
Component: general
unspecified
Other Linux
: Normal minor
: ---
Assigned To: pango-maint
pango-maint
Depends on:
Blocks:
 
 
Reported: 2015-02-20 00:07 UTC by Gene Hightower
Modified: 2015-05-05 22:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Gene Hightower 2015-02-20 00:07:32 UTC 
$ git diff
diff --git a/pango/pango-layout.c b/pango/pango-layout.c
index 68c50f0..6fcf61c 100644
--- a/pango/pango-layout.c
+++ b/pango/pango-layout.c
@@ -2010,7 +2010,7 @@ pango_layout_move_cursor_visually (PangoLayout *layout,
  * @trailing: (out): location to store a integer indicating where
  *             in the grapheme the user clicked. It will either
  *             be zero, or the number of characters in the
- *             grapheme. 0 represents the trailing edge of the grapheme.
+ *             grapheme. 0 represents the leading edge of the grapheme.
  *
  * Converts from X and Y position within a layout to the byte
  * index to the character at that logical position. If the
Comment 1 Behdad Esfahbod 2015-02-26 00:45:06 UTC 
I have a vague memory that about a decade ago, before I became the pango maintainer, I stumbled upon that also and thought the documentation is wrong...  Will check it out.  Thanks!
Comment 2 Behdad Esfahbod 2015-05-05 22:05:13 UTC 
Should be fixed.  Please check:
https://git.gnome.org/browse/pango/commit/?id=9117a1c6c7717287f2f8d76c5926bd2e5551c52e