Thread: [cedet-semantic] semantic-ectags-version Regex problem in version 8621
Brought to you by:
zappo
From: Qian Wu <yer...@gm...> - 2013-12-13 15:25:00
|
hi, :) maybe i found a problem in module ectags. source file path is `lisp/cedet/semantic/ectags/utils.el` in function `semantic-ectags-version` ... (if (re-search-forward "Exuberant Ctags *\\(\\([0-9.]+\\)\\(~svn[0-9]+\\)?\\|Development\\)*," nil t) (match-string *1*) nil) ... and since the regex is changed, *the params in match-string should be 2 instead of 1.* otherwise the function will return 5.9~svnxxxxx instead of 5.9 which will make the other functions error. |
From: David E. <de...@ra...> - 2013-12-17 21:04:18
|
Qian Wu writes: > hi, :) > maybe i found a problem in module ectags. > source file path is `lisp/cedet/semantic/ectags/utils.el` > > in function `semantic-ectags-version` > ... > (if (re-search-forward "Exuberant Ctags \\(\\([0-9.]+\\)\\(~svn[0-9]+\\)?\\| > Development\\)," nil t) > (match-string 1) > nil) > ... > and since the regex is changed, the params in match-string should be 2 instead > of 1. Thank you for reporting this. I committed a similar patch which should fix this. -David |