Skip to content

Conversation

@miraleung
Copy link
Contributor

Closes #807, and needed for #804.

@miraleung miraleung requested review from a team and vam-google August 2, 2021 21:04
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 2, 2021
@codecov
Copy link

codecov bot commented Aug 2, 2021

Codecov Report

Merging #811 (552af72) into dev/throw-expr (d5fc5cf) will increase coverage by 0.00%.
The diff coverage is 94.73%.

Impacted file tree graph

@@               Coverage Diff               @@
##           dev/throw-expr     #811   +/-   ##
===============================================
  Coverage           91.30%   91.30%           
===============================================
  Files                 140      140           
  Lines               14996    15012   +16     
  Branches             1083     1086    +3     
===============================================
+ Hits                13692    13707   +15     
  Misses               1009     1009           
- Partials              295      296    +1     
Impacted Files Coverage Δ
...i/generator/engine/writer/ImportWriterVisitor.java 92.94% <75.00%> (+0.02%) ⬆️
...le/api/generator/engine/ast/TryCatchStatement.java 96.87% <96.00%> (-3.13%) ⬇️
...api/generator/engine/writer/JavaWriterVisitor.java 99.21% <100.00%> (ø)
...common/AbstractServiceClientTestClassComposer.java 84.26% <100.00%> (ø)
.../composer/grpc/ServiceClientTestClassComposer.java 93.60% <100.00%> (+<0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d5fc5cf...552af72. Read the comment docs.

Copy link
Contributor

@vam-google vam-google left a comment

Choose a reason for hiding this comment

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

LGTM

+ " try-catch block");
tryCatchStatement.catchVariableExpr().accept(this);
statements(tryCatchStatement.catchBody());
for (int i = 0; i < tryCatchStatement.catchVariableExprs().size(); i++) {
Copy link
Contributor

Choose a reason for hiding this comment

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

super minor: I guess these technically should be done via two simultaneous iterators, but good enough.

Base automatically changed from dev/throw-expr to master August 2, 2021 21:52
@miraleung miraleung merged commit 55ef1a6 into master Aug 2, 2021
@miraleung miraleung deleted the dev/try-catch-list branch August 2, 2021 21:53
suztomo pushed a commit that referenced this pull request Mar 21, 2023
Source-Link: googleapis/synthtool@f58d313
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:f14e3fefe8e361e85752bd9890c8e56f2fe25f1e89cbb9597e4e3c7a429203a3
suztomo pushed a commit that referenced this pull request Mar 21, 2023
…onfig to v1.3.3 (#811)

[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [com.google.cloud:google-cloud-shared-config](https://togithub.com/googleapis/java-shared-config) | `1.3.2` -> `1.3.3` | [![age](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-config/1.3.3/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-config/1.3.3/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-config/1.3.3/compatibility-slim/1.3.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/maven/com.google.cloud:google-cloud-shared-config/1.3.3/confidence-slim/1.3.2)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>googleapis/java-shared-config</summary>

### [`v1.3.3`](https://togithub.com/googleapis/java-shared-config/blob/HEAD/CHANGELOG.md#&#8203;133-httpsgithubcomgoogleapisjava-shared-configcomparev132v133-2022-04-19)

[Compare Source](https://togithub.com/googleapis/java-shared-config/compare/v1.3.2...v1.3.3)

</details>

---

### Configuration

📅 **Schedule**: At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/java-core).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ggj] Support multiple catches in try-catch statements

2 participants