All of lore.kernel.org
 help / color / mirror / Atom feed
* [Patch 1/4] cld: Drop comment about struct layout
@ 2009-08-08  6:27 Pete Zaitcev
  2009-08-08  6:40 ` Jeff Garzik
  0 siblings, 1 reply; 2+ messages in thread
From: Pete Zaitcev @ 2009-08-08  6:27 UTC (permalink / raw)
  To: Jeff Garzik; +Cc: Project Hail List

We do not rely on precise placement of fields in unrelated structs anymore.

Signed-Off-By: Pete Zaitcev <zaitcev@redhat.com>

diff --git a/include/cld_msg.h b/include/cld_msg.h
index 124acbb..89ab066 100644
--- a/include/cld_msg.h
+++ b/include/cld_msg.h
@@ -188,10 +188,6 @@ struct cld_msg_get {
 struct cld_msg_get_resp {
 	struct cld_msg_resp	resp;
 
-	/* should mirror struct raw_inode, except that inum's type
-	 * should always be uint64_t, regardless of server's
-	 * cldino_t definition
-	 */
 	uint64_t		inum;		/**< unique inode number */
 	uint32_t		ino_len;	/**< inode name len */
 	uint32_t		size;		/**< data size */

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Patch 1/4] cld: Drop comment about struct layout
  2009-08-08  6:27 [Patch 1/4] cld: Drop comment about struct layout Pete Zaitcev
@ 2009-08-08  6:40 ` Jeff Garzik
  0 siblings, 0 replies; 2+ messages in thread
From: Jeff Garzik @ 2009-08-08  6:40 UTC (permalink / raw)
  To: Pete Zaitcev; +Cc: Project Hail List

Pete Zaitcev wrote:
> We do not rely on precise placement of fields in unrelated structs anymore.
> 
> Signed-Off-By: Pete Zaitcev <zaitcev@redhat.com>

applied 1-4



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-08-08  6:40 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-08-08  6:27 [Patch 1/4] cld: Drop comment about struct layout Pete Zaitcev
2009-08-08  6:40 ` Jeff Garzik

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.