Skip to content

Remove a private internal method that is not used.#4667

Merged
RickWinter merged 1 commit intoAzure:mainfrom
RickWinter:rawresponse
Jun 9, 2023
Merged

Remove a private internal method that is not used.#4667
RickWinter merged 1 commit intoAzure:mainfrom
RickWinter:rawresponse

Conversation

@RickWinter
Copy link
Copy Markdown
Member

@RickWinter RickWinter commented May 30, 2023

Unused private virtual method

Fixes #4662

Copy link
Copy Markdown
Member

@antkmsft antkmsft left a comment

Choose a reason for hiding this comment

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

LGTM, but let's wait until after this month release (releasing this would require releasing all libraries), and let's still mention it in the changelog in Breaking Changes section.

Copy link
Copy Markdown
Member

@Jinming-Hu Jinming-Hu left a comment

Choose a reason for hiding this comment

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

Changes to storage LGTM

Comment thread sdk/core/azure-core/inc/azure/core/operation.hpp
@antkmsft
Copy link
Copy Markdown
Member

antkmsft commented Jul 5, 2023

See also #4749

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Azure.Core KeyVault Storage Storage Service (Queues, Blobs, Files)

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Azure::Core::Operation<T>::GetRawResponseInternal() is never used

4 participants