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

Rename Matrix's "Item" members to "Row" #2

Open

aaronfranke opened this issue Jun 26, 2020 · 0 comments

aaronfranke commented Jun 26, 2020

NeoAxisEngine/Sources/Engine/NeoAxis.Core/MathEx/Matrix4.cs

Lines 26 to 30 in e4d0998

    
           /// <summary> 
        
           /// The first row of the matrix. 
        
           /// </summary> 
        
           [Browsable( false )] 
        
           public Vector4 Item0;

Since they are rows, it would make a lot more sense to call them "Row0/Row1" etc.

Wrapper properties for "Item0/Item1" etc can be used to preserve API compatibility if desired.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment