[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [augeas-devel] [PATCH] Add support for an 'i' flag in regexp builtin function



On Thu, Aug 2, 2012 at 6:10 PM, Raphaël Pinson <raphael pinson camptocamp com> wrote:


On Wed, Aug 1, 2012 at 11:53 PM, Raphaël Pinson <raphael pinson camptocamp com> wrote:
This is a new version of the 3rd patch.

As suggested, I added a test against nodesets (test regexp6),
but it fails for a reason I cannot comprehend. It seems that
when two arguments are passed, the nodeset is not properly
parsed, resulting in an empty nodeset, even though the path is
strictly the same as test regexp3 which doesn't fail.



In fact, it seems pop_value() fails when a second argument is provided and the first argument is a nodeset. It just doesn't return anything for any of the two arguments.


To be more specific, nargs is set to 1 in func_regexp when args are T_NODESET and T_STRING, but set to 2 when args are T_STRING and T_STRING.
 


--
Raphaël Pinson
Administrateur Systèmes & Réseaux
Camptocamp France
Savoie Technolac
BP 352
48, avenue du Lac du Bourget
73372 Le Bourget du Lac, Cedex
www.camptocamp.com


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]