• Join/Login
  • Business Software
  • Open Source Software
  • For Vendors
  • Blog
  • About
  • More
    • Articles
    • Create
    • SourceForge Podcast
    • Site Documentation
    • Subscribe to our Newsletter
    • Support Request
SourceForge logo
For Vendors Help Create Join Login
SourceForge logo
SourceForge logo
Business Software
Open Source Software
SourceForge Podcast
Resources
  • Articles
  • Case Studies
  • Blog
Menu
  • Help
  • Create
  • Join
  • Login
  • Home
  • Browse
  • avrprog
  • Patches

Minor Cleanup of intelhex_readline()

Brought to you by: betoes, pzn
  • Summary
  • Files
  • Reviews
  • Support
  • Mailing Lists
  • Patches
  • Code
Menu ▾ ▴
  • Create Ticket
  • View Stats

Group

Searches

  • Changes
  • Closed Tickets
  • Open Tickets

Help

  • Formatting Help

#1 Minor Cleanup of intelhex_readline()

open
nobody
None
5
2005-05-10
2005-05-10
Robert James Kaes
No

Added the <string.h> header to obtain the definition
for strlen().

Removed magic numbers for the carriage return, line
feed, and colon.

Change the fgets() call to use the size of the "line"
buffer, rather
than hard coding the size. Also, the hard-coded size
was one less than
could be used since fgets() already reads in one less
character than the
size provided in order to have room for the NUL.

Removed a second call the strlen() in the CR+LF test.
If the CR was
removed, the length is obviously one less.

Discussion

  • Robert James Kaes

    Robert James Kaes - 2005-05-10
     
    01_minor_cleanup_of_intelhex_readline().patch
    If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.

SourceForge
  • Create a Project
  • Open Source Software
  • Business Software
  • Top Downloaded Projects
Company
  • About
  • Team
  • SourceForge Headquarters
    225 Broadway Suite 1600
    San Diego, CA 92101
    +1 (858) 422-6466
Resources
  • Support
  • Site Documentation
  • Site Status
  • SourceForge Reviews
SourceForge logo
© 2025 Slashdot Media. All Rights Reserved.
Terms Privacy Opt Out Advertise
×
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.