Michael Zimmermann
Michael Zimmermann
that'd make sense. btw the longest_line calculation was insane. It's better to either use for loops like this: ```rust self.longest_line = 0; if let Some(diff) = self.diff.as_ref() { for hunk...
additionally, there should be an option to remove the typedefs or remove the leading`_` from the structs so you can use `struct name` instead of `Name` as required by many...
which distros do have nanopb packages? IMO that's somewhere between useless and dangerous considering that you can change the struct layouts via defines.
@carlescufi done :)
@bpiepiora I've rebased the branch, that's all I can do. The shippable failure looks like something that was not caused by my change.
@tejlmand done :)
yea. @tejlmand can you take another look and report if there are any (conceptional) problems? If there are I'll fix that as well.
well the beauty of this PR was that no additional local commands were necessary for setting up a workspace and to not have any `ZEPHYR_BASE` variable set. But I agree...
> how would you know that Zephyr was found ? well west does that already. It looks for `ZEPHYR_BASE` and falls back to looking for a project with name `zephyr`....
I also have to admit that we recently started a new project based on 1.14.x because bluetooth SIG certification was a requirement and for 2.2.x that was only available since...