Please Review: (518084) Fix out of order retro change log records

[Nathan Kinder]

I've attached a revised patch.  Noriko caught an issue where I was 
casting the precedence to the wrong type in slapi_pblock_set().  I had 
missed this in testing since I don't actuall call slapi_pblock_set() to 
set the precedence in the server code, but wanted to supply that 
capability to plugins.

On 12/15/2009 12:33 PM, Nathan Kinder wrote:
> --
> 389-devel mailing list
> [e-mail]
> https://www.redhat.com/mailman/listinfo/fedora-directory-devel

[Rich Megginson]

Nathan Kinder wrote:
> I've attached a revised patch.  Noriko caught an issue where I was 
> casting the precedence to the wrong type in slapi_pblock_set().  I had 
> missed this in testing since I don't actuall call slapi_pblock_set() 
> to set the precedence in the server code, but wanted to supply that 
> capability to plugins.

ack
> On 12/15/2009 12:33 PM, Nathan Kinder wrote:
> > --
> > 389-devel mailing list
> > [e-mail]
> > https://www.redhat.com/mailman/listinfo/fedora-directory-devel
>
> --
> 389-devel mailing list
> [e-mail]
> https://www.redhat.com/mailman/listinfo/fedora-directory-devel

[Nathan Kinder]

On 12/15/2009 01:12 PM, Rich Megginson wrote:
> Nathan Kinder wrote:
> > I've attached a revised patch.  Noriko caught an issue where I was 
> > casting the precedence to the wrong type in slapi_pblock_set().  I 
> > had missed this in testing since I don't actuall call 
> > slapi_pblock_set() to set the precedence in the server code, but 
> > wanted to supply that capability to plugins.
>
> ack

Thanks for the reviews!  Pushed to master.

Counting objects: 40, done.
Delta compression using 2 threads.
Compressing objects: 100% (20/20), done.
Writing objects: 100% (21/21), 2.50 KiB, done.
Total 21 (delta 17), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
    24e6ca2..cf0fcc5  master -> master
> > On 12/15/2009 12:33 PM, Nathan Kinder wrote:
> > > -- 
> > > 389-devel mailing list
> > > [e-mail]
> > > https://www.redhat.com/mailman/listinfo/fedora-directory-devel
> >
> > -- 
> > 389-devel mailing list
> > [e-mail]
> > https://www.redhat.com/mailman/listinfo/fedora-directory-devel
>
> -- 
> 389-devel mailing list
> [e-mail]
> https://www.redhat.com/mailman/listinfo/fedora-directory-devel