rizard/floodlight — pull request #3 by hwchiu (2014-08-16 17:50:42+00:00)
Title: Openflow 1.3#3

Hi. I have test all Device and Topology Rest API and it works now.

Solved
---------
- Modify MacAddress serializer to use the MacAddress Type instead of Long
- Change IPv4Address serializer to use the Ipv4Address Type rather than Integer
- Adding two new serializer for OFPort and VlanVid.
- Formatting the tab with 4 spaces.
- For the topologyInstace, we should use the equals method of DatapathId instead of the double equals operator to compare two DataoathId because the comparison is based on value not reference.
If we use the double equals, it will meet errors sometimes when the buildRoute method has been called.

Not Yet
----------
- I use the Rest API "/wm/core/module/all/json " to fetch the modules but meets the error message, it shows a NullPointerException [at here](https://github.com/rizard/floodlight/blob/master/src/main/java/net/floodlightcontroller/core/module/ModuleLoaderResource.java#L68), I think the reason it that we can only get the serviceImpl by context.getAttributes().get(). That's why we will get a NULL when we want to get FloodlightModuleLoader.
Do you have any idea about this ? Maybe adding the static attribute back to FloodlightModulerLoader?
- When i use the "curl -s http://localhost:8080/wm/core/controller/switches/json" to fetch the switches information, it shows an error at the floodlight console for the OF1.0 but works for OF1.3. You can see the error messages [here](https://gist.github.com/hwchiu/3d296bfa43fc01515c5e)