Page MenuHomePhabricator

remove render2 from master
Closed, ResolvedPublic

Description

This doesn't work and hasn't been actively developed for over 2 years. The master branch is for functional code, so this should be either moved to a (private) branch for continued development or removed entirely if it's a dead pile of code.

zmike created this task.Jul 30 2018, 2:26 PM
zmike triaged this task as TODO priority.

Please do not overlook users such as https://phab.enlightenment.org/D6696

zmike added a comment.Jul 30 2018, 5:10 PM

Please do not overlook users such as https://phab.enlightenment.org/D6696

Yeah that patch is why this ticket exists. How/why are you using this, what downstream modifications do you have, literally nothing about this makes sense. This is the functional equivalent of using a beta API.

@kimcinoo can you help verify that that user is actually using render2? As I mentioned on D6696, I don't see how render2 can be activated except by using X11 and setting an env var.

Something is wrong :-(

Turns out the render2/region.c code is also being used by Eina_Tiler (for region_new function). Before this ticket happens, the region code will likely need to be relocated (perhaps made into Eina_Region??)

zmike edited projects, added Active Work Proposal; removed Voting.Aug 20 2018, 4:55 AM