Hi Nicolas, 2014ko irailak 19an, Nicolas Goaziou-ek idatzi zuen:
[...] > > In any case, I think you can apply the patch once the docstrings > questions above have been sorted out. Thanks for all your feedback. The patch is now applied on master. > This can do for now. Ultimately, however, I think we could merge > `org-latex--wrap-label' into `org-latex--caption/label-string' (which > should then produce "\captionof" command when appropriate). By then, the > `replace-regexp-in-string' would not be needed anymore. WDYT? I’m confused about how ‘org-latex--wrap-label’ works. It tries to put \label commands outside of floats. That will yield links that jump to the preceding section, not the specific point in the document where the \label command occurs. Probably this command should insert \phantomsection before the \label, so that resultant links jump to the right point. (I’ll make this change if no one objects.) Merging the functions might only be worth doing if there were any more places where \captionof needed to be used. As long as it’s just a particular kind of source block that needs it, I think this can stay as a little special case in the source block transcoder. The docstrings seem to spell out the criteria for using one function vs. the other well, so there’s little danger of confusion. Adding \phantomsection to ‘org-latex--wrap-label’ would also make them diverge further in their internal workings. Thanks, -- Aaron Ecay
