- getc returns an int.

src/platform-linux.cc

 // Copyright 2006-2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 215 matching lines @@
 void OS::LogSharedLibraryAddresses() {
 #ifdef ENABLE_LOGGING_AND_PROFILING
   FILE *fp;
   fp = fopen("/proc/self/maps", "r");
   if (fp == NULL) return;
   while (true) {
     uintptr_t start, end;
     char attr_r, attr_w, attr_x, attr_p;
     if (fscanf(fp, "%" V8PRIxPTR "-%" V8PRIxPTR, &start, &end) != 2) break;
     if (fscanf(fp, " %c%c%c%c", &attr_r, &attr_w, &attr_x, &attr_p) != 4) break;
-    char c;
+    int c;
     if (attr_r == 'r' && attr_x == 'x') {
-      while (c = getc(fp), c != EOF && c != '\n' && c != '/');
+      while (c = getc(fp), (c != EOF) && (c != '\n') && (c != '/'));
       char lib_name[1024];
       bool lib_has_name = false;
       if (c == '/') {
         ungetc(c, fp);
         lib_has_name = fgets(lib_name, sizeof(lib_name), fp) != NULL;
       }
       if (lib_has_name && strlen(lib_name) > 0) {
         lib_name[strlen(lib_name) - 1] = '\0';
       } else {
         snprintf(lib_name, sizeof(lib_name),
                  "%08" V8PRIxPTR "-%08" V8PRIxPTR, start, end);
       }
       LOG(SharedLibraryEvent(lib_name, start, end));
     }
-    while (c = getc(fp), c != EOF && c != '\n');
+    while (c = getc(fp), (c != EOF) && (c != '\n'));
   }
   fclose(fp);
 #endif
 }
 
 
 int OS::StackWalk(Vector<OS::StackFrame> frames) {
   // backtrace is a glibc extension.
 #ifdef __GLIBC__
   int frames_size = frames.length();
@@ skipping 418 matching lines @@
   }
 
   // This sampler is no longer the active sampler.
   active_sampler_ = NULL;
   active_ = false;
 }
 
 #endif  // ENABLE_LOGGING_AND_PROFILING
 
 } }  // namespace v8::internal