Fluid API #460

DaanVanYperen · 2016-09-11T19:43:28Z

Working. Components on classpath go in, fluid interface comes out. Room for performance improvements in the plugin. See #458 for more information.

Getting started

Install this branch locally. (specifically artemis-fluid and child modules).
https://github.com/junkdog/artemis-odb/wiki/Fluid-Entities

Todo

…vide convenience mappers.

… comparison to spot ambiguous methods better.

…ermination.

…onent getter prefix.

preferences.prefixComponentGetter = "get" // prefix for E::[get]pos() preferences.prefixComponentCreate = "" // prefix for E::[]pos() preferences.prefixComponentHas = "has" // prefix for E::[has]Pos() preferences.prefixComponentRemove = "remove" // prefix for E::[remove]Pos() preferences.generateTagMethods = true // add tag convenience methods. preferences.generateGroupMethods = true // add group convenience methods. preferences.generateBooleanComponentAccessors = true // Generate boolean accessors for flag components? preferences.swallowGettersWithParameters = false // Globally swallow getters?

…ing.

jeffzi · 2016-09-18T12:56:54Z

First of all, thanks for the awesome work :)

Pos::set(x,y) generates E::pos(p0,p1) It would be nicer to use the same parameter names as the setmethod.

DaanVanYperen · 2016-09-22T19:33:15Z

@junkdog if you're happy where it sits give it a MERGE so I can start working on other things D:

DaanVanYperen · 2016-09-22T19:36:26Z

@shatterboom since we reflect over class files there's no easy way to obtain parameter names, unless compiling with debug information or running sort of metadata weaving utility. It's part of #461.

DaanVanYperen · 2016-10-14T17:07:08Z

@junkdog :'(

junkdog · 2016-10-31T00:05:51Z

on Sep 11

ewww.... there should have been more 🍪 🍪 🍪 and much less 🕐

junkdog · 2016-10-31T00:08:18Z

want to do the spaces to tab conversion? feels revisionist to commit it myself.

DaanVanYperen added 16 commits September 10, 2016 20:41

Prep module for fluid api generator.

3913b1c

Initial draft.

98443e0

Add direct accessor generator strategy.

60fd420

Add initial source generator and tests we can extend easily.

fdba51e

Javadoc.

b0b46c0

Make strategy more generic and less about components.

2419431

Use types instead of strings for returnType.

350dcb2

I should commit before going to bed.

0d7b03d

Access level for fields and methods. Create supermapper system to pro…

dca0c1c

…vide convenience mappers.

Separate fluid API into module for maven and core.

b35952f

Add PoC maven plugin.

27d148a

Fleshed out maven plugin.

7f7bc34

Move complexity away from maven plugin to keep it light.

d454091

Add gradle plugin.

95640d1

Javadoc + remove placeholder classes.

1ff5f18

Further cleanup.

1d3f501

DaanVanYperen added the API Discussion label Sep 11, 2016

DaanVanYperen assigned junkdog and DaanVanYperen Sep 11, 2016

DaanVanYperen added 11 commits September 12, 2016 20:30

Add implicit field getters/setters to fluid interface.

7c56d1a

Expose component methods to fluid interface.

48347f3

Basic model validation.

4e0425b

Report duplicate methods in validator. Ignore method variable name in…

3192adb

… comparison to spot ambiguous methods better.

Add check for ambiguous method name regardless of return type.

fa3c6b5

Fix tests.

24d1a60

Fix tests. Add field ambiguous validation.

e3d8edb

Add boolean

ce57c4b

Travis JDK8

ee00a40

Cache component public methods, fields lookup, and flag component det…

a2ee1a2

…ermination.

Simplify boolean accessor.

95978af

DaanVanYperen added 18 commits September 16, 2016 22:19

Add global preferences to both fluid plugins, settings to change comp…

cd34d30

…onent getter prefix.

Fluid classpath filtering preference + multithreaded reflection scann…

37fed7a

…ing.

Name component collect strategy to match framework used.

587decf

Speeeeeeeeeeeeeeeeed. Exclude plugin classpath.

a9d68d9

Fix artifact resolution in maven.

3c96274

Add void E::deleteFromWorld()

1311a27

Add @Fluid(exclude=true).

3393ab9

Add basic integration tests for fluid interface.

0f6bc4a

E::DeleteFromWorld integrationtest.

09ed8eb

Fluid group methods integrationtest.

0d3d774

Fluid tag methods integrationtest.

7df7797

Component interation methods integrationtest.

fe5b071

Flag component integrationtests.

7abdb92

Integrationtests for @Fluid renaming or excluding.

47a26f3

Add integrationtests for parameterized getters.

63c6d86

Add integrationtests for suppressing parameterized getters return value.

3a0d722

Workaround for entity isActive test.

8cb2247

DaanVanYperen added 2 commits September 21, 2016 21:22

Guard against reserved keywords as method names.

0c3a683

Flush just cause.

80a69de

junkdog and others added 2 commits September 25, 2016 02:45

only expose setters for non-final fields

5795ae9

Fix possible field/parameter clash.

4e72294

junkdog merged commit 182cbd6 into master Oct 31, 2016

DaanVanYperen deleted the 458-fluid-entity-api branch December 22, 2016 10:12

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fluid API #460

Fluid API #460

DaanVanYperen commented Sep 11, 2016 •

edited

Loading

jeffzi commented Sep 18, 2016 •

edited

Loading

DaanVanYperen commented Sep 22, 2016 •

edited

Loading

DaanVanYperen commented Sep 22, 2016

DaanVanYperen commented Oct 14, 2016

junkdog commented Oct 31, 2016

junkdog commented Oct 31, 2016

Fluid API #460

Fluid API #460

Conversation

DaanVanYperen commented Sep 11, 2016 • edited Loading

Todo

jeffzi commented Sep 18, 2016 • edited Loading

DaanVanYperen commented Sep 22, 2016 • edited Loading

DaanVanYperen commented Sep 22, 2016

DaanVanYperen commented Oct 14, 2016

junkdog commented Oct 31, 2016

junkdog commented Oct 31, 2016

DaanVanYperen commented Sep 11, 2016 •

edited

Loading

jeffzi commented Sep 18, 2016 •

edited

Loading

DaanVanYperen commented Sep 22, 2016 •

edited

Loading