neop/raytracer
1
0
Fork 0
Code Issues 12 Pull requests Projects Releases Packages Wiki Activity

Move implementations #31

Merged
neoq merged 11 commits from moveimpl into rmincludes 2019-07-17 08:44:59 +02:00
Conversation 1 Commits 11 Files changed 11 +182 -166
neoq commented 2019-07-17 08:41:55 +02:00 (Migrated from github.com)
Copy link

out of headers to their respective cpp files.
I realized #25 needs to be fixed before it makes sense to fix #30

out of headers to their respective cpp files. I realized #25 needs to be fixed before it makes sense to fix #30
neoq commented 2019-07-17 08:44:45 +02:00 (Migrated from github.com)
Copy link

the definitions that are still in the headers are all tiny constructors or getters

the definitions that are still in the headers are all tiny constructors or getters
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
CRITICAL

Extremely urgent, needs to be done NOW!

  
bug

Something isn't working

  
code quality

Related to the typical metrics of code quality

  
commentation

Concerning the comments in the code

  
documentation

Docs of public interfaces, wikis, and readmes

  
duplicate

This issue or pull request already exists

  
feature

Regarding a feature, existing or not

  
improvement

Enhancements and improvements

  
incomplete

Not all of it is in place

  
meta

Not about the content, but about the project itself

  
missing

It is not there, but it SHOULD be

  
parallelism

Task that are run at the same instant in parrallel

  
performance

Physical resource usage of the sw, execution time, optimisation

  
pig: 🐖

   
postponed

Will be opened at a later time when implementation is feasible

  
structure

Scaffolding like folders, build files

  
testing

Anything pertaining to testing

  
urgent

The issue needs to be handled soon

No labels
CRITICAL
bug
code quality
commentation
documentation
duplicate
feature
improvement
incomplete
meta
missing
parallelism
performance
pig: 🐖
postponed
structure
testing
urgent
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
neop/raytracer!31
No description provided.