[Lldb-commits] [PATCH] D18228: Make File option flags consistent for Python API

Francis Ricci via lldb-commits lldb-commits at lists.llvm.org
Thu Mar 17 10:25:05 PDT 2016


fjricci added a comment.

So this was definitely a decision that I was debating. But I assume that the Append flag must imply the Write flag, since you can't open a file for Append without also opening it for write ("a" and "a+" both include write privileges). So I figured that having both "Append" and "Write" was redundant. And since "a" was previously using "Append" while "a+" was previously using "Write", I think that one of them needed to be changed to be consistent with the other. I can do it the other way around though if that's preferable.


http://reviews.llvm.org/D18228





More information about the lldb-commits mailing list