Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Added extra rotations for 1.8 #52

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Aaron1011
Copy link


Edited by turt2live

Related links:

this.degree = degree;
}

public int getDegree() {

Choose a reason for hiding this comment

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

DOUBLE.

this.degree = degree;
}

public double getDegree() {

Choose a reason for hiding this comment

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

getRotation

Also JavaDocs

@@ -5,26 +5,37 @@
* <p>
* It represents how something is viewed, as opposed to cardinal directions.

Choose a reason for hiding this comment

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

Missing documentation on deprecation reason.

/**
 * ...
 * @deprecated This does not handle the various degrees of rotation available
 */

@turt2live turt2live self-assigned this Nov 11, 2014
@deathcap
Copy link

deathcap commented Apr 6, 2015

Is this API used where arbitrary rotation angles are possible? (given the deprecation of enum values in favor of double)

deathcap referenced this pull request in GlowstoneMC/Glowstone Apr 6, 2015
…nstead of doubles

Modification to:
#514 Continuation of @wow4201's Item Frames

instead of taking the Glowkit change to add getRotationAngle() etc.:
https://github.com/GlowstoneMC/Glowkit/pull/52 Add degree-based rotations

change the implementation to use new rotation values in the enum:
GlowstonePlusPlus/Glowkit-Legacy@b6c156d Support the new Rotation values.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants