Code review comment for lp://qastaging/~seb128/whoopsie-preferences/touch-writable-image

Revision history for this message
Iain Lane (laney) wrote :

I think this is fine. I was confused about why you use a static string for the return variable, but I suppose it's so you don't have to free it?

I have one small comment but I'll fix it when merging. Will test and upload.

review: Approve

« Back to merge proposal