Ok, some comments:
1. You did not change VERSION_INFO in configure.ac. I guess it was just overlooked, right? The API is changed, even though in the compatible way.
2. The xkl_config_item_new_with_data - I do not have strong objection, but why are just plain setters/getters not sufficient?
3. Why is that in Makefile.am?
-xklavier_headers = xklavier.h xkl_config_registry.h xkl_engine.h \
- xkl_config_rec.h xkl_config_item.h xkl_engine_marshal.h
+xklavier_headers = xkl_engine.h xkl_config_item.h xkl_config_registry.h \
+ xkl_config_rec.h xkl_engine_marshal.h xklavier.h
4. set_log_appender is mostly for debugging, so for now let's ignore it
5. I will check around get_current_state. Perhaps making it const would be affordable...
Ok, some comments: item_new_ with_data - I do not have strong objection, but why are just plain setters/getters not sufficient? registry. h xkl_engine.h \ marshal. h registry. h \ marshal. h xklavier.h
1. You did not change VERSION_INFO in configure.ac. I guess it was just overlooked, right? The API is changed, even though in the compatible way.
2. The xkl_config_
3. Why is that in Makefile.am?
-xklavier_headers = xklavier.h xkl_config_
- xkl_config_rec.h xkl_config_item.h xkl_engine_
+xklavier_headers = xkl_engine.h xkl_config_item.h xkl_config_
+ xkl_config_rec.h xkl_engine_
4. set_log_appender is mostly for debugging, so for now let's ignore it
5. I will check around get_current_state. Perhaps making it const would be affordable...