Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#17 Changed an array declaration, shortened initiation

closed-rejected
Sean Morrison
Modeling (12)
5
2008-04-20
2008-04-08
No

The grpnam variable was declared as an array of 9 chars, when it only needed to be 6. There was very robust code to set up the solid, region, and group names that I shortened with simple strncpy's.

Discussion

  • mk_bolt.c

     
    Attachments
  • Sean Morrison
    Sean Morrison
    2008-04-20

    Logged In: YES
    user_id=785737
    Originator: NO

    Not a functional change. Patch should have used bu_strlcpy and should not have been broken up into multiple trackers, just increases review/apply burden (for an already trivial patch).

     
  • Sean Morrison
    Sean Morrison
    2008-04-20

    • assigned_to: nobody --> brlcad
    • status: open --> closed-rejected