[PATCH] D46509: [llvm-rc] Implement the BITMAP resource type

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 7 10:20:44 PDT 2018


zturner accepted this revision.
zturner added inline comments.
This revision is now accepted and ready to land.


================
Comment at: tools/llvm-rc/ResourceFileWriter.cpp:705
+  // Skip the 14 byte BITMAPFILEHEADER.
+  if (Buffer.size() < 14 || Buffer[0] != 'B' || Buffer[1] != 'M')
+    return createError("Incorrect bitmap file.");
----------------
Can you add a line above this `constexpr size_t BITMAPFILEHEADER_size = 14;` and then use this here and below?


Repository:
  rL LLVM

https://reviews.llvm.org/D46509





More information about the llvm-commits mailing list