Menu

#51 Change tempnam to mkstemp

Unstable (example)
closed-accepted
nobody
None
5
2015-04-28
2015-03-11
Neos3452
No

Using tempnam is deprecated and should be replaced with mkstemp. This issues a warning on clang. This is an upstreamed change from gstreamer repo.

1 Attachments

Related

Patches: #51

Discussion

  • Branden Archer

    Branden Archer - 2015-03-12

    Thank you for your interest in Check, and thanks for the patch! I'll put
    the patch in and it will appear in the next release of Check. The only
    modifications I'll add is to make the spacing consistent with surrounding
    code and adding CMake support for Windows.

    Is there a particular name that should be put into our AUTHORS file?

    • Branden

    On Wed, Mar 11, 2015 at 3:47 PM, Neos3452 neos3452@users.sf.net wrote:


    Status: open
    Group: Unstable (example)
    Created: Wed Mar 11, 2015 07:47 PM UTC by Neos3452
    Last Updated: Wed Mar 11, 2015 07:47 PM UTC
    Owner: nobody

    Using tempnam is deprecated and should be replaced with mkstemp. This
    issues a warning on clang. This is an upstreamed change from gstreamer repo.


    Sent from sourceforge.net because you indicated interest in
    https://sourceforge.net/p/check/patches/51/

    To unsubscribe from further messages, please visit
    https://sourceforge.net/auth/subscriptions/

     

    Related

    Patches: #51

  • Neos3452

    Neos3452 - 2015-03-14

    Hey, I'm glad I could be useful.

    It's really a small change, but if you insist you can put "Michał Dębski".

     
  • Branden Archer

    Branden Archer - 2015-04-28
    • status: open --> closed-accepted
     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.