JonasToth added a comment.

Sorry for the long delay!

I have one question about a slighty different code constellation.

  std::optional<std::array<int, 4>> my_array;
  
  for(int i = 0; i < 42; ++i)
      (*my_array)[i] = 42;

Will this be fixed properly?

  for(int i = 0; i < 42; ++i)
      gsl::at(*my_array, i) = 42;

If the braces in `(*my_array)` would be moved into the function that is ok as 
well.

This is just of general interest and should not block this patch in my opinion, 
as its a different bug.


Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56661/new/

https://reviews.llvm.org/D56661



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to