nginx-0.7.8

[Denis F. Latypoff]

Hello Igor,

Tuesday, August 5, 2008, 9:29:28 PM, you wrote:

> On Tue, Aug 05, 2008 at 09:00:29PM +0700, Denis F. Latypoff wrote:

>> Hello Igor,
>> 
>> Tuesday, August 5, 2008, 8:23:03 PM, you wrote:
>> 
>> > On Tue, Aug 05, 2008 at 05:19:20PM +0400, Igor Sysoev wrote:
>> 
>> >> > > То есть, нужно искать по импортам до первого ненулевого mediaType ?
>> >> > > Ну, и естественно, сначала проверять mediaType в самом xsl.
>> >> > 
>> >> > Да. И encoding тоже.
>> >> 
>> >> OK.
>> 
>> > Патч. Те же и encoding.
>> 
>> 1) s/ngx_log_debug1/ngx_log_debug3/ ngx_http_xslt_filter_module.c:780
>> 2) на indent можно забить: "XSLT-процессор не обязан выполнять это
>>                             требование, и в этом случае трудно
>>                             предугадать точную форму вывода"
>> 
>> Все в порядке, вроде потестил с многоэтажными import'ами, все как по
>> спеке.

> Новый патч. Включает всё, что было, + единственные копии XSLT-деревьев.

Патч срежектил пару строк (в кое каких местах табуляция
протиснулась), плюс пара полей у структур не синхронизировна с кодом
(или наоборот), я интуитивно поправил, все в порядке, для каждого
location (около 20) прописал одну и ту же таблицу стилей, все
в порядке.

патч к вашему патчу:

@@ -26,8 +26,8 @@
      xmlDtdPtr            dtd;
 -    ngx_array_t          sheets;        /* ngx_http_xslt_sheet_t */
 +    ngx_array_t          sheets;       /* ngx_http_xslt_sheet_t */
-     ngx_hash_t           types;
-     ngx_array_t         *types_keys;
+     ngx_hash_t           types_hash;
+     ngx_array_t         *keys;
 -} ngx_http_xslt_filter_conf_t;
 +} ngx_http_xslt_filter_loc_conf_t;

@@ -85,8 +85,8 @@
 +      NGX_HTTP_MAIN_CONF|NGX_HTTP_SRV_CONF|NGX_HTTP_LOC_CONF|NGX_CONF_1MORE,
        ngx_http_types_slot,
        NGX_HTTP_LOC_CONF_OFFSET,
--      offsetof(ngx_http_xslt_filter_conf_t, types_keys),
-+      offsetof(ngx_http_xslt_filter_loc_conf_t, types_keys),
+-      offsetof(ngx_http_xslt_filter_conf_t, keys),
++      offsetof(ngx_http_xslt_filter_loc_conf_t, keys),
        &ngx_http_xslt_default_types[0] },

        ngx_null_command


-- 
Best regards,
 Denis                            mailto:denis at gostats.ru