<div class="gmail_quote">one minor nit</div><div class="gmail_quote"><br></div><div class="gmail_quote">On Fri, Aug 19, 2011 at 8:00 AM, Stefan Tauner <span dir="ltr"><<a href="mailto:stefan.tauner@student.tuwien.ac.at">stefan.tauner@student.tuwien.ac.at</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">+static int read_content_from_file(romlayout_t *entry, uint8_t *newcontents)<br>
+{<br>
+       char *file;<br>
+       FILE *fp;<br>
+       int len;<br>
+<br>
+       file = entry->file;<br>
+       len = entry->end - entry->start + 1;<br>
+       if (file[0] != '\0') {<br>
+               int numbytes;<br>
+               if ((fp = fopen(file, "rb")) == NULL) {<br>
+                       perror(file);<br></blockquote><div><br></div><div>Hmmmm, I'm not sure why we used perror()... Perhaps it would be better to use msg_gerr(strerror(errno))?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">


+                       return 1;<br>
+               }<br>
+               numbytes = fread(newcontents + entry->start, 1, len, fp);<br>
+               fclose(fp);<br>
+               if (numbytes != len) {<br>
+                       perror(file);<br></blockquote><div><br></div><div>same as above.</div><div><br></div></div>-- <br>David Hendricks (dhendrix)<br>Systems Software Engineer, Google Inc.<br>