Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Junk #208

Merged
merged 5 commits into from
Jun 10, 2012
Merged

Remove Junk #208

merged 5 commits into from
Jun 10, 2012

Conversation

Hoikas
Copy link
Member

@Hoikas Hoikas commented Jun 10, 2012

These changes remove lots of dead, obsolete, and useless code.

  • CSR and SimpleNet were removed for offering no visible benefit.
  • All apps and tools that were obsoleted by eap's netcore rewrite have been removed.
  • Obsolete library components (except pfKI and some pfCharacter stuff) have been removed.
  • Obsolete PythonLib has been removed due to Boost dependencies and dependency on the old networking core.

@dpogue
Copy link
Member

dpogue commented Jun 10, 2012

Apps/plPageInfo should not be removed, as it is especially useful for testing changes to the ResManager, KeyedObject, and page loading code.
plPageOptimizer should also probably not be removed, although I believe it needs updating to compile properly.

Both of those (and the plMD5 and plSHA tools) should probably be moved to the Tools folder.

@Hoikas
Copy link
Member Author

Hoikas commented Jun 10, 2012

plPageInfo has been restored

@Hoikas
Copy link
Member Author

Hoikas commented Jun 10, 2012

plPageOptimizer has been restored

branan pushed a commit that referenced this pull request Jun 10, 2012
@branan branan merged commit 1fac13c into H-uru:master Jun 10, 2012
Deledrius pushed a commit to Deledrius/Plasma that referenced this pull request Dec 4, 2012
Hoikas pushed a commit to Hoikas/Plasma that referenced this pull request Dec 13, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants