Skip to content
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.

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

Implement ImageMagick pixel enumeration parser #217

Merged

Conversation

rexim
Copy link
Member

@rexim rexim commented Feb 6, 2016

Close #216

@rexim
Copy link
Member Author

rexim commented Feb 6, 2016

@rexim also don't forget to check that probably a dash.el bug that you found on the stream

@@ -1,10 +1,14 @@
PILL=Pill.json Pill.xpm
TXTS=Big-Pill.txt Blinking-Terrified-Ghost.txt Pacman-Chomping-Down.txt Pacman-Chomping-Left.txt Pacman-Chomping-Right.txt Pacman-Chomping-Up.txt Pacman-Death.txt Pill.txt Red-Ghost-Down.txt Red-Ghost-Left.txt Red-Ghost-Right.txt Red-Ghost-Up.txt Red-Ghost-Win.txt Terrified-Ghost.txt
Copy link
Member

Choose a reason for hiding this comment

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

I don't like the TXTS name very much. Maybe name it like TXT_IMGS? These are not ordinary text files, after all.

Copy link
Member Author

Choose a reason for hiding this comment

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

Agree. Will give a better name

@ForNeVeR
Copy link
Member

ForNeVeR commented Feb 7, 2016

@rexim please evaluate if this PR will require to change our Melpa recipe.

@rexim rexim assigned rexim and unassigned ForNeVeR Feb 7, 2016
@rexim rexim changed the title Implement ImageMagick pixel enumeration parser [WIP] Implement ImageMagick pixel enumeration parser Feb 7, 2016
@rexim
Copy link
Member Author

rexim commented Feb 7, 2016

@ForNeVeR since #216 is a subtask of #41 (see #41 (comment)) we don't need to update MELPA recipe right now. But we definitely need to change it if the outcome of #41 will be positive. We need to keep track of such things somehow. I think I will create an issue for that.

@ForNeVeR
Copy link
Member

ForNeVeR commented Feb 7, 2016

Acknowledged.

rexim added 3 commits February 7, 2016 14:13
And also regenerate all the IM pixel enumeration sprites
Stands for ImageMagick pixel enumeration
@rexim
Copy link
Member Author

rexim commented Feb 7, 2016

@ForNeVeR fixed all the remarks. Please verify.

@rexim rexim changed the title [WIP] Implement ImageMagick pixel enumeration parser [Implement ImageMagick pixel enumeration parser Feb 7, 2016
@rexim rexim changed the title [Implement ImageMagick pixel enumeration parser Implement ImageMagick pixel enumeration parser Feb 7, 2016
@rexim rexim assigned rexim and unassigned ForNeVeR Feb 7, 2016
@rexim
Copy link
Member Author

rexim commented Feb 7, 2016

Sorry, assigning back to myself. Please see #218 (comment). I need to change the Cask file to include those sprites/*.txt files. Let's keep track of the MELPA recipe changes in our Cask file.

@rexim rexim assigned ForNeVeR and unassigned rexim Feb 7, 2016
@rexim
Copy link
Member Author

rexim commented Feb 7, 2016

@ForNeVeR done. Please, review now. :)

@ForNeVeR
Copy link
Member

ForNeVeR commented Feb 7, 2016

LGTM 👍

@ForNeVeR ForNeVeR assigned rexim and unassigned ForNeVeR Feb 7, 2016
rexim added a commit that referenced this pull request Feb 7, 2016
…es-216

Implement ImageMagick pixel enumeration parser
@rexim rexim merged commit 5cf1b85 into feature/one-canvas-technique-41 Feb 7, 2016
@rexim rexim deleted the feature/infernal-format-sprites-216 branch February 7, 2016 09:11
@rexim
Copy link
Member Author

rexim commented Feb 7, 2016

Thank you Mr @ForNeVeR

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.

2 participants