Page MenuHomePhabricator

Revert "elm config - dont leak config if re-loading it again later on"
AcceptedPublic

Authored by cedric on May 7 2018, 11:26 AM.

Details

Reviewers
zmike
Summary

This reverts commit cffb31f4a768ba6112646960f6e197f85f02f2ca.

After this commit, my enlightenment keep crashing at very frequent interval. I am
guessing that somewhere I have set a more agressive policy for saving the config
on disk, but I can't find which flags trigger it (It only happen on one of my laptop,
not on my two others system).
Depends on D6059

Diff Detail

Repository
rEFL core/efl
Branch
devs/cedric/lifecycle
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 6166
cedric created this revision.May 7 2018, 11:26 AM
zmike accepted this revision.May 8 2018, 8:16 AM

This should probably have test cases added to showcase the issue(s) and verify that there are no further issues raised when fixing the initial issue(s).

This revision is now accepted and ready to land.May 8 2018, 8:16 AM

I haven't landed this one as this is still pending investigation with T6879, but I am giving up the only laptop where I see this crash today and don't have much time digging in. So there will be a call to be made if nobody understand why there is a crash to revert this patch or keep it @stefan_schmidt .

ManMower added a subscriber: ManMower.

Until someone decides what to do with this, I'm going to tag it unmergeable.