labofsahil/github-event-dataset-2012 huggingface dataset:
should this be "nonRootUserShouldNotBeAbleToModifyOwnSettingsWhenNotMemberOfADomain"? (drop "Domain"), or should it be UserSettings instead of DomainSettings?

It seems to convey the impression that it's their own domain but they're not a member of it.
this file is a duplicate and it should be deleted.
`? true : false` is unnecessary.
Maybe "which have not been cleared yet" if you're going to be making more changes?
"Can be used to perform actions when the user/group colour/avatar/rank is changing"

I fail to see the connection between event name and this description.
spacing around ()
which.
removed in 8c3296e -- we use the version.py now everywhere.
let's include X out of Y attempts.
then remove it?
Damn this is messy code :)
I would prefer to drop the "lib" prefix convention personally. It essentially identifies things originating from a `require`, and I don't see much value in that on node.js.
@mridgway the main reason for refactoring this is because we could have a dummy before_install script, and a custom one for our bleeding branches (yaf-develop for example) to use the bleeding YUI version (branch 3.x). Otherwise we will have to have a different .travis.yml file in those branches.
This RS addon is now the official way to read JSON files, at least for other parts of the RS (including its addons).
That's an interesting idea, @aszlig do you have any sense of if the main binary is kernel version dependent?
0<=Age<120
I don't believe you need 'extends BaseOpenmrsService' here
System.out for the dao doesn't seem useful... 
Only important logging information should be System.out or logged with correct level (INFO, ERROR... etc).
The NERC PUMA dictionary does not currently provide coverage for these STASH codes. Hence the need for this dictionary.
I've been in discussion with @marqh regarding this issue, and I assume that his work will deprecate this dictionary once it is folded into Iris.
Not a fan of the bracket followed by indentation approach.
Why " '%s' % name " vs. simply " name "?
`.gitignore` を gitignore する理由はなんだろう？
Spurious else clause after return.
I like your approach. I think though that my approach may be a bit more readable but your approach is definitely cleaner
link to "Using different testing frameworks" in topics/testing/advanced?
Shouldn't we be using `modifyMVar_`?
Space here.
Yes, I'm very interested to find out environment + circumstances that cause this to fail, because I thought it was really solid.
It reduces a small amount of memory.  Not a big deal.  But I see this pattern in a lot of other software.
What is the column "jump" here for?  Isn't it related to jump in tree_table?

By the way, where is the result table's comments/descriptions?  I don't find it even in the madlib document.
Love it! I can reign in this (and possibly other bits) based on this.
omg, you're right about the first entry.

And thank you for recommendation for `url = ARGV[1]…`.

I'll update the patch soon.

This may be late, Sorry for I can't take time for this right now.
hmmm :/ 
I'll have a look tomorrow. If so and the submodules are helpful (I never used them and we had the tarbar up to version 3.5) we better stick with the git download.
For X-code only systems, we need at least the `CFLAGS`. Perhaps the `LDFLAGS`, too if linking is done.
Cool, I found nothing about this in the Formula Cookbook. Probably worth to add a note about these variable replacements.
Why are we passing the prefix as DESTDIR here, and one line above specifying an empty prefix?
Are the provided autotools with Xcode < 4.3 sufficient? If so, this should be conditional:

```rb
if MacOS.xcode_version.to_f > 4.3 and ARGV.build_head?
  depends_on 'automake' => :build
  depends_on 'libtool' => :build
end
```

(Depending on automake pulls in autoconf as well).
Pick a consistent place to put suppress_wizard - this is not in the same spot as for server.rb
>The problem is that linker is very stateful. If you add additional linkers, it's unclear where that state goes. If you want to move behavior out of linker, that's okay.
Yeah - I see it... As I said, I wasn't sure about this one, and was more inclined to move out the functionality.  I'm still wrapping my head around a few of the flows.  I definitely see what you mean here.
Yeah, I think that's a bit more elegant. As-is we're baking in more assumptions than we should.
As above, the suffix includes a $
Do you have a test for this case?  Good catch, but I'd love to see how it would happen (and therefore, be prevented from happening by virtue of the test)
All my javadocs may need work.
Why make these local vars?
`Redirects`? not `Redirect`?
should be \OutOfBoundsException
It would be nice to factor this common brackets namespace initialization code out of brackets.js and SpecRunner.js into a new module. Is that feasible?
But the items don't have ID's?
Sorry it is nitpicking, but what do you mean by "el-get" here is bit unclear.  I think it should be quoted so that reader can know that it is a function name, not the name of this entire module.  Like this:

    Used by `el-get'.

Same goes for `el-get-flatten`
> For other.version, can you double check that Version.new(Version.new("1.2.3")) returns the same instance.

Cool, it does.

https://github.com/josh/useragent/blob/master/lib/user_agent/version.rb#L7-L8
Seems not. :-)
Err.. Should I use `default_error_messages` or `error_messages` or are these two somehow related?
add space after =
Related to my above comment, why not just do:

options._container.style.left = options.left + "px";
Nvm. You're right. It's too tiny to matter.
You might want to sanitize or escape this text, since you're putting it directly into HTML.
This doesn't look like it's actually used for anything. Is it?
Might be cool to have some kind of visual divider between the two columns
Well, I'm not familiar with JS, not familiar with pdf.js code base and have a lack of time, so it is very up to you how to split interfaces. But I think that:

1) If annotation is visible, it showl'd be displayed.
2) Verification of digital signatures relies on regional standards and laws, so developers must have access to the following things: pdf blob, pages enumeration, annotations enumeration, annotation dictionaries and streams. 
3) There's no need in separate annotationOperatorList.
sure, that name would be fine too.
I think this is not so bad, it's because PEP8 said that limit all lines to a maximum of 79 characters. 
http://www.python.org/dev/peps/pep-0008/#maximum-line-length
But, if you don't like above, i'll modify it like this.

```Python
    def update_line(self, current):
        num_bar = int(current / 100.0 * (self._term_width - 5))
        bars = u'#' * num_bar
        spaces = u' ' * (self._term_width - 5 - num_bar)
        percentage = u'%3d' % int(current) + u'%\r'
        return bars + spaces + u' ' + percentage
```
4 lines normal comment only //
错误信息改成: invalid argument of \"%V\", &value[i]

在错误日志中，也尽量把错误的配置打印出来或者把出错的信息描述清楚，方便配置的时候调试。
That linter doesn't check style, just sharp corners. I have my vimrc configured to format things according to the twitter style, this is typically an editor function not necessarily a linter function.
please add a comment and commit-msg "why"
I can see the confusion with my previous comment as I do mean an immediate function definition, or a self-executing function definition, let me clarify:

`KT.domain_page = (function(){
       //all the bits
}());
Yes - this is the core method responsible for creating actual hub instances. If null is returned from here a null target is being passed to the method invoker - as one should expect.
Note: This is equivalent to:

```
# pgbackup
```

Other hash marks are unnecessary.
Ok, sounds good.  Or, @iguananaut, if you prefer, we can merge this when its ready and you can PR against master (either way is fine with me)
@ADmad i updated the name in the next commit.
Private functions require `__`.  We generally try to avoid private methods in favour of protected ones.  Lastly this method is missing a doc block.
It may be worth saying that setting this is optional.
When I try the example I don't know why but sometimes I pass in this code with the azimuth tool, and because the metric is undefined it crash and the map become unusable :(
"Create" might be more appropriate, as used in the French translation.
Why are you creating a class that does nothing? Simply so it evaluates as an object instead of an integer?
Cast?
sampledict = { 512 : self._on_message }

try:
sampledict[item[0]](item[1])
except KeyError, e:
log.error("unimplemented method: %s" % e)
This function shouldn't be defined anew every time this is called.  Just make it a method.
Sounds good, but I don't know what the t and f arguments would need to be for a URL...  Do you know?  I just don't feel like spending more time writing Windows-specific API code :)
I think this should be `IPython.utils.py3compat.input()`.
this should just be a call to hasMetadata (name correction to follow)
I think this would be more concise as .filter(expectedStatus).transform(toId()) equals transform(toId) or something
Maybe you can mark those members readonly in their jsdoc comments at least, this way people know not to play with them.

Generally I think _ prefixes are fine for emphasizing that a member is private, especially since with a JS library like this there is otherwise no obvious public interface.

If possible I'd like to see the private functions grouped, preferably at the end of the class (so its easier to ignore them when reading the source code).
It seems that this could come fairly late in the checkout process (always in the second stage? possibly as the very last item of the second stage?) Is it possible that progress could jump back quite a bit? (say, from ~100% to ~66% in the worst case...)?
Do you think, in the spirit of keeping things organized, we could create another file to manage all the activity related bot functions? I just don't want to see this bot file get loaded up with tons of hooks over time.
Because it's not executable js on its own, it's a place to macro in configuration variables. It's as much a "konf" file as base_konf.konf is.

I was initially abusing default.konf for this, but that's the wrong place for this to live, as all you can do there is add to defaults on the Mobify.conf.data object for your adaptation, whereas this file holds compile-time configuration parameters used by functions in the mobifyjs API themselves.

config.js does run-time configuration.
A few lines down from here is a "product" field that also needs to get changed. It currently pulls its choices from a hard-coded apps/wiki/config.py, but it should pull it from the products_product table.
Mmm... I'm not going to remember this--can you add a TODO comment for this?
I just checked the file and it looks like these two lines have tabs in them.
you can probably do the same trick here to get the transaction behavior:

```
def run(self, date):
    database = self.config.database_class(self.config)
    executor = self.config.transaction_executor_class(self.config, database)
    exeuctor(self._do_my_postgres_work, date)
```
This will cause PHP warnings as it doesnt exist. You can use `->build('admin/login', compact(array('redirect')));` as a shortcut if you like, or `->set('redirect', $redirect)` is potentially nicer.
This should be in same line as `foreach`
shouldn't you make this service private ?
I can see using the variables handy. Means whenever we reference the paths, we could just use ```$this->commandPath``` rather than ```'/Command';``` . Essentially avoiding [magic number](https://en.wikipedia.org/wiki/Magic_number_(programming)). But this was not in the original class, so it might be best to leave it out.
これscaffoldでできたコードなんですねぇ…
失礼しました…
Does this actually work? I'd expect the (ns) to mess with project.clj.
This seems to not actually implement the length-coded string. What am I missing?
changed to .initialize_with_name_and_email
Right, but I think we should stick to double quotes in HTML.
in the json_payload method below line 53, 54 I think you're going to get make two calls to the database.
Trying to load servlet class maybe is not sufficient. It's better to try to new an instance of a servlet class. Same for filter and listener.
In YARD you usually specify multiple types using a comma separator.
For more information, check out https://github.com/emberjs/ember.js/blob/master/packages/ember-metal/lib/mixin.js
Oh?  I’m curious about that.  Could you explain it to me?  

If it’s anywhere in the official docs, could you link me as well?

##### My reasoning was this:

* `is` tests for object identity
* Integers are immutable in Python
* Therefore, `2` is always going to have the same object ID as `2`

Since `len()` is always going to return an Integer, I believe that this:

```python
if len(args) is 2: 
    ...
```

Is no different than this:

```python
if bool(x) is False:
    ...
```

Naturally, I wouldn’t use `is` for most comparisons.  

----
K. I'll fix all the whitespace in the AM. Just leave the comments.
> а зачем надо отдельный haveAnswers, это не то же самое что !childs.isEmpty() ?

исправил
handleOpenActivity() will probably be invoked before the localized event fires. Maybe you can just call mozSetMessageHandler from the onlocalized function above...  That will slow your startup time a little bit, but you'll be sure that your unknown strings are defined.
Done, though I don't understand the reason. :)
I find this hard to read. IMO if you can fit it into one line its awesome otherwise I think the var should be defined here and then the value set after.
how about an `#answer` id? would facilitate formatting :)
This worth a class since this is reused a few times.
I'm a bit reluctant to r+ this for the browser given that we currently have no evidence it actually performs better than the much simpler change of just replacing the animated gif with the new one provided by Patryk.

This adds complexity and complexity = risk.

I'm happy to write a patch for the browser part myself if you want to land the gallery part.
If we always wait for 1 sec, that means the alarm will always 1 sec late. Is this really necessary?
Intentional empty line?
What about creating a |debug| function to not pollute too much the already dirty Gaia output?
Please explain what this `.actions` class means with a comment in the stylesheet.
So that stupid apps couldn't automatically   call expensive foreign numbers ? :)
I am told by @vingtetun that we should not correct this.
This comment confused me.  I think you mean something like:

  // If this app is not currently running, then we can't take a current screenshot of it, so let's try to get one from the db

Also, does this function need to check for the case where app is undefined?
Please remove all the fixed bugs.  No need to track those here anymore.
This will break if the message element is moved to a different place in the DOM tree.  You could either just make the element display:none, or get the element's parentNode and call removeChild on that.  Also, 'message' is too vague to be a good element id, I think.  I'd suggest "nosongs" instead.
Does 'disabled' works for links? I know it does for buttons, but not for 'a' tags. You sure of that?
The scenario is when any of the callbacks involved here is called more than once. Like `onsave` or `onsuccess`.
It should not happen, but the modification made in `pendingDBUtils` proved that it could by code mistake or errors.
Could you please tell me where to get the actual precision?

I wasn't able to find it.
why runsynchronous is true.
Yes, the functional tests are very slow to run. Check out the wiki docs here https://github.com/projectkudu/kudu/wiki/Running-tests
Maybe name it RebaseMaster to be explicit.
It is useful when you want to run only one file by hand ;-)
I know it already was in the code but anchors that don´t lead anywhere should not be anchors because anchors should be a reference to a resource. I prefer `div` or `span`.
This whole `P.` thing is kind of stupid, but the `include Pygments` into all the test classes and subclasses was causing weird inheritance behaviors and making the tests unreliable. Maybe there's a better solution, but I couldn't come up with anything..
also seems odd for this to be hear to me - but your call :)
... Or maybe use ```cascading.tuple.Tuples#coerce``` seems cool ... what do you think ?
Put this in a different file. Maybe `datastore_web_server`?
I have a KML file with co-ordinates split by line. I would suggest:
split(/[\s\n]+/)
well, not necessarily, it depends what is given as the languages parameter to the list() or query() methods of the services. The unittests at the moment sometimes give lists as it's easier, and I thought it'd be nice to have an API that accepts lists as well as sets of languages. The cost of building a set from a set is not that much either, especially given that most of the time you only look at a couple of languages at most.

You can also decide that the API requires a set, in which case I'd happily fix the unittests, your call.
No because this file didn't exist in 2011 (it was only created this year).
@stof yes I will do that
i'll explain that:

cmake builds are only supported on debhelper 7.6+ so this lines are nonsense due debhelper in lenny are too old for "--buildsystem=cmake"

so then, let the only line with buildsystyem premade due lenny build must use recent v 8.0.0 from backports
This might be clearer using the block form of `link_to`
My first inclination is to move the decleartion of `BuildmasterConfig` to a second file (without a `test_` prefix, so it doesn't get picked up by trial). My second thought is that we shouldn't need to write out a file, just to read it back in. My third is that we should use something like [exocet](https://launchpad.net/exocet) to isolate the tests (maybe even for loading the config??). And lastly, that for end-to-end testing, we should be launching a separate process.
Is there a real need? You can specify `[*]` defaults and if there is some file that needs a different indent style it can be defined and will override the default. In chaplin (at least as far as I've seen while digging around) everything is 2-space indent. 

This was indented as a start and if chaplin's needs grow where it needs these js files over here to be 4-space then that's where it can be extended nicely.
I know it's not part of this PR but why is RC uppercased here on line 44 as well (and maybe elsewhere too)? The regex is case-insensitive so you should be consistent with alpha/beta/... I think.
I just tried to make sense of what it should mean without success. Somebody who does understand what it is trying to say, please fix it.
This will be returned to its previous state.
In principle, it is fine to split the interface and implementation.
Not sure I follow what the issue was, seems to work fine while playing if I remove this line and return false.
Please rethink naming INVERT. I guess you mean low or high active.
Right, I'm just saying that if you don't specify port_number you'll get the behaviour you want without the loop around of_ports. If exp_pkt is received on any port (distinct from any_port) then dataplane.poll will return it.
指摘ありがとうございます
Yes, all are using that method. 

What I mean is to get everything "in line", look at the current com_installer and com_modules getActions helpers. com_installer is using variable $assetName, com_modules not. Still the same, but personally prefer consistency in it, also because people look at it for examples in their own components.
I tried this out and i fail to see how it's more flexible.  Seems to me if we go this route we can either set a universal metric name or use event fields but not mix the two.  I advocate for event fields, but I would like to support both in tandem, by having the ability to give a universal default if desired which can be overridden by event fields.
this appears to drop the message when no region is set, is that intentional?
Strange, I ran the unit tests (cfx testaddons) using your latest revision and they passed. Firefox 17, OS X and Windows:

    Testing content-permissions...
    Using binary at 'C:\Program Files (x86)\Mozilla Firefox\firefox.exe'.
    Using profile at 'c:\users\work\appdata\local\temp\tmpuxye7t.mozrunner'.
    Running tests on Firefox 17.0/Gecko 17.0 ({ec8030f7-c20a-464f-9b0e-13a3a9e97384}) under winnt/x86-msvc.
    info: content-permissions: executing 'content-permissions/main.testCrossDomainIframe'
    info: content-permissions: pass: received iframe html content
    info: content-permissions: executing 'content-permissions/main.testCrossDomainXHR'
    info: content-permissions: pass: received XHR content
Sorry, I made it more confusing now. At the moment `load` is passed in via `options.load` (see line 70) which is one from `Cu.import`. Although I personally would prefer not to pass it in and use `require('api-utils/loader').load` instead,  but  either way that requires fix for Bug 674195 to be shipped, so we can differ discussion weather it's worth it until it's shipped.
I don't think it really worth it keeping this code as cuddlefish will always be loaded as a jsm by bootstrap.js.
Might it make sense to return a new KeyPage object from this method?
test might fail sometimes if the file is read in below *before* the async writes are complete. you could either add a callback to the logger (per Logger or passed to the log method) to be notified when the write is complete.
This doesn't state where the U, C and P tags should be. It really should say something like, "Any bugs in Bugzilla with U, C and P tags in the whiteboard will appear in the backlog. Example whiteboard: "u=dev c=infrastructure p=2"."
Fixed. And the same at line 100
This is just a mistake. I looked over it. In my original patch, the approach was more general envisioning to support for other encodings. But in the end, I realized the support isn't actually needed in this time. So I trimmed down the patch. This is the remnant of it....
Why doesn't this dictionary have a ``timeout`` key (at least for clarity), even if it's just set to something like None?
So we're only giving 3 more seconds?  I thought you said that it could take up to 30 seconds.  If that is the case, how about making the sleep 5-10 seconds and printing a short message, like "Mongo database connection failed, retrying in 5 seconds...".  With a 5-second sleep, that gives the database something like 15 seconds to start up.
this regex is more powerful than what you have in the test/resources. Consider making this more restricted, for speed.
the parens here are a little wonky, I can remove those tho
@nschonni Do you think there'd be much value in doing so? The original variations of ``white-space: ...;`` that were replaced with force-wrap in commit #46d3680 mainly consisted of hacks targeted towards really old browsers (none of which could be considered A-grade nowadays apart from IE7, which is on its last legs). Maybe there'd be more value in adding another commit to this pull request that adds ``word-wrap:break-word;`` into *src/grids/sass/includes/_fix-ie7.scss* (which would restore IE7 compatibility)?
Nope, just curious because I didn't see any references to a jquery instance below.
Do you want this to be called also when you have `count`, `targetelement`, `allowadd` and `shouldcreatetemplate`? Otherwise it should be placed in the `default:` section of your switch, and you should return `$this` here.
Instead of doing all this parsing why not have a signature like:

func(object a, object x, int? lo = null, int? hi = null)

And let the binder handle all of this?  That sig might not be perfect, maybe some overloads are necessary (e.g. (object a, object x) + (object a, object x, int lo) + (object a, object x, int lo, int hi)).   But having some overloads would be much faster and cleaner then parsing the args manually.
same here (and some other places below)
Ok but I'm not sure to understand why not type them ?
You don't need this if you implement the template change I suggested.
I think this :blah: syntax is specific to restructured text, isn't it?
maybe calling `temp_view` or `raw_temp_view` would be better. Or we could just put them as methods of the `Database` class
this change is wrong. you should keep using the routing to generate the path (it makes it work when the site is not at the root of the domain thanks to the RequestContext for instance, like for me as I'm lazy about creating a new virtual host for each project I contribute to)
you should use the spacial placeholder name ``_format`` as it is the meaning your placeholder has
@vesak Tässä kohtaa $alehinta_val on aina $yhtiorow[valkoodi] mutta tuotteen_myyntihinta() palauttaa hinnan aslkun valuutassa jos valuuttahinta löytyy. Eli mennään hieman solmuun.
Cool! Sitä ei diffistä nähny!

On 19.11.2012, at 9.55, Sami Luukkonen <notifications@github.com> wrote:

> In tilauskasittely/keraa.php:
> 
> > @@ -122,8 +122,8 @@
> >  	}
> >  	else {
> >  
> > -		if ($yhtiorow['kerayserat'] == 'K' and $toim == "") {
> Koska ollaan if ($toim != '') else haarassa.
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
Blank lines are not allowed between case statements.
Missing line break before and after this line. Both lines are assignments while this one is a composite assignment, defining a different logical group
@beberlei this place is now in Common. If you want to trigger an event in this place, you will need to add a hook in Common.
It seems to be array now.
is it "--posterous"?
move comment above this line
list is a list? packages is a list, you meant
Not sure what introspection has to do with this. While the example in the current state is technically correct, it's not like it is particularly HATEOAS compliant - e.g. the task list does not return links to actual resources, but resource ids (returning the link on task creation is, however, HATEOAS compliant). The developer would have to know what is the actual URL scheme is for retrieving an individual task. It seems like we really want to do things in our own special way here. As as I said, it's not how this is handled elsewhere (including our own Forge plugin), and it's not an example I would personally recommend to follow (if anyone asked me :)). So I would change it (mapping the class to "tasks" instead while doing so). This is my feedback, I leave the call to Pete and Sande.
And a style thing: arguments need to line up.
don't use ``date``, it's a stdlib module and data type. Also, the fact that our HMAC keys use a date is not required, they can use any sortable string. so perhaps ``hmac_id`` or so would make more sense.
This should be a more restrictive regex.
Shouldn't the api_key be attached to the user_profile model?

I thought the standard was attaching things to the user profile model:
```python
    def get_api_key(self):
        try:
            return self.api_key.key
        except ApiKey.DoesNotExist:
            ApiKey.objects.create(user_profile=self)
            return self.api_key.key
```
This is actually part of ticket 1367
Don't leave commented out code
As I commented in source, this is copy-paste of the code we use in Nexus:
https://github.com/sonatype/nexus/blob/master/nexus/plugins/restlet1x/nexus-restlet1x-plugin/src/main/java/org/sonatype/nexus/rest/RemoteIPFinder.java

I did not want to muck with it, but it works for us. So, am really unsure about it's details.
Not true - a sysadmin still needs to be able to handle NAT networks, regardless of UUID=true|false.

I'll see what I can do next week. Not working hard today ;)
or not run this method at all if its disabled?
the whole point of compute resources and the fact that they have subclasses (e.g. ec2, openstack etc) is that you can override the methods :) so you dont need to hack hacks like this.

simply deifne a new console method in the lib/foreman/model/openstack.rb file
I don't know a simple way around this since I'm specifically looking for that function, not empty, null, etc. In all non snarkyness maybe I should open up a PR that documents `Handlebars.VM.noop` in the handlebars repo.
Imo this could still live in the imported modules, no?
"allready" is speled wrongly.
this is missing a '
FYI: `tmp/knife.rb` will never exist unless created manually...
Maybe this URL don't works fine on windows, in windows the path separator is \\ and this value is a URL not is a path, check this please
Is there a test for exception reporting?
Should not be necessary, since foreach would not loop an empty array
What do you propose in our case then? Thing is - we need to be able to updatePanelsHeight from listener of "postRenderEvent" in YUI datatable - "ezajaxsubitems_datatable.js".
If we fail to do that - table can "hang down" from the main content column. 
We are using fixed height (via JS) for our columns, because we want background in every column to be level with bottom.
I found _scannedLinks[link] has different keys like "link" and "link ", so I added this trim.
Useless method, no ?
Well, one of the main motivations for my changes was to change the TextileParser's behavior for this test. Currently (at 53e190f35574c7bd09e03102bff406fa522566e0), the string
```
"""<div class="vcard">
   <div class="fn">Joe Doe</div>
   <div class="org">The Example Company</div>
   <div class="tel">604-555-1234</div>
   http://example.com/
 </div>"""
```
is transformed into the XML:
```
<p><div>
   <div>Joe Doe</div>
   <div>The Example Company</div>
   <div>604-555-1234</div>
   <a href="http://example.com/">http://example.com/</a>
 </div></p>
```

I wanted a) to be able to include classes (which would break this test, but probably noone's existing markup) and b) to get rid of the surrounding &lt;p&gt; element (partly because nesting div into p is not allowed). In fact, I think that if we leave the input string unchanged (i.e., including spaces), the parser *should* transform it to:
```
<div class="vcard">
   <div class="fn"><p>Joe Doe</p></div>
   <div class="org"><p>The Example Company</p></div>
   <div class="tel"><p>604-555-1234</p></div>
   <p><a href="http://example.com/">http://example.com/</a></p>
 </div>
```
and I can try to make it behave like this – what do you think?
should be removed now
selenium is passed in as 's' :)

Nice use of find_element<b>s</b> too!
Don't know, will give it a try... I believe I just tried "str" without the "getValue" at first, and it just printed the classname and memory address, so it was totally useless.
Errr... I'm an idiot, I misread the docs below, and forgot that you switched it over so that it *is* actually the fact name here, not the node name.  Disregard :)
Perhaps there should be a /api/tenant endpoint for the global admin that has a isGlobalTenant: true property, which can then be used here.
A couple of thoughts on the DSL. First of all, I think it's useful, and we'll use it more and more as we get to do conditional breakpoints, watches, etc.

* `Either` seems overkill and obscures the logic of DSL users. I'd go for exceptions, thinking that in the majority of cases failure is "exceptional". It's debatable, but the DSL should make it easy to read and use, or otherwise it's better to just have a *classical* API, without trying to hard to make it nice
* wrap more objects coming form the VM. For instance, I think that `ObjectReferenceExt` is not needed, instead we should have a proper wrapper on `ObjectReference`, say `JDIObjectReference` where we have a few convenience methods:

```
  /** retrieve field */
  def f(fieldName: String): JDIField = ...
  /** retrieve method, no overloads allowed */
  def m(methodName: String): JDIMethod = ...
  /** retrieve overloaded method */
  def m(methodName: String, signature: String): JDIMethod =
```
(instead of strings, one could use `'symbols`.
Then, each of the wrappers have a nicer syntax:

```
  class JDIMethod {
    def apply(args: Value*): Value = ..
  }

  class JDIField {
    def apply(): Value = ..
    def set(newValue: Value) = ...
  }
```

And at the use site, you'd have something like: 
```
  val objRef = ..
  val manifest = virtualMachine.objectByName("Predef")

  objRef.m("toArray")(manifest.m("Any")())
  objRef.f("$MODULE")()
```
I'm sure I missed some important aspects of the API, but this seems like a more natural fit to Scala.
```
-jar "%%~dp0/sbt-launch.jar" ^^
```
としないと Windows XP で C:/Documents and Settings にインストールされたときにおかしくなるようです。
（XP まだ使うの？はともかくとして）
ну це приклад. треба і самому думати :)

якщо взяти файл без екстеншена і додати до нього ".jpg", то картинкой він від того не стане :)
помилку треба кидати
what is ``$jar`` in your comment ?
В данный момент нет, но в последствии могут быть изменения в других частях кода. Хотя лучше тут try-catch на диапазон i проставить.
Более "программистый" подход в данном случае - завести массив с элементами A T G C и по рандомному индексу выбирать один из его элементов.
будет заменена на, в связи с замечанием, о том что мы анализируем неравные элементы последовательности
if(![[self.DNA objectAtIndex: i] isEqual: [otherDNA.DNA objectAtIndex: i]])
{
// to do...
}
When using ActiveRecord
As above, this method should note that the streams must be of the same type.
It would deadlock. I thought that's the behavior that you said should occur here. There's no sane way to implement a blocking call that also does work on the same thread without using `NSRunLoop`, and `NSRunLoop` was what caused all the problems here in the first place.
This is <b>JDatabase::insertObject</b>, base class for all drivers, so this is common for all driver while "RETURNING" clause works only on PostgreSQL and maybe on Oracle, but surely it doesn't work on MySQL .
This change, and the following on this file, can be useful for <b>JDatabasePostgresql::insertObject</b>, or change this to use [implementation merged inside platform](https://github.com/joomla/joomla-platform/blob/staging/libraries/joomla/database/driver/postgresql.php#L884) .
Anyway, thank you to submit this change I think I'll push inside platform's driver.
```ruby
.should be_valid
```
Fixed absolute.  Should I override gofmt and add a blank line then the addservice line?

Oh, I don't think I can do it.  Hrm, if I make it:

    "github.com/kylelemons/go-rpcgen/examples/add/addservice"

Then I can't compile it, because mine is a fork.  I'll leave it at   "github.com/kylelemons/go-rpcgen/examples/add/addservice" for now.
The oid is [the identifier of the blob containing the text of the note](https://github.com/libgit2/libgit2/blob/development/src/notes.c#L221). Try to `repo.Lookup()` it.
This should be `SIGINT` instead of `SIGNIT` :-)
... which than would be saved in a nice fie format with an .mne extension :) Ok, make sense.
raw[..] wil also return the time, you should use

data, _ = raw[picks, first_samp:last_samp]
Hmmm actually we're not in a service ;-) I'd test how @Autowired performs here.
I'm not really sure what this code are doing.
Well, when used in the transceiver, a private member is ok, since the
message and its delivery receipt are managed by the same object. When using
transmitter + receiver, the receiver must access the ids of the sent
messages, so... it "must" be accessible in order to work :).


On Fri, Dec 7, 2012 at 5:29 PM, Ray Krueger <notifications@github.com>wrote:

> In lib/smpp/transmitter.rb:
>
> > @@ -3,6 +3,8 @@
> >
> >  class Smpp::Transmitter < Smpp::Base
> >
> > +  attr_reader :ack_ids
>
> Define "must" :)
>
> It only needs to be declared as a readable attribute if an external client
> wants to call transmitter.ack_ids. As it is, @ack_ids is an internal only
> concern to be managed by smpp::base.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/raykrueger/ruby-smpp/pull/11/files#r2350351>.
>
>



-- 

Quando hai fretta, l'unica cosa sensata da fare e' rallentare.
Is this a lie? It looks like nothing's happened yet
@stof it's more semantical and it looks like it has [a bug in IE](http://stackoverflow.com/a/3543695/1149495) but that resource is a bit outdated and I don't know if it is true today.
There's a second entry that needs to be added (most people miss it) - it's in the `map.inc.rst` file in the components directory.
I've debated myself on this - I use Response obviously to nicely mean/remind that we're talking about the Response object here. But, including the `:class:` in so many places really clutters up the source and I'm not sure it's necessary. So, my goal has been to make it a link when we're initially introducing a class, or if we're mentioning it in a way where being able to see its API docs is important. It's nice, because if we do this well, then when you see a link in the docs, you know that it's actually a spot where clicking that link has some value. If we add a link to the API docs here, it's not actually a time when a user would be interested in seeing the Response API.

If you think otherwise, let me know :).
If I'm not mistaken, the non-prefixed rule must come after the prefixed rules.
Why do you start a new list?
Please indentate correctly
I'm pretty sure cond is False should return False.
[Added in docrails](https://github.com/lifo/docrails/commit/69786d1d4a2cb861755d47dfb37d79ca504a474b)
Y U NO `const auto&`?
+2 (и ваще все это после =) можно не указывать руками, в енуме же они и так последовательно увеличиваются.
Is this needed? Shouldn't it be the callee who decides what to do with the exception?
`bank_account` should not be passed in for this request.
Why not use `lsbmajdistrelease`?
Thanks for having more detailed comments after my first remarks. However, comments should always describe the _intention_ of the code, not the code itself. Therefore the part "so return false" should not be here.
How do we ensure that the test are always executed during the build? Who removes test-nova.xml?
Thts exactly what its doing!! :smile:
@ed- it wouldnt do all of the same job by just checking the `'links'` because we are also checking for multiple root keys but i agree with the `.keys()`
I don't mind removing methods on public but semi-hidden interfaces, but I feel the configuration is too exposed to make it non-compilable from one minor version to the next.
This is actually a very simple parser for commands executed within the cli. I will use getopt for proper command line. I'll rename the class to remove ambiguity
maybe you should split the services.xml into separate file per guesser and in the extension only load those that are activated/configured by the config.yml
Your suggestion for a name, "sketch" would not help--it is no more descriptive of what "s" is than "s".  "s" is not a sketch, it is an argument of unknown type being checked to see if it takes one of several valid forms.
Isn't this needed to register the 3d projection?
I think we can simplify this by bringing a lot of the code inline. We can later split it out if we find another need for checking if WiFi is present.

The following works for me (removing the changes to GaiaData and gaia_data_layer:

    wifi_present = self.marionette.execute_script('return window.navigator.mozWifiManager !== undefined')
    if wifi_present:
        # forget any known networks
        self.data_layer.enable_wifi()
        self.data_layer.forget_all_networks()
        self.data_layer.disable_wifi()
This test is a bit different than the above one - I'd actually say this is a valid Gaia test, as this validates that the "entry points" manifest property operates correctly. This is a valid app manifest property that can be used in Gaia to support different launch points of an app.

Let's say entry_points regressed where it would launch the first app only (say dialer) - what this test could do then is catch that the "right" app was launched.

It's possible that the right spot for this style of test is within tests that would be things such as "test_dialer.py" and "test_contacts.py." But I wouldn't necessarily stomp on this not being a valid test.
Very clear! I like this doc.
is `relative-period` doing the right thing if you have to call `inc` on the results?
I think we could solve this case statement by changing it to:

```puppet
  case $::osfamily {
    'debian' : {
      ...
    }
    'redhat' : {
      ...
    }
  }
```
I looked through the documentation for `jQuery.support` and there is nothing there I read that could be used to identify this bug.
That blog post mentions they'll remove `$.browser` in 1.9, so there is still 1.8 before then.
For 1.9 they mention we can use `navigator.userAgent` which isn't ideal, but I don't think there is any other way.

[1] sounds like this bug so maybe it'll be fixed by then.

[1] http://bugs.jquery.com/ticket/5520
Would it be possible to hide this complete div when there is only one (or no) theme?
there is no need to cast to void ...
please alsways use {} if more than one line instructions after an if
Je to tak, jak říkáš. A ten výpočet v kódu je myslím stejný jako ten, který navrhuješ. Možná plus minus jedna, což se mi teď nedaří rozmyslet. Já jsem si provedl důkaz obrázkem.

Komentář dodám.
That's better, but I still think just allowing multiple saves would be the best fix. Why is it that tracking conversion rates doesn't suffer this multiple save issue? Is it something fundamental in the differences between tracking one metric and multiple?
This change is a no-op, in case you actually wanted to change something?
Need to check the return code of all of these.
Something tells me this is a problem...
Are you sure ``$parentPath`` has the correct type as "object<PHPUnit_Framework_MockObject_MockObject>" is not a sub-type of "string|object<Symfony\Component\Form\Util\PropertyPath>"?
do you want to show printing the arguments, like the JS version does?
also the "int i" is a bit confusing, if I didn't know C# it would make me think it was the length of extra or something (especially since i == 5 and there are 5 args passed)
It's unlikely to happen but I guess we could end up in an infinite
loop here if two objects hold references to each other.  Something
like this, for example:

```javascript
    var alice = {name: "alice"};
    var bob = {name: "bob"};
    alice.conspirator = bob;
    bob.conspirator = alice;
```

I don't think it's an issue we should try to do anything about right
now, but just noticed that it could be an issue so am mentioning it.
You're missing the throws tag on extract() and _getFilePosition() in this file.
Passei os preços para inteiros, (em cêntimos). Achei que não haveria problema... :S
You're right. I've made it protected instead, and renamed it as suggested.
Please use consistent names: Either change the name to `LinkesSources` or the value to `linked-folders`. If I understand this Eclipse feature correctly, I think you should change the value to `linked-folders`
GRAAAAAAAAAAAAAAAGHHHHHHHHHHHHHHHHHHHH. How are you merging???
I would rename this to `error_obsolete` because `error_date` is too close to `error_data`.
You can make this into one import line.
It seems most browsers support only [JavaScript 1.5](https://en.wikipedia.org/wiki/JavaScript#Versions).
.trim() removed!
Add your module to `CELERY_IMPORTS`.
Not well. DetailView and ListView are not suitable to work with mongodb (too much overriding). I can try basic view and just override get(). Would that be OK ?
They don't border - the problem is not in the border but in the padding that is added when you press the button - so yes, they behave different than in other browsers. Adding position:relative to a button is not enough (I don't know what is it good for) - it has to be added to child element like span. 

Anyway I can find a good reset.css if you think we should use it.
we don't need to check use_types any time in for loop below :)
oh I had everything messed up. I thought this view was the one to select the payment. That's also why I initially made another view and didn't understand why you didn't like it. Everything makes sense now. New patch coming up.
Hi @stof  , I'm working on defining this better, but I'm not sure what do you mean by move to the `Twig` namespace.
Should I make a PR in twig extensions to add the SQL formmater then continue with this? Best regards.
I think setUp() is run by JUnit before running these tests.
Is this needed? I don't believe thrift codegen is needed to build elephant-bird-core. Its just needed for the tests.
done. moved UnitTestUtil to pig/src/main from pig/src/test. Also updated how PigServer() is created (the util method sets Pig temp directory to 'test.build.data')
Yep, will do something like that next.
Correct.
Ideally we should not create a new ResourceInfo object if the corresponding networkLoggerEntry hasn't changed.

Either way the caching here should be fine since `metadata` is only accessed when displaying the detail view of that resource.
It seems this member is not used?
Likewise, a dash here ("pretty-printing").
Does daemon() really need to be called  *before* the QApplication is constructed? If not, I would move this code *after* we check if the app is already running (see app.isRunning() below). There is no point in daemonizing if we only need to pass command line arguments to the existing qbittorrent instance.
nit: move else if up a line
Suggest the shorter:
```
+s = galsim.Shear(eta=0.05, beta=45.0*galsim.degrees)
+s = galsim.Shear(q=0.5, beta=0.0*galsim.radians)
```
Shouldn't be any need for an explicit Angle construction.
This should probably just be repository instead of repository_object.
Embarrassingly, I don't know anything about this, nor did I spot it.
It must have come from a change in to the Iris code.
Hello Rene,
I think you should check for attributes[i].specified here. That way you can differentiate between default attributes (approx 75 attributes on IE7 if I remember correctly) and attributes that were really specified.
just one more small thing, you can push this line above the "if" and then can remove same line from 61 ;)
Not quite as far as I recall. I think Whirr assumes you have different machines for different services like in [this example](https://issues.apache.org/jira/browse/WHIRR-285?focusedCommentId=13020369&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13020369).
But let me get back to you with a detailed answer on that. I may be wrong.
I will rewrite it to cocoa.id.
yep, this passes my es6 test suite.
Yeah. Because your last "it" is referring to "the value of the named property", just like the "it" before it.
The conversions here aren't bad - just a new Array(...) and a set - it's
the sending that's slow: the postMessage here does a JSON.stringify on the
payload. I'd like to file some bugs on this - they really should have a
better way.


--
Ben Vanik
http://www.noxa.org


On Sat, Dec 1, 2012 at 9:39 AM, rsturgell <notifications@github.com> wrote:

> In injector/wtf-injector-chrome/wtf-injector.js:
>
> > @@ -195,6 +214,9 @@ function setupCommunications() {
> >          packet['wtf_ipc_sender_token'] == localId) {
> >        return;
> >      }
> > +
> > +    // NOTE: Chrome ports do not suppor transferrables! Need to convert!
>
> Wow, that's brutal. How long do these conversions take? Does it make sense
> to spin up a worker so that we can pass the data there as a transferrable,
> and then do the slow send from there? I guess that would only (possibly) be
> necessary in the streaming case.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/google/tracing-framework/pull/25/files#r2286013>.
>
>
Suggest changing:

    billed every 1 month

To:

    monthly
Sound good. I do want to make MongoLog a bit more useful though still. Please also note that there is already a debug.inc - which does something similar. I added that to aid debugging test cases - but don't use it anywhere I think (please check though ;-) ).
Use `querystring = parse_qsl(parsed_url.query, True)`
este código poderia ser: `return self.MONTHS.get(num_month, '')`
And now that I look at it, I'm not convinced that it's not going to screw up if there are wildcards in the default namespace. *sigh*
Actually, whenever running 'php bin/vendors' with any command
isn't this what the diff command does?
All of theses files get loaded sequentially (see ``JSLoader.loadAll``) which makes loading without the combinedModules very slow. If we would be more consistent about encapsulating code in modules then all of these files could be loaded in parallel.
Ach verdammt ... sorry ... sekunde das besser ich noch aus
Это точно медленнее toLowerCase()
Можно заменить на element.get(); if (null) .. else ..
Не компилируется из-за этого. Откатил pull request.
No need for an empty line