Skip to content

Implement full logic of publish and subscribe#341

Merged
yang-g merged 7 commits intogrpc:masterfrom
chen-wang:master
Feb 3, 2015
Merged

Implement full logic of publish and subscribe#341
yang-g merged 7 commits intogrpc:masterfrom
chen-wang:master

Conversation

@chen-wang
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was static because I need to use it twice. Here since you only need it once, maybe a local string will do.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe change the function body to

  std::ifstream json_key_file(FLAGS_service_account_key_file);
  std::stringstream key_stream;
  key_stream << json_key_file.rdbuf();
  return key_stream.str();

yang-g added a commit that referenced this pull request Feb 3, 2015
Implement full logic of publish and subscribe
@yang-g yang-g merged commit 51d7716 into grpc:master Feb 3, 2015
@lock lock bot locked as resolved and limited conversation to collaborators Feb 2, 2019
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