On 3/13/19 10:04 AM, Eric Farman wrote:
+static void run_readipl(SubChannelId schid) +{ + if (do_cio(schid, 0x00, CCW_FMT0)) { + panic("dasd-ipl: Failed to run Read IPL channel program");Maybe add a newline here...+ }
There are three places you are suggesting a newline. All of them appear to be before an if statement's closing brace. Can you explain why you feel these are needed? I don't really see any improvement in readability with these newlines.
-- -- Jason J. Herne ([email protected])
