16 Oct
2012
16 Oct
'12
5:19 p.m.
Evan Rempel <erempel@uvic.ca> writes:
The last set of patches I posted contained an error that would result in segmentation fault. I forgot to check if the parameter was defined before using it.
These fixed patches don't have the error.
I only had a quick glance, but I liked what I saw, thanks! However, since 3.4 already has a MACADDR parser, I was thinking it'd make sense to move that over to be a light wrapper around LLADDR, appropriately parameterised? (I can do that once I get around to pull your patches into my sandbox tree, but if it was done before I merge the patches, that'd make my life a tiny bit easier :) -- |8]