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.

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

Comparison method violates its general contract in FogUtil.calculateVisibility(FogUtil.java:81) #80

Closed
JamzTheMan opened this issue Jun 25, 2018 · 0 comments
Closed

Comparison method violates its general contract in FogUtil.calculateVisibility(FogUtil.java:81) #80

JamzTheMan opened this issue Jun 25, 2018 · 0 comments
Assignees
@JamzTheMan
Labels
bug fixed bug fix is committed for next release MapTool issue relates to MapTool code vs 3rd party libs
Milestone
1.4.5.3

Comments

@JamzTheMan
Copy link
Owner

Ug, Not sure why I "fixed" this in the last release now and yet the "fix" is now causing the issue?

Need to revert it back to previous revision of VisibleAreaSegment...
@JamzTheMan JamzTheMan added bug MapTool issue relates to MapTool code vs 3rd party libs in progress issue is being worked labels Jun 25, 2018
@JamzTheMan JamzTheMan self-assigned this Jun 25, 2018
JamzTheMan added a commit that referenced this issue Jun 26, 2018
@JamzTheMan
#80 Comparison method violates its general contract in...
a4ed88f 
 * Reverting VisibleAreaSegment to previous version...

Signed-off-by: Jamz <Jamz@Nerps.net>
@JamzTheMan JamzTheMan added fixed bug fix is committed for next release and removed in progress issue is being worked labels Jun 26, 2018
@JamzTheMan JamzTheMan added this to the 1.4.5.3 milestone Sep 2, 2018
JamzTheMan added a commit that referenced this issue Sep 2, 2018
@JamzTheMan
Release 1.4.5.3 (#91)
bac72a2 
* #77: Drag path does not seem to be working correctly.

	* Tweaked the A* algorithm for more natural/straighter moves though
open spaces for both square and hex grids
	* Tweaked the A* algorithm to find shortest path more consistently

Task-Url: #77

Signed-off-by: Jamz <Jamz@Nerps.net>

* #77: Drag path does not seem to be working correctly.

 * Change log amended
 * Spotless applied

Task-Url: #77

Signed-off-by: Jamz <Jamz@Nerps.net>

* Minor update to PDF Extract

 * Updated PDFBox lib to latest
 * Sync'd PDF extract code from TokenTool (similiar but not 100% same on
purpose)

Signed-off-by: Jamz <Jamz@Nerps.net>

* #80 Comparison method violates its general contract in...

 * Reverting VisibleAreaSegment to previous version...

Signed-off-by: Jamz <Jamz@Nerps.net>

* #81: Cell Highlight distance text not sizing for grid sizes

 * Cell Highlight distance text now properly scales with grid size
 * Adjusted logging slightly

Task-Url: #81

Signed-off-by: Jamz <Jamz@Nerps.net>

* CI Updates

* CI Updates

* CI Updates

* CI Updates

* Gradle Process Updated

 * Vendor property now comes from gradle.properties
 * Version now comes from Git, uses SNAPSHOT-{commit} for
development/non-tagged releases and uses Tag name if commit is tagged
 * Updated Apache logging versions
 * sentry.properties file now generated, DSN is now only configured for
tagged releases, eg Production. This prevents sentry logging during
development

Signed-off-by: Jamz <Jamz@Nerps.net>

* #41: Bug Fix to allow tokens without sight to move

 * Tokens without sight can move within current PC tokens field of view
when iFoW server options are checked
 * Tweaked sentry.properties to use dummy DSN when in development mode

Task-Url: #41

Signed-off-by: Jamz <Jamz@Nerps.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JamzTheMan JamzTheMan

Labels
bug fixed bug fix is committed for next release MapTool issue relates to MapTool code vs 3rd party libs
Projects
None yet
Milestone
1.4.5.3
Development

No branches or pull requests
1 participant
@JamzTheMan