Skip to content

Remove unused RingBuffer class#1803

Merged
Gerg merged 1 commit intomasterfrom
remove_ring_buffer
Aug 24, 2020
Merged

Remove unused RingBuffer class#1803
Gerg merged 1 commit intomasterfrom
remove_ring_buffer

Conversation

@Gerg
Copy link
Copy Markdown
Member

@Gerg Gerg commented Aug 24, 2020

Call was removed in 25efe93 as part of removing varz

@cf-gitbot
Copy link
Copy Markdown

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/174478509

The labels on this github issue will be updated when the story is started.

Call was removed in 25efe93 as part of removing varz
@Gerg Gerg force-pushed the remove_ring_buffer branch from 12caa9a to 3ac1c9a Compare August 24, 2020 19:32
@Gerg Gerg requested a review from tcdowney August 24, 2020 20:56
Copy link
Copy Markdown
Member

@tcdowney tcdowney left a comment

Choose a reason for hiding this comment

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

this was my go to ring buffer implementation 😞

looks alright to me

Comment thread lib/cloud_controller.rb

require 'uaa/token_coder'

module VCAP; end
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What's this for?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Previously, RingBuffer was the first file loaded with the VCAP module defined:

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

So, rather than shifting module definitions around for what random file gets to be the first one to define VCAP, I pulled it up to the top level.

@Gerg Gerg merged commit 9c6b8f8 into master Aug 24, 2020
@tjvman tjvman deleted the remove_ring_buffer branch August 24, 2021 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants