labofsahil/github-event-dataset-2012 huggingface dataset:
case-sensitive, I'm guessing. I tried "Etag" and "ETag" without success,
but not quite sure what to expect. I eventually called
ne_response_header_iterate just to see if anything etag-ish was even
available, and it indicated "etag" all miniscule.


On Wed, Dec 19, 2012 at 6:36 PM, David Strauss <notifications@github.com>wrote:

> In src/ldb-filecache.c:
>
> > -
> > -int ldb_filecache_release(ldb_filecache_t *cache, const char *path, struct fuse_file_info *info) {
> > -    struct ldb_filecache_sdata *sdata = (struct ldb_filecache_sdata *)info->fh;
> > -    int ret = -1;
> > -
> > -    assert(sdata);
> > -
> > -    log_print(LOG_DEBUG, "release(%s)", path);
> > -
> > -    if ((ret = ldb_filecache_sync(cache, path, info)) < 0) {
> > -        log_print(LOG_ERR, "ldb_filecache_unref: ldb_filecache_sync returns error %d", ret);
> > -        goto finish;
> > +    // We continue to PUT the file if etag happens to be NULL; it just
> > +    // means ultimately that it won't go into the filecache
> > +    if (etag) {
> > +        value = ne_get_response_header(req, "etag");
>
> Is this match case-insensitive (HTTP spec says it should be, but it often
> isn't)?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/pantheon-systems/fusedav/pull/4/files#r2473523>.
>
>
Signals are often a bit cryptic - they can be hard to debug.  However they work quite well in libraries like oscar as they decouple different components.  I generally try to use them sparingly but this seemed like a good fit.
I agree we first need to finalize what are general webinos core components. These components should be able to provide support to different API's. For example, using webinos core, we should be able to extract path for NFC.
Can you remind me again why we can't use a class here?
r- on this
This is exactly what `self.assertEqual` does. 
Also, "assert_expected" is an ambiguous name as it's not obvious what it does from the function name.
Fixed in 3aed07a
Ar fi mai bine sa nu facem operatii inutile in plus pe imagine. http://stackoverflow.com/questions/9643228/test-if-file-is-an-image e o solutie care nu face modificari pe imagine care ocupa resurse ci doar se uita in header.
From my understanding, stores are for when you explicitly DON'T want a call back.  If you do you will use a Pipe.
Ты, кстати, можешь использовать ~~паттерн модуль~~IEFE и избавиться от кучи `"use strict"`;

```javascript
(function (exports) {
    "use strict";

    var Events = function (data) {
        Collection.apply(this, arguments);
    };

    inherits(Events, Collection);

    // your stuff

    exports.Events = Events;
}(window));
```
Используй для получения Integer статический метод Integer.valueOf
Там множество строк кода, а исключение может быть выброшено в любой момент
Having a ten second timeout for running the two local tests was pretty useful.
We should escape this data either in the route or in the template. Since we're using `<%-` in the template, let's do it in the router to mitigate a XSS.
this doesn't appear to be used so get rid of it.
That's strange. Could that have something to do with the fact the session needs to be purged?
Hooray! :rainbow:
```Bio-Formats'``` ?
Ok now func8 and func16 are called via 
    
    def _apply(func8, func16, image, selem, out, mask, shift_x, shift_y):
    ...
Good catch. BTW, I don't understand what "OvR, not softmax" means since liblinear doesn't do multinomial logistic regression...
what if len(result) > minlength? You'll get a exception, won't you?
Is there a reason why this is not True by default?
allow to pass the residual
ups, sorry then. that was what i would have wanted.
Yikes, not sure what I was on when I wrote that.
#include <stdbool.h>
inclusion removed
false candidateS
Sure. My comments is more about the fact that there is no single annotation. Maybe we could get the _DocumentBuilder_ for _InvertedIdOrdererEntity_ and assert it is not null.
These were added due to an issue with Debian Squeeze and Ubuntu 11.04/11.10. 

An upstream Debian issue caused pam settings to not correctly include pam_limits.so. This made it so the only time this module was loaded was on an ssh session but not on usage of sudo or su.

This has been fixed in the upstream Debian repo and propagated into later Ubuntu versions with a patched version in 11.10.

No patch update was ported to Ubuntu 11.04. I would suggest removing these ulimit lines completely and deprecating the building of Ubuntu 11.04 as it is EoL on 10/28/12.
In fact, our routes to save/load the xml make no guarantee of order of elements either, it just so happens that siblings of the same name are stored internally in an array vs hash table, so they will be written and loaded, in the order they were added.

Everywhere else seems to use a 'getSubPath' to access, so I think it would safer for now to assume that they might not be in order.
This should still be done in the class constructor/destructor
'rows' doesn't change from call to call, I think it would be better to set up once before getNextSorted as a CHeapSorter member.
Shouldn't this condition be cur < end?
You're right as you said in #32. I haven't gotten around to fix this yet. But if you want you can do it too.
I think this should be:

```
<title>{% block page_title %}Crash Data for {{ product }}{% endblock %}</title>
```

otherwise, every time you need to override this you have to write the `<title>` tag which you'll never need to do anything else interesting with.
Get rid of the blank line please.
For the EULA middleware we are more explicit about what we match:

    # ignore URLs we declare exempt from this check
    path = request.path_info.lstrip('/')
    if any(url.match(path) for url in self.EXEMPT_URLS):
       return None
@tyx please don't forget this as well. The attributions discussion is diverting attention :)
I'd give this an extra newline (between the } and the """), so the generated file can end with a newline. git (and diff) usually mumble complaints about files that don't have a trailing newline.
This is probably the only interesting change in this pull request :-P
Just re-read my own comment. Please ignore it! :)
getCacheFiles is already in use.
@shacker can we remove the `px` here?
`first` returns `nil` for empty collections
no way. I'm not allowing the use of QRegExp. (http://blog.rburchell.com/2011/12/why-i-avoid-qregexp-in-qt-4-and-so.html)

I also don't want wildcard searching here. Something similar to the N9's filtering (split by words, match at the start of the word) would be much better
array.splice(index--, 1);
I do agree. only way to filter would be to read GPOs, and resolve which computers applies a specific GPO policy
Yes, it appears temperamental, I have been updating to correct for this.

e.g. Previously used username against groups.xml and always got results, but trying it against some of the different xml files it did not return unless 'userName' was specified.
Should check for existence of railgun before attempting to use it.
A description would be absolutely helpful!
88118 : Oracle MySQL Server FILE Privilege Database Privilege Escalation
Is `tform` necessary? Couldn't a user pass in `tform.inv` to `reverse_map` to accomplish the same thing?
You don't need to set this variable.
No extra space here please.
Holy hell I didn't even know you could use ``[...]`` on the LHS with tuple unpacking.

At any rate, that indeed raises a ValueError if there's not exactly 1 item.  Clever.
Move patching to testing layer, if possible, for security reasons.
The doc for the point parameter is missing.
This member isn't needed.  It can be computed from the root of the polygon hierarchy and passed to `_appendTextureCoordinates`.  In general, less member variables is better.  The more members, the harder a class is to understand.
There shouldn't be a need for insertAfter/insertBefore. Rewrite whatever uses this to use insertBefore (or alternatively rewrite everything to use insertAfter). These if statements just waste space and performance
It would be more idiomatic to say 

class_platforms = []

Since it's an array later. An empty array is still falsey.
Get rid of the curlies, and make this a one liner.
Should set the encoding... https://maven.apache.org/general.html#encoding-warning
shouldn't this be fixed by at least fixing the constant instead of removing it ?
TODO: Refactor this
This was pre-existing code, but technically, if you are going to walk through RoR, then you can extract its length.

This is extra work that is outside of the scope of this pull, but I wouldn't mind you putting a "todo/note" on this line. For future reference.
`inout`-qualified module level function should not be accepted. I think it is a compiler bug.
opCall in code font
What are these additional tests testing?
This is an array, right? Rather ``if f == "-":`` ?
Again, please follow the coding style. Put the space before the colon, not after it.
True, afaik you mentioned some enforcement in an earlier PR, is there any progress or timeline on this?
That's a test, probably needs deletion :)
Another thought — instead of having an option to enable regex redirects, why don't we just be smart and apply regex instead of pattern matching if the redirect_from includes # or @ or some other standard delimiter? Seems like it would be simpler.
Can you add a comment  pointing to where this file comes from? I'm assuming Ant's source code, but not sure...
Use the deprecated directive   ".. deprecated:: 1.2.0"
http://matplotlib.sourceforge.net/devel/documenting_mpl.html#formatting
:hand:

I can think of a case where you have a large number of expressions - shouldn't there be some wrapping allowed? 

On the other hand, these could be refactored (see `Fowler, Martin. 1999. Refactoring - Improving the Design of Existing Code. Addison Wesley`).
Instead of having separate images for each hover, can we combine these in a sprite and use the background position?
put the description in `get_or_create` defaults and avoid the extra save
Was being lazy (no great need really), but now I have time and you've pointed it out.
It seems like an abuse to require the use of the Jersey string `ClientConfig.PROPERTY_CONNECT_TIMEOUT` as-is in our config. It also will make it harder for customers to reason about its behavior; do we pass it directly to the Jersey config story, or reparse it?

It also entangles our layer more with the Jersey layer in a customer-facing way.

I think it would be clearer to have our own timeout value, which we map internally to the Jersey junk.
Would be good to convert these to alphanumeric at some point.
copy-pasted it from passenger-rails vhost file :)
This ... is very error prone. :-)

Maybe use jQuery queues?
should be:

    if ('ARRAY' !== $tableMap->getColumn($name)->getType()) {
Well storageType="YES" allows testing of multilist values on Demo page, so up to you as to whether it's included. Safe to exclude.
I think we need an interface here ..
Cool, this answers my first comment :P
I hate PayPal :(

I guess unless there's another way to override those things, then it probably does make sense to have this as a separate class.
but does easyXDM provide the same ?
Should hidden fields really be reset?
cuz _1 is first, duh ;P
Do we need to enforce profile/ on these URLs?
If the "purchase" method is successful, we then need to listen for event. and after that we can call "success" callback.
I will pass fail(-1, e);
Clicking login opens the Browser ID popup, so I doubt you'll need to have this method for now.
what does this get us? If they use django which they probably will since this is a django job and we've gone as far as to tell them to write a webapp now this is basically free
Why the .update calls instead of just making a dictionary literal in the first place?  It seems like a literal would be easier to parse for both humans and machines.
userInitials (use lower camel casing)
tn and t could be the same. Maybe we could find a slightly better error message.
Is it worth adding that ass a comment into the code?
What's management group?
You can short circuit a return at this point if controller is nil
I saw some .java files have lines between imports and some are not. Anyway, i will delete the lines.
Firstly, you want to check for null.

```
HttpEntity entity = response.getEntity();
if (entity == null) {
  // No cleanup needed.
  return;
}
```

Secondly, you might be getting a `RuntimeException` here. Add a clause:

```
} catch (RuntimeException e) {
  Logger.warn(LOG_TAG, "Unexpected exception.", e);
  // TODO: get a handle to the HttpGet itself, and call .abort() on it.
}
```
Zut alors ?
Hold of on the formatting for this particular issue. Open a new jira
and do the whole code base in one.
`if (snUsShare == '' && !$('.sn-us-smilies').hasClass('sn-mouseenter')) {`
type should actually be {}. When the parser parses an empty element it sets it's value to {} (see below console.log output).

    [ { '@': { id: 'net.rim.webworks.invoke.appcustomaction' },
        type: {},
        filter: { action: 'bb.action.CUSTOM', 'mime-type': 'text/*' } } ]
It needs to have local context so you can use local:/// in image icons for Context menus

If you load a url into the webview with platform:// it loses the ability acess local:///
~Nukul

From: Jeffrey Heifetz [mailto:notifications@github.com]
Sent: Tuesday, October 30, 2012 12:22 PM
To: blackberry-webworks/BB10-WebWorks-Framework <BB10-WebWorks-Framework@noreply.github.com>
Cc: Nukul Bhasin
Subject: Re: [BB10-WebWorks-Framework] Updating WP to use on device version, removing the copying of build (#234)


In dependencies/bootstrap/ui.html:

> +        <div id="contextMenuModal" style="display:none;"></div>
> +<div id="contextMenu" class="hideMenu">
> +    <div id="contextMenuHeader">
> +        <div id="contextMenuHeadText"></div>
> +        <div id="contextMenuSubheadText"></div>
> +    </div>
> +    <div id="contextMenuContent"></div>
> +    <div id="contextMenuHandle"></div>
> +    <div id="contextMenuDelete"></div>
> +</div>
> +
> +        <div id="dialog" class="hidden">
> +    <div class="dialog-top-third">
> +        <div id="dialog-panel">
> +        </div>
> +    </div>


Why isn't the ui.html ondevice?

—
Reply to this email directly or view it on GitHub<https://github.com/blackberry-webworks/BB10-WebWorks-Framework/pull/234/files#r1980736>.



---------------------------------------------------------------------
This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.
I didn't think so which is why blackberry.event was being pulled in like that :( I looked at window.webworks.event but Im not sure if it does the same thing

```javascript
    add: function (featureId, name, cb, success, fail) {
        _add(featureId, name, cb, success, fail, false);
    },
```

Maybe Im looking at the wrong code?
If you want to make sure a spy has been called once, you should use expect(spy).toHaveBeenCalled(). Unless you want to check for a specific number of calls, you shouldn't be writing the check like this, it's very difficult to read.
or it could be messed up some more......
You can also use `(- n)` instead of `(- 0 n)`