|
|
Boost-Commit : |
Subject: [Boost-commit] svn:boost r59811 - sandbox/filesystem-v3/libs/filesystem/doc
From: bdawes_at_[hidden]
Date: 2010-02-21 11:09:37
Author: bemandawes
Date: 2010-02-21 11:09:36 EST (Sun, 21 Feb 2010)
New Revision: 59811
URL: http://svn.boost.org/trac/boost/changeset/59811
Log:
WIP beta 1 comments
Text files modified:
sandbox/filesystem-v3/libs/filesystem/doc/do_list.html | 43 ++++++++++++++++++++++++++++++++++++++-
sandbox/filesystem-v3/libs/filesystem/doc/reference.html | 7 ++++-
2 files changed, 46 insertions(+), 4 deletions(-)
Modified: sandbox/filesystem-v3/libs/filesystem/doc/do_list.html
==============================================================================
--- sandbox/filesystem-v3/libs/filesystem/doc/do_list.html (original)
+++ sandbox/filesystem-v3/libs/filesystem/doc/do_list.html 2010-02-21 11:09:36 EST (Sun, 21 Feb 2010)
@@ -16,9 +16,48 @@
<body>
<h1>Boost Filesystem Do List<br>
-18 February 2010</h1>
+<!--webbot bot="Timestamp" S-Type="EDITED" S-Format="%d %B %Y" startspan -->20 February 2010<!--webbot bot="Timestamp" endspan i-checksum="40539" --></h1>
-<h2>Docs</h2>
+<h2>Beta 1 comments</h2>
+<ul>
+ <li>Add ack for commenters.</li>
+ <li>Rename absolute. make_absolute?</li>
+ <li>Rename unique_path -> generate_unique_path.</li>
+ <li>Should "change_extension" be "replace_extensioin"? What did V2 call it?</li>
+ <li dir="ltr">
+
+<p dir="ltr">Zach Laine:</li>
+</ul>
+<blockquote>
+ <pre dir="ltr">The descriptions for portable_name() and portable_directory_name()
+appear to be at odds.
+
+portable_name() : ... && (name is "." or "..", and the first character
+not a period or hyphen)
+
+portable_directory_name(): ... && (name is "." or ".." or contains no periods)
+
+Should portable_name() be "... && (name is "." or "..", or contains no
+periods) && (first character not a hyphen)"? Maybe I'm missing
+something?</pre>
+</blockquote>
+<ul>
+ <li dir="ltr">
+ <p dir="ltr">Zach Laine:</li>
+</ul>
+<blockquote>
+ <pre>Also, in the tut3.cpp tutorial code, there's no mention of the dangers
+of using "*it" inside the directory_iterator loop. IIRC, in previous
+Filesystem revisions, this would throw if you did not have adequate
+permissions to see the file to which "it" refers. Has this changed?
+If not, I think that you should either wrap the body of the loop in a
+try-catch block, or use one of the non-throwing function overloads,
+and indicate to the user what the exception safety concerns are. I
+think the tutorial code should exhibit enough robustness that the user
+can just cut-and-paste to get the basic code needed to use the
+library.</pre>
+</blockquote>
+<h2 dir="ltr">Docs</h2>
<ul>
<li>Reorganize files</li>
<li>Create Jamfile for examples</li>
Modified: sandbox/filesystem-v3/libs/filesystem/doc/reference.html
==============================================================================
--- sandbox/filesystem-v3/libs/filesystem/doc/reference.html (original)
+++ sandbox/filesystem-v3/libs/filesystem/doc/reference.html 2010-02-21 11:09:36 EST (Sun, 21 Feb 2010)
@@ -969,7 +969,10 @@
and ending at the path's end. Otherwise, returns an empty <code>path</code>
object. </p>
<p>[<i>Note:<b> </b></i>The dot is included in the return value so that
- it is possible to distinguish between no extension and an empty extension. <i>-- end note</i>]</p>
+ it is possible to distinguish between no extension and an empty extension. See
+ <a href="http://permalink.gmane.org/gmane.comp.lib.boost.devel/199744">
+ http://permalink.gmane.org/gmane.comp.lib.boost.devel/199744> for more
+ extensive rationale. <i>-- end note</i>]</p>
<p>Implementations are permitted but not required to define additional
behavior for file systems which append additional elements to extensions, such
as alternate data streams or partitioned dataset names.</p>
@@ -3096,7 +3099,7 @@
<p>Distributed under the Boost Software License, Version 1.0. See
<a href="http://www.boost.org/LICENSE_1_0.txt">www.boost.org/LICENSE_1_0.txt</a></p>
<p>Revised
-<!--webbot bot="Timestamp" S-Type="EDITED" S-Format="%d %B %Y" startspan -->18 February 2010<!--webbot bot="Timestamp" endspan i-checksum="40554" --></p>
+<!--webbot bot="Timestamp" S-Type="EDITED" S-Format="%d %B %Y" startspan -->20 February 2010<!--webbot bot="Timestamp" endspan i-checksum="40539" --></p>
</body>
Boost-Commit list run by bdawes at acm.org, david.abrahams at rcn.com, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk