Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: self intersecting paths
https://bugzilla.redhat.com/show_bug.cgi?id=702058
Summary: self intersecting paths Product: Fedora Version: rawhide Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: low Priority: unspecified Component: lohit-devanagari-fonts AssignedTo: psatpute@redhat.com ReportedBy: bmassot@free.fr QAContact: extras-qa@fedoraproject.org CC: fonts-bugs@lists.fedoraproject.org, psatpute@redhat.com, i18n-bugs@lists.fedoraproject.org Classification: Fedora Story Points: ---
Description of problem: Most glyphs have a point with the same coordinates as the first point. This extraneous point is useless and creates self intersecting points, which should be avoided.
Version-Release number of selected component (if applicable): 2.4.5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
--- Comment #1 from Pravin Satpute psatpute@redhat.com 2011-05-09 05:43:30 EDT --- Thanks for reporting bug. Can you submit patch as well against trunk?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
--- Comment #2 from Bernard Massot bmassot@free.fr 2011-05-09 16:02:30 EDT --- Created attachment 497905 --> https://bugzilla.redhat.com/attachment.cgi?id=497905 patch to remove self intersecting paths
I corrected self intersecting glyphs using FontForge's "remove overlap" function. I had to "simplify" u0938_u094D.half before removing overlapping.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
--- Comment #3 from Pravin Satpute psatpute@redhat.com 2011-05-18 05:04:34 EDT --- Hi Bernard, i gone through your patch, its very useful -> https://bugzilla.redhat.com/attachment.cgi?id=497905&action=diff
It is removing double points like as below 1068 306 -220 l 1,14,-1 1069 306 -220 l 1,0,1
i tried to apply patch on lohit-devanagari but its look like patch: **** malformed patch at line 341: @@ -1304,7 +1282,6 @@
can you please attach Devanagari sfd file, so i can try it.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
--- Comment #4 from Pravin Satpute psatpute@redhat.com 2011-05-18 05:30:38 EDT --- i can see some more things in patch other than just removing self intersecting.
how did you worked on this?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
--- Comment #5 from Bernard Massot bmassot@free.fr 2011-05-18 12:48:57 EDT --- Created attachment 499639 --> https://bugzilla.redhat.com/attachment.cgi?id=499639 Lohit Devanagari's FontForge sfd file corresponding to patch 497905
(In reply to comment #3)
i tried to apply patch on lohit-devanagari but its look like patch: **** malformed patch at line 341: @@ -1304,7 +1282,6 @@
I just used "svn diff". Should I have done it another way?
can you please attach Devanagari sfd file, so i can try it.
Done.
(In reply to comment #4)
i can see some more things in patch other than just removing self intersecting.
how did you worked on this?
As I said, I had to simplify u0938_u094D.half. Also FontForge keeps some extra info like window size, currently open glyph, etc. I may have done some minor corrections as well, like removing overlap in glyph actually made of two paths (ie. not just only with first and last points overlapping). I don't remember exactly.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
--- Comment #6 from Pravin Satpute psatpute@redhat.com 2011-05-19 05:36:55 EDT --- steps for creating patch :
1) keep copy of "original source" and "modified source" 2) $diff -rup "original source" "modified source" > "patch-name" 3) vi "patch-name"
I will test sfd by tomorrow. yesterday while checking changes i found it good.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
Pravin Satpute psatpute@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
--- Comment #7 from Pravin Satpute psatpute@redhat.com 2011-05-23 07:33:49 EDT --- (In reply to comment #5)
Created attachment 499639 [details] Lohit Devanagari's FontForge sfd file corresponding to patch 497905
Tested the new sfd, it looks nice to me
Now Element->Validation->Find Problems->Paths-> Intersecting Paths not reporting any problems
Rendering also happening properly
I will build in Fedora 16 and Fedora 15 with patch and will push this patch after one week (lets other also test in fedora before making upstream)
Thanks Bernard for you contribution !!
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
--- Comment #8 from Fedora Update System updates@fedoraproject.org 2011-05-24 06:13:14 EDT --- lohit-devanagari-fonts-2.4.5-2.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/lohit-devanagari-fonts-2.4.5-2.fc15
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #9 from Fedora Update System updates@fedoraproject.org 2011-05-24 22:25:23 EDT --- Package lohit-devanagari-fonts-2.4.5-2.fc15: * should fix your issue, * was pushed to the Fedora 15 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing lohit-devanagari-fonts-2.4.5-2.fc15' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/lohit-devanagari-fonts-2.4.5-2.fc15 then log in and leave karma (feedback).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
--- Comment #10 from Pravin Satpute psatpute@redhat.com 2011-06-10 01:35:52 EDT --- Hi Bernard applied patch to upstream !! added your name in the Author list in upstream. see https://fedorahosted.org/lohit/browser/trunk/AUTHORS
congrats you are now the contributor to Lohit :)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
Pravin Satpute psatpute@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Resolution| |RAWHIDE Last Closed| |2011-06-16 04:59:52
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
--- Comment #11 from Fedora Update System updates@fedoraproject.org 2011-06-22 05:16:28 EDT --- lohit-devanagari-fonts-2.4.5-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/lohit-devanagari-fonts-2.4.5-3.fc15
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
--- Comment #12 from Fedora Update System updates@fedoraproject.org 2011-07-12 17:57:52 EDT --- lohit-devanagari-fonts-2.4.5-3.fc15 has been pushed to the Fedora 15 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=702058
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version| |lohit-devanagari-fonts-2.4. | |5-3.fc15 Resolution|RAWHIDE |ERRATA
fonts-bugs@lists.fedoraproject.org