Skip to content
This repository was archived by the owner on Sep 2, 2021. It is now read-only.

Removed string_util since is unused#556

Merged
nethip merged 1 commit intoadobe:masterfrom
ficristo:remove-string-util
Jun 21, 2016
Merged

Removed string_util since is unused#556
nethip merged 1 commit intoadobe:masterfrom
ficristo:remove-string-util

Conversation

@ficristo
Copy link
Copy Markdown
Collaborator

In the cefclient example the corrispondent methods are in testrunner files and used only for tests.

@ficristo
Copy link
Copy Markdown
Collaborator Author

/cc @ingorichter if you have a bit of time...

@nethip
Copy link
Copy Markdown
Contributor

nethip commented Jun 21, 2016

LGTM .

@ficristo Could you resolve the conflicts, so I could go ahead with the merge?

@ficristo ficristo force-pushed the remove-string-util branch from df0ca92 to 4441bd8 Compare June 21, 2016 17:48
@ficristo
Copy link
Copy Markdown
Collaborator Author

I've rebase the patch.

@nethip nethip merged commit b68893f into adobe:master Jun 21, 2016
@nethip
Copy link
Copy Markdown
Contributor

nethip commented Jun 21, 2016

Thanks @ficristo

@ficristo ficristo deleted the remove-string-util branch June 21, 2016 17:54
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