thanks that you worked on gendef tool. It would be helpful, if you
could provide to this list a diff file of your changes. So your
changes have a chance to be reviewed, and maybe applied to our master,
too.
Of course, it is always helpful to describe a bit the subject of
improvement you did, so that other people can understand the intention
of your changes more easily.
I'm thinking to create a patch for gendef and merge it with original gendef,
but the problem is, I did the modification not in one way.
What I mean is I did this more than one time and took couple of month to accomplish it (a lot of change but unnecessary).
So I'm thinking to backtrack a little bit, so this modification can be merge with original gendef
without taking to much differences between 2 of them,
but still can accomplish what I want with gendef.
The major difference is (I think):
1. Always generate @Ordinal, not @Retpop, event with a symbol start with question mark (this is for binutils-dlltool git-35fd2d).
2. Add NONAME keyword for symbol without a name, eg: "ord_1000 @1000 NONAME" (binutils-dlltool will add 1000 ordinal number into 2 IDATA section).
3. Add STACKSIZE and HEAPSIZE (well, looks like binutils-dlltool processing it).
4. Add command line option to disable @Retpop generation for 32-bit dll.
5. Rename some command line option.
6. Add direct approach for ms-mangle header file.
7. Remove unnecessary comment in def.
8. No space for forward function, eg: "= NTDLL.<symbol>" -> "=NTDLL.<symbol>" (it should remove in the future, maybe this is just for comment thing).
9. Disable FS redirector for 64-bit gendef.
ToDo:
1. Primary: Double Quote symbol name with a dot in it, eg: "abcd.efgh" (found in stdc++-6).
(Got an error with Binutils-dlltool if no quote for that kind of symbol name)
2. Secondary: Major: Postpone?: Add STRUCT keyword so binutils-dlltool can generate _nm_<symbol> (found in stdc++-6).
(First must do modification in binutils-dlltool/defparse/deflex - urgh).
I'm also include a patch file to see what difference between both of them.
Please don't merge it unless you do modification for my patch.
Please Delete this. I got an error for this.
Last edit: Rudy 2016-11-16
Hello Rudy,
thanks that you worked on gendef tool. It would be helpful, if you
could provide to this list a diff file of your changes. So your
changes have a chance to be reviewed, and maybe applied to our master,
too.
Of course, it is always helpful to describe a bit the subject of
improvement you did, so that other people can understand the intention
of your changes more easily.
Regards,
Kai
2016-11-16 20:47 GMT+01:00 Rudy rudyy@users.sf.net:
I'm thinking to create a patch for gendef and merge it with original gendef,
but the problem is, I did the modification not in one way.
What I mean is I did this more than one time and took couple of month to accomplish it (a lot of change but unnecessary).
So I'm thinking to backtrack a little bit, so this modification can be merge with original gendef
without taking to much differences between 2 of them,
but still can accomplish what I want with gendef.
The major difference is (I think):
1. Always generate @Ordinal, not @Retpop, event with a symbol start with question mark (this is for binutils-dlltool git-35fd2d).
2. Add NONAME keyword for symbol without a name, eg: "ord_1000 @1000 NONAME" (binutils-dlltool will add 1000 ordinal number into 2 IDATA section).
3. Add STACKSIZE and HEAPSIZE (well, looks like binutils-dlltool processing it).
4. Add command line option to disable @Retpop generation for 32-bit dll.
5. Rename some command line option.
6. Add direct approach for ms-mangle header file.
7. Remove unnecessary comment in def.
8. No space for forward function, eg:
"= NTDLL.<symbol>" -> "=NTDLL.<symbol>"(it should remove in the future, maybe this is just for comment thing).9. Disable FS redirector for 64-bit gendef.
ToDo:
1. Primary: Double Quote symbol name with a dot in it, eg: "abcd.efgh" (found in stdc++-6).
(Got an error with Binutils-dlltool if no quote for that kind of symbol name)
2. Secondary: Major: Postpone?: Add STRUCT keyword so binutils-dlltool can generate
_nm_<symbol>(found in stdc++-6).(First must do modification in binutils-dlltool/defparse/deflex - urgh).
I'm also include a patch file to see what difference between both of them.
Please don't merge it unless you do modification for my patch.
Last edit: Rudy 2016-11-22