Opened 12 years ago
Closed 6 years ago
#8935 closed enhancement (fixed)
HAIKU_INCLUDE_PATENTED_CODE is now useless
Reported by: | leavengood | Owned by: | bonefish |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Build System | Version: | R1/Development |
Keywords: | Cc: | ||
Blocked By: | Blocking: | #9658 | |
Platform: | All |
Description
The HAIKU_INCLUDE_PATENTED_CODE define and its associated configure option --include-patented-code is now useless since freetype has now been moved out into an optional package.
I think it should be removed. But we all know how I feel about the patented code nonsense.
Change History (14)
follow-up: 2 comment:1 by , 12 years ago
comment:2 by , 12 years ago
Replying to aldeck:
Either way we need to provide a freetype package with subpixel aa enabled. I had to rebuild freetype from source to get subpixel aa working again here. Appearance preflet needs to be updated too.
Yeah that is on my list, maybe for alpha4. The incorrect text from the Appearance preflet should at least be removed. Also turning on the freetype subpixel filter (FT_Library_SetLcdFilter(fLibrary, FT_LCD_FILTER_DEFAULT);) really, really improves the rendering.
comment:3 by , 12 years ago
We could provide a secondary freetype optionalpackage if desired. What's the specific freetype configure option that you use for enabling this?
comment:4 by , 12 years ago
Yes that would be awesome. I'm not on my Haiku box right now, but that should be in freetype2/freetype/config/ftoption.h by uncommenting the define FT_CONFIG_OPTION_SUBPIXEL_RENDERING.
follow-up: 8 comment:5 by , 12 years ago
added a bep file to Haikuports to build with the subpixel rendering enabled. I'll leave it to someone from a FREE COUNTRY to build and upload the built files for gcc2 and gcc4.
comment:6 by , 12 years ago
So HAIKU_INCLUDE_PATENTED_CODE
is still relevant then -- it should choose the right optional package depending on it It .may still be used in the future for other code in the repository as well.
comment:8 by , 12 years ago
Replying to scottmc:
added a bep file to Haikuports to build with the subpixel rendering enabled. I'll leave it to someone from a FREE COUNTRY to build and upload the built files for gcc2 and gcc4.
I live in Italy. If the patents permit, i can build and upload :-)
follow-up: 11 comment:9 by , 12 years ago
Since May 2010 patents to subpixel rendering are expired so HAIKU_INCLUDE_PATENTED_CODE is now useless. Subpixel rendering could be enabled without law violation in any country.
Reference: http://www.freetype.org/patents.html
follow-up: 12 comment:11 by , 12 years ago
Replying to X512:
Since May 2010 patents to subpixel rendering are expired so HAIKU_INCLUDE_PATENTED_CODE is now useless. Subpixel rendering could be enabled without law violation in any country.
Reference: http://www.freetype.org/patents.html
As that page clearly shows - only the hinting patents are free now - the subpixel rendering may likely be covered by the ClearType patents, which it specifies are still in effect and would require using a different color filter to evade.
comment:12 by , 12 years ago
Replying to umccullough:
As that page clearly shows - only the hinting patents are free now - the subpixel rendering may likely be covered by the ClearType patents, which it specifies are still in effect and would require using a different color filter to evade.
Does complete hinting bytecode interpreter enabled by default? Why not to enable this different color filter? This is probably better than nothing...
Also does rendering glyph outlines with subpixel antialiasing by AGG violate patents?
comment:13 by , 11 years ago
Eventually a proper solution would be to provide two versions of the package: One which has Subpixel Rendering and Hinting disabled and is installed by default. Another one in the repository which has it enabled and can be installed by users who live in a country where there are no software patents (most of the world except US).
Either way we need to provide a freetype package with subpixel aa enabled. I had to rebuild freetype from source to get subpixel aa working again here. Appearance preflet needs to be updated too.