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

Refactoring attempts by Tom den Braber & Michel Kraaijeveld #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Refactoring attempts by Tom den Braber & Michel Kraaijeveld #8

wants to merge 1 commit into from

Commits on Apr 10, 2016

  1. refactored switch statements and moved code blokcs to seperate functions 

    added tests for UVCoordinatesGenerator

now with the actual test file

moved math utility functions to proper classes, added tests for the functions that were moved

added tests for functions that were moved

fixed indentation, added comments

added tests to gradle for plugins

wrote tests for retrieving subclassname, moved code to proper place

created configuration file, refactored and wrote tests for dispatching FbxElements to FbxObjects

removed outcommented code

removed repositories that are not used anymore

fixed wrongly initialized loggers

added method comments

fixed wrong description

Added more UVCoordinatesGenerator tests

reverted back to v2.0-beta of mockito

removed unused import and restructured import list

removed redundant variable

took out material loading from sceneloader, added tests for material to object linking

more tests for SceneLoader concerning textures

extracted FbxTextureLoader, added tests for image/texture link

removed incorrect gradle file

deduced interface for two loader classes

cleaned up gradle file
    Michel committed Apr 10, 2016
    Configuration menu
    Copy the full SHA
    51d14ae View commit details
    Browse the repository at this point in the history