Skip to content

Dirty Flag pattern #560

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
iluwatar opened this issue Apr 1, 2017 · 10 comments
Closed

Dirty Flag pattern #560

iluwatar opened this issue Apr 1, 2017 · 10 comments
Labels
epic: pattern
Milestone
1.20.0

Comments

@iluwatar
Copy link
Owner

iluwatar commented Apr 1, 2017

http://gameprogrammingpatterns.com/dirty-flag.html
@abhi-agr
Copy link

I am working on this.

@iluwatar
Copy link
Owner Author

Ok @abhi-agr

@prakis
Copy link

prakis commented Jun 9, 2017

@abhi-agr I implemented something similar in my scene graph code without knowing this pattern. If you need collaborator in this pattern please let me know.

@iluwatar
Copy link
Owner Author

@abhi-agr are you still implementing this?

@prakis
Copy link

prakis commented Dec 25, 2017

Sorry I am not working on this, I got busy with my work.

@iluwatar
Copy link
Owner Author

This issue is free for taking again

@waisuan
Copy link
Contributor

waisuan commented Feb 3, 2018

@iluwatar

Hello. I would like to work on this particular pattern. I'll be drawing up inspiration from https://www.takeupcode.com/podcast/89-design-patterns-dirty-flag/ .

Is that alright?

@iluwatar
Copy link
Owner Author

iluwatar commented Feb 5, 2018

@waisuan ok

@waisuan
Copy link
Contributor

waisuan commented Feb 20, 2018

Kindly take a look at my pull request and lemme know if I've missed out on something. Thanks! @iluwatar

npathai added a commit that referenced this issue May 16, 2018
@npathai
Merge pull request #717 from waisuan/master
eddfe76 
Dirty Flag pattern #560
@npathai
Copy link
Contributor

npathai commented May 16, 2018

Closing this issue. PR successfully merged.

@npathai npathai closed this as completed May 16, 2018
@iluwatar iluwatar added this to the 1.20.0 milestone May 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
epic: pattern
Projects
None yet
Milestone
1.20.0
Development

No branches or pull requests
5 participants
@prakis @iluwatar @npathai @waisuan @abhi-agr