Le mardi 23 mars 2010 17:36:57, Romain Beauxis a écrit :
> >  - Forcing temporary to false in annotate is useless, and meaningless
> > if the argument URI is not a file.
> 
> I do not understand very well this parameter, it seems it describe wether
>  the  request can be kept or should be revoled each time one need it. Can
>  you describe a bit more ?

Sorry, I just understood this.

On the contrary, this is highly relevant with the modifications.

annotate is registered as a post_processor, I believe it is more relevant (in 
fact we may eventually remove the .ml and put it in utils.liq)

As such, the post-processors have a strict specifications about the indicators 
they push: it should be temporary = true iff they create a new file, and in 
this 
case *always* create a new file.

Indeed, the post-processors are executed with the local file resolved by the 
request. The temporary nature of this file is already documented in its own 
indicators. When the post_processor returns a new request, the code just push 
the new indicators merging the metadata with the old metadata.

That way, the issue of cleaning temporary files is kept local to each 
indicator.

Hence, annotate will operate on a uri which is now the local file. I never 
create a new file, so the indicators it return should force temporary to 
false..

This is documented in request.mli. I will also document this in the exported 
operators before commiting.


Romain

------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Savonet-devl mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/savonet-devl

Répondre à