[Args. Why does ctrl-enter(?) send the message :/] On Mon, Sep 1, 2014 at 9:45 PM, Stefan Fuhrmann < stefan.fuhrm...@wandisco.com> wrote:
> On Mon, Sep 1, 2014 at 10:18 AM, Branko Čibej <br...@wandisco.com> wrote: > >> On 30.08.2014 22:42, Branko Čibej wrote: >> > On 30.08.2014 21:37, Stefan Fuhrmann wrote: >> >> I updated the wiki page now to make it match the code again, updated >> >> terminology to make it match your API and (hopefully) addressed the >> >> comments to had on the page. I left the comments in for you to review. >> > OK, thanks! I'll try to get through the docs on Monday, then I'll have >> > to fix the wildcard normalization logic in the parser to match the >> > updated docs. >> >> Stefan, I went through the docs (and commits); there are a couple new >> comments in the doc, one is a minor nit, the other affects the rights >> lookup workflow. > > > We already talked about that shortcut. > I'll give it a go tomorrow. > > >> IIUC, you already removed the foo/** to foo/* >> normalization from the code? >> > > Yes, at least that was the intent. I might have missed or broken > something, though. > Parser states can be non-obvious at times. > >> I propose to remove the svn_authz_t::groups hash now, as I believe it is >> not used anywhere any more. This will slightly reduce the memory >> footprint of the parsed authz representation. >> > Go ahead. -- Stefan^2.
