[xmlsec] mscrypto api support, initial (alpha) release
Wouter
wsh@xs4all.nl
Tue, 2 Sep 2003 10:07:23 +0200 (CEST)
Hi Aleksey,
Thanks for creating the cvs branch. I've checked out the new branch, and
discovered there are a few glitches causing the software won't compile out
of the box. These will be fixed, together with adjusting the code style
and layout. I'd already noticed that I wasn't using the identing/layout
used throughout the xmlsec lib, but in my enthousiasm I didn't take notice
of that (until now :)
The documentation found inside the souces is not complete at all, and
cannot be regarded as anything usefull at this time. I didn't spend any
time to that so far.
Thanks for the remarks, I'll update the code a bit following those.
Updates will follow (hopefully shortly :)
Wouter
PS If you want to know more about the company I work for, check this
website out: http://www.cordys.com.
> Hi, Wouter!
>
> First of all, I would like to say "thanks" to you and Cordys R&D BV for
> sharing
> your work with others! This is really great and I think many people
> would join me
> in saying "Thanks a lot!"
>
> I have created a special branch in xmlsec CVS for MSCrypto work. As we
> discussed before, we would merge this branch to the trunk as soon as we
> would
> feel that the code is ready. The branch name is "XMLSEC_MSCRYPTO_083103"
> and the branch start tag is "XMLSEC_MSCRYPTO_083103_START". You can
> check out the branch using following CVS command:
>
> cvs -z3 -d ':pserver:anonymous@anoncvs.gnome.org:/cvs/gnome' checkout -r
> XMLSEC_MSCRYPTO_083103 xmlsec
>
> As usual, there might be some delay before changes would be propagated to
> anonymous CVS.
>
> I did not do a full review and I just did a quick look. I hope to do a
> more in depth
> review in a couple weeks. However, I have spotted few problems I would
> like to
> resolve. All of these are minor things but they help to maintain the
> project :)
>
> 0) AUTHORS file. Please update ./AUTHORS file and include appropriate
> information about you and your company. If you give a URL to your
> company
> Web site then I can publish it on xmlsec pages too.
> 1) End-of-line. The patch has DOS (\r\n) end of line and the rest of
> the code
> uses UNIX (\n) one. I would prefer to keep the consistent code style
> thus
> I have converted all your files to use UNIX eol. I am not sure that
> anything
> could be done about that on your side but if you can check your
> editor settings
> and if there is an "eol" mode and set it to Unix. If you would not
> find it then it's
> not a big problem because I have a script to convert DOS eol to UNIX
> one.
> 2) Indentation. Well, it's completely screwed up :) I have fixed it
> in a few files
> but it's just wrong in all other places. Again, I would like to have
> consistent source
> code style thus I would appreciate if you can change your editor
> settings
> to have 4 character tabs and fill tabs with spaces.
> 3) Code style. There are few other things that are dif