Skip to content
This repository has been archived by the owner on Jun 10, 2020. It is now read-only.

Proguard : reduce the size of the binaries #395

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

tyrcho
Copy link
Contributor

@tyrcho tyrcho commented Jun 11, 2014

DO NOT MERGE AS IS !
I'm creating this PR to start the conversation

I've been trying to integrate Proguard to reduce the size of the downloaded executable, to no success at the moment. I lack experience with Proguard to fix it.

If anyone want to give a hand on this he is very welcome !

@gtch
Copy link
Member

gtch commented Jun 11, 2014

I think a useful step in making Proguard work will be to get rid of the code in Config.programHelper() that instantiates classes using strings, because tends to confuse tools that do compile-time analysis like Proguard.

I actually started on that last weekend in my branch config-cleanup but I haven't yet finished it. That branch moves the code in net.hearthstats.Main to two classes — one for Windows and one for OS X — which means that dependencies can be resolved at compile time instead of runtime. I should finish that off in the next few days... and it should be easier to get Proguard working once that is done.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants