avr-libc-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[avr-libc-dev] [patch #2876] Extend pgmspace.h with pgm_read_long()


From: Eric Weddington
Subject: [avr-libc-dev] [patch #2876] Extend pgmspace.h with pgm_read_long()
Date: Wed, 07 Apr 2004 13:11:06 -0400
User-agent: Mozilla/4.0 (compatible; MSIE 5.01; Windows NT 5.0; Avant Browser [avantbrowser.com])

This mail is an automated notification from the patch tracker
 of the project: AVR C Runtime Library.

/**************************************************************************/
[patch #2876] Latest Modifications:

Changes by: 
                Eric Weddington <address@hidden>
'Date: 
                Wed 04/07/04 at 17:11 (America/Denver)

------------------ Additional Follow-up Comments ----------------------------
Should the naming of the macros be consistent? Right now, we have:
pgm_read_byte()
pgm_read_word()

Wouldn't the next logical name be: pgm_read_dword()?

Not pgm_read_long()? Since the meaning of "long" in C can change (or will 
change) with the use of -mint8.






/**************************************************************************/
[patch #2876] Full Item Snapshot:

URL: <http://savannah.nongnu.org/patch/?func=detailitem&item_id=2876>
Project: AVR C Runtime Library
Submitted by: 0
On: Tue 03/30/04 at 14:22

Category:  None
Priority:  5 - Normal
Resolution:  None
Assigned to:  troth
Originator Email:  address@hidden
Status:  Open


Summary:  Extend pgmspace.h with pgm_read_long()

Original Submission:  The header file pgmspace.h

/* $Id: pgmspace.h,v 1.16.2.2 2003/09/18 18:41:17 troth Exp $ */

does not support pgm_read_long(). I suggest to extend this
file by applying the attached patch.

Follow-up Comments
------------------


-------------------------------------------------------
Date: Wed 04/07/04 at 17:11         By: arcanum
Should the naming of the macros be consistent? Right now, we have:
pgm_read_byte()
pgm_read_word()

Wouldn't the next logical name be: pgm_read_dword()?

Not pgm_read_long()? Since the meaning of "long" in C can change (or will 
change) with the use of -mint8.

-------------------------------------------------------
Date: Wed 04/07/04 at 16:37         By: troth
There's a few copy/paste errors in the originally submitted patch.

Also, the original patch didn't include any context information. For future 
reference, please try to submit patches generated using diff's "-u" option. 
This makes the patch easier to read (most of the time) and allows the patch 
program to apply the patch to a file that may have been modified since the diff 
was generated.

I've attached an updated patch which fixes the copy/paste errors I noticed.

I don't have any objections to applying this patch to HEAD or 1.0 since it is a 
simple extension of the API.

I'll try to get it into 1.0.4, but I need to do a better review of the inline 
asm first. So far, from a quick review it looks ok to me.






File Attachments
-------------------

-------------------------------------------------------
Date: Wed 04/07/04 at 16:37  Name: avr-libc-patch-2876.diff  Size: 7KB   By: 
troth
Updated version of original patch. Fixes copy/paste errors and includes context.
http://savannah.nongnu.org/patch/download.php?item_id=2876&amp;item_file_id=3071

-------------------------------------------------------
Date: Tue 03/30/04 at 14:22  Name: pgmspace.h.patch  Size: 5KB   By: None

http://savannah.nongnu.org/patch/download.php?item_id=2876&amp;item_file_id=3027






For detailed info, follow this link:
<http://savannah.nongnu.org/patch/?func=detailitem&item_id=2876>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/







reply via email to

[Prev in Thread] Current Thread [Next in Thread]