So, the question is whether to change the documentation or to change the code. The particular problem here is that the test should have been strlen(id) != 1 as opposed to '2'. The comment was a hold over after the following attempted fix:
11 November 2020
Correct documentation on how to specify the idtype (i.e. the single letter is not followed by a colon).
So, to keep things sane. The code will change to secretly allow two characters but only consider the first. That makes it backward compatible without needing the change the documentation as it was already changed but the code was not properly changed. Must have been a pandemic driven change, sigh.
--
Reply to this email directly or view it on GitHub:
https://github.com/xrootd/xrootd/issues/2061#issuecomment-1681673342
You are receiving this because you are subscribed to this thread.
Message ID: <[log in to unmask]>
########################################################################
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1
