Skip to content

Conversation

@memre
Copy link

@memre memre commented Sep 25, 2011

I have some source code organization and fixes.
Added features:

  • distclean: "make distclean" command clears source with unneccessary autobuilt files
  • x86_64 problems fixed.
  • install libusbk, usbk and man into /usr, not /usr/local.
  • .gitignore file updated for cmake
  • some cpp files converted into pure c, since there is no need to use c++ to these files
  • added a todo list in near future.

I can begin qusbk project to my x86_64 Pardus 2011 machine.

Mehmet Emre Atasever added 3 commits September 26, 2011 00:53
- fix 64bit problem
- install files to /usr, not /usr/local
@memre
Copy link
Author

memre commented Sep 25, 2011

Oh, I just saw that my old changes already merged :) Most of the "changes item" can be ignored, since there is not much fixes but the cosmetics :)

Mehmet Emre Atasever added 15 commits September 26, 2011 22:27
Writing functions inside libusbk.h header causes multiple
definitions if two or more module includes libusbk.

When a module includes libusbk.h, functions includes
in the module too, and if another module includes libusbk.h
then other all functions will be placed that module again and
linker causes multiple definition functions error.

headers should contain just prototypes of the functions.
- some functions should be inline
- remove unneccessary library functions
- enumerated types reorganazed both their typedefs and types.
- elements of the usbk struct renamed
- elements of the usbk structure documented.
- libudef clears all contexts as default, so manually unrefs causes
  segfaults.
- add libusbk++ skeleton
- add config.h to libusbk
- some refactors to libusbk for c++ bindings
By the time developing libusbk++ and usbk_cli, if there is a header
installed in the system, sometimes that header might be included
instead of most current version of libusbk.h{,pp}.

This will be fixed soon, but we wrote relative path as temporary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant