Skip to content
This repository was archived by the owner on Mar 15, 2025. It is now read-only.

Some cleanup in Transform. #59

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

Some cleanup in Transform. #59

wants to merge 1 commit into from

Conversation

@Possseidon
Copy link
Owner

  • Removed static create in favor of just using make_unique/make_shared.
  • Added implicit conversion from a dual quaternion.
  • Added a hint, that copying is currently disabled.
  • Made full_transform_ mutable.
  • Fixed some refs/consts.

@Possseidon Possseidon added dang-gl cleanup Some simple cleanup in the codebase labels Aug 27, 2021
@Possseidon Possseidon self-assigned this Aug 27, 2021
@Possseidon Possseidon force-pushed the dev/transform-cleanup branch 3 times, most recently from 786412d to 10e3189 Compare June 5, 2022 16:14
@Possseidon
Some cleanup in Transform.
a9a3257 
- Removed static create in favor of just using make_unique/make_shared.
- Added implicit conversion from a dual quaternion.
- Added a hint, that copying is currently disabled.
- Made full_transform_ mutable.
- Fixed some refs/consts.
@Possseidon Possseidon force-pushed the dev/transform-cleanup branch from 10e3189 to a9a3257 Compare June 5, 2022 20:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@Possseidon Possseidon

Labels

cleanup Some simple cleanup in the codebase dang-gl

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Possseidon