quote:

---

Originally posted by feline

These ideas make a lot of sense mwb1100, thank you Normally I try to avoid the idea of adding options to VA, but we would need one to use this path compacting code, and it seems reasonable.

I have put in a feature request to see what our developers make of this:

case=16131

I would quite like to see a "good" solution for this implemented myself

---

#include <windows.h>
#include <iostream.h>
#include "tchar.h"
#include "Shlwapi.h"


BOOL PathCompactPathEx2( LPTSTR pszOut, LPCTSTR pszSrc, UINT cchMax, DWORD cProtectDirs);


void main( void )
{
    LPCTSTR pFilename = TEXT( "C:\\\\Documents and Settings\\\\MyName\\\\My Documents\\\\Visual Studio Projects\\\\SolutionName\\\\ProjectName\\\\somefile.cpp"); 
    
    TCHAR outString[MAX_PATH];

    // Variable to get the return from "PathCompactPathEx".
    int retval;

    cout << "The un-truncated path is               " << pFilename << endl;

    cout << "Protecting 2 path components:" << endl;

    retval = PathCompactPathEx2( outString, pFilename, 100, 2);
    cout << "The truncated path at 100 chars is :   " << outString << endl;

    retval = PathCompactPathEx2( outString, pFilename, 80, 2);
    cout << "The truncated path at  80 chars is :   " << outString << endl;

    retval = PathCompactPathEx2( outString, pFilename, 60, 2);
    cout << "The truncated path at  60 chars is :   " << outString << endl;

    retval = PathCompactPathEx2( outString, pFilename, 40, 2);
    cout << "The truncated path at  40 chars is :   " << outString << endl;

    retval = PathCompactPathEx2( outString, pFilename, 20, 2);
    cout << "The truncated path at  20 chars is :   " << outString << endl;
    
    cout << "Protecting 1 path components:" << endl;

    retval = PathCompactPathEx2( outString, pFilename, 100, 1);
    cout << "The truncated path at 100 chars is :   " << outString << endl;

    retval = PathCompactPathEx2( outString, pFilename, 80, 1);
    cout << "The truncated path at  80 chars is :   " << outString << endl;

    retval = PathCompactPathEx2( outString, pFilename, 60, 1);
    cout << "The truncated path at  60 chars is :   " << outString << endl;

    retval = PathCompactPathEx2( outString, pFilename, 40, 1);
    cout << "The truncated path at  40 chars is :   " << outString << endl;

    retval = PathCompactPathEx2( outString, pFilename, 20, 1);
    cout << "The truncated path at  20 chars is :   " << outString << endl;
    
}


BOOL PathCompactPathEx2( LPTSTR pszOut, LPCTSTR pszSrc, UINT cchMax, DWORD cProtectDirs)
{
    if (0 == cProtectDirs) {
        return( PathCompactPathEx( pszOut, pszSrc, cchMax, 0));
    }
    
    size_t pathSeparators = 0;
    
    {
        for (LPCTSTR pSrc = pszSrc; *pSrc; pSrc = CharNext( pSrc)) {
            TCHAR ch = *pSrc;
            
            if ((ch == TEXT('\\\\')) || (ch == TEXT('/'))) {
                ++pathSeparators;
            } 
        }
    }
    
    if (cProtectDirs >= pathSeparators) {
        // not enough path components in the original string to do anything special
        return( PathCompactPathEx( pszOut, pszSrc, cchMax, 0));
    }
    
    pathSeparators = pathSeparators - cProtectDirs;
    
    TCHAR tempName[MAX_PATH];
    
    LPTSTR  pNew = tempName;
    LPCTSTR pSrc = pszSrc;
    
    for (;;) {
        TCHAR ch = *pSrc;
        
        if ((ch == TEXT('\\\\')) || (ch == TEXT('/'))) {
            if (0 == pathSeparators) {
                break;
            }
            --pathSeparators;
        }
        
        // odd way to copy next char from src to dst that deals with multi-byte
        LPCTSTR pNext = CharNext( pSrc);
        while (pSrc != pNext) {
            *pNew++ = *pSrc++;
        } 
    }
    
    *pNew = TEXT( '\\0');
    
    size_t excludedChars = _tcslen( pSrc);
    
    if ((excludedChars >= cchMax) || ((cchMax - excludedChars) <= 8)) {
        // not enough space for the protected portion of the path
        // just let the default implementation do its work
        return( PathCompactPathEx( pszOut, pszSrc, cchMax, 0));
    }
    
    // Pass on the call to Microsoft's implementation, with the 'protected' 
    // part of the path striped off
    BOOL result = PathCompactPathEx( pszOut, tempName, cchMax - excludedChars, 0);
    
    if (!result) {
        // our funky version of PathCompactPathEx() failed - call it using the 
        // unmodified parameters in case someone's going to depend on some odd 
        // side effect that I'm unaware of
        return( PathCompactPathEx( pszOut, pszSrc, cchMax, 0));
    }
    
    // put the protected part of that path back in place inthe output buffer
    _tcscat( pszOut, pSrc);
    
    return( result);
}

quote:

---

Originally posted by feline

One problem with trimming the path is where do we trim it? Any simple and consistent method of trimming / shrinking the path is going to run into "problems", since it will trim the important bit for some common directory structure layout.

---

quote:

---

Originally posted by feline

One problem with trimming the path is where do we trim it? Any simple and consistent method of trimming / shrinking the path is going to run into "problems", since it will trim the important bit for some common directory structure layout.

---

quote:

---

Originally posted by feline

Are you aware that if you hover the mouse over the symbol column in the Find Symbol dialog that the path and line number are shown in a tooltip?

Not quite the same but this should help.

I am slightly concerned about fitting in the file path (you need the full path to handle duplicate file names) as another column, since the text in both the symbol and definition columns already tends to get rather long.

---