Ah, this cannot be used, because I don't just need to get it - I need to set the value also.

So, I'd have to do:
```
outline = outlines.get(result["severity"], [])
outline.append(line)
outlines[result["severity"]] = outline
```
Which is not as clean as what is currently there


I was thinking about the `cors` handler's `*` configuration, but you're right, the handler will always use the contain of the `Origin` header, so we always need the `Vary`. Nevermind.


Oops wrong account :-)


i think we can do it in case of similar tasks (i mean of you will update something in method which have old logs - you will update them also)


No. ES5 parses fine as ES6, so there is no reason to push this choice upon the user.


It's useful for the case when you have __*one header declaration written in Ceylon, and, say, the implementation directly written in Java*__.
In such as case, the header will be typechecked from sources and the corresponding Java implementation will be loaded from the model loader, and should be added to the overloads list of the typechecked header declaration.
The following code is for *this* use case.


Cygwin  in commit message looks like it's cygwin specific


C'est plutôt de la simplification d'API. Admettons qu'on ait besoin de mettre à jour plusieurs ressources (list batch par ex), il peut être préférable de faire une action, l'alternative étant de mettre à jour chaque ressource individuellement, ou de PATCH la ressource de la liste filtrée (?)

La considération de perfs réseau est un effet de bord, que je ne cite pas directement d'ailleurs.

Fondé sur http://restful-api-design.readthedocs.org/en/latest/methods.html#actions.


Good catch, will do.


Like to comment, no?


agree.


Ok understood, thx!


I usually give more hints than full solutions ;) I meant

```python
outlines[result["severity"]] = outlines.get(result["severity"], [])
outlines[result["severity"]].append(...)
```


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/18339/testReport/) for this commit**


Isn't this required for runtime as well?
`build_depends_on` will not introduce a dependency when the project is packaged while `depends_on` will.


According to the phpdoc of ::send, the return value is always Unirest\Response.


I agree. Need to rename the macro. Thanks.


I was not sure whether to put CONTAINER here, I think it depends on whether it is necessary to refer to the mapped meta definition via editor.


We dont have a driver for the MSELECT controller. Since there's only one register read/modify/write thought against adding a new driver


Hi,

these two functions are almost identical (the difference is around two format strings).
What if we would have
 * ConverterOptions (FormatterOptions?) struct which could store these format strings
 * a `text_converter_options_init`
 * a `binary_converter_options_init`
 * `append_unsafe_utf8`

In that case:

```C
void
append_unsafe_utf8_as_escaped_text(GString *escaped_string, const gchar *str, const gchar *unsafe_chars)
{
  ConverterOptions text_converter;
  text_converter_init(&text_converter);
  append_unsafe_utf8(&text_converter, escaped_string, str, unsafe_chars);
 }

void
append_unsafe_utf8_as_escaped_text(GString *escaped_string, const gchar *str, const gchar *unsafe_chars)
{
  ConverterOptions binary_converter;
  binary_converter_init(&binary_converter);
  append_unsafe_utf8(&binary_converter, escaped_string, str, unsafe_chars);
 }
```

What do you think?


Trying to get rid of gh-pages / master confusion


Constraints are generally named as: FK_Vendor.

To be confirmed from other colleagues at Optimus


Any reason for moving this? There is no actual change to the method impl, right?


there are different purposes of getting that gameUserbean in these methods...
In your method you need to return
in my method i need to do some operations on it (for example to check and skip if there are multiple calls to this method with same params)
And i think we should throw exception if gameUser not found. We can discuss this


Rename to Cab


This fix seems incomplete vs what I reported on google code


Congratulations


@dgageot probably all new scripts should use
```
set -euo pipefail
```


Identity can be used if we want to auto-increment


PEP20, "Errors should never pass silently." - so there should be at very least a warning.
Since we have a logger here anyway, we should use it.

However, such warnings are only noticed when (or **if**) someone inspects the logfile.
IMO, we should be able tell the user at least about the number of warnings during the execution.
See as well issue #6.


@parasyte should we create an overload method of another name?


I think the if condition is cleaner - that seems more difficult to
understand because it'd be simply replacing the variable with itself.

This may be a cleaner option if youre thinking 1liner is better:
`if not outlines.has_key(result["severity"]): outlines[result["severity"]]
= []`

On Mon, Jul 27, 2015 at 5:28 PM, Lasse Schuirmann <notifications@github.com>
wrote:

> I usually give more hints than full solutions ;) I meant
>
> outlines[result["severity"]] = outlines.get(result["severity"], [])
> outlines[result["severity"]].append(...)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala-sublime/commit/fc9e0e54d13580e4daa323e5d2b773c71ec58e62#commitcomment-12370329>
> .
>


s/adter/after in commit message


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Students/Goia I. Paula Ioana/Assig 2/Assig2.1/src/source/Functions.java

Why are they all static? :(


Also read about Composite Keys for knowledge.


Ton action n'est pas forcément un changement d'état de la ressource, exemple du item/50/translate
Ta ressource ne change pas, c'est la représentation qui va changer. Donc on ne simplifie pas forcément l'API, on l'enrichie avec du RPC car on a pas vraiment le choix.

Si je change en:

Bien que généralement à éviter, une action est utile car elle peut permettre de:
* simplifier l'API (avoir une action "enable" plutôt que de devoir mettre à jour la ressource)
* ajouter des "fonctionnalités" qui ne change pas l'état de la ressource (translate)


Students/Goia I. Paula Ioana/Assig 2/Assig2.1/src/source/Main.java

why do you call this here


and here again


- yes I guess we can use volatile cache (actually I have not much experience about module-level cache, what are the risks here ?)
- with the ZCA, I know how to unregister an utility, but not a provider, any hint ?


I have cherry picked it into the maintenance but this should have definitely been cherry picked before.

When we take ownership of an issue and it fixes a bug it is important to put our fix in the maintenance as patch releases are released from there....

Now we released 5.3.1 with that bug in it when we discovered it and fixed it days before....


Students/Goia I. Paula Ioana/Assig 2/Assig2.1/src/source/Polynomial.java

this is why `constructors` exist. move this into a constructor


Haha! You're going up against YouGov!
Good luck :)


Nevermind, I realized I can simplify it further

On Mon, Jul 27, 2015 at 5:34 PM, Abdeali Kothari <abdealikothari@gmail.com>
wrote:

> I think the if condition is cleaner - that seems more difficult to
> understand because it'd be simply replacing the variable with itself.
>
> This may be a cleaner option if youre thinking 1liner is better:
> `if not outlines.has_key(result["severity"]): outlines[result["severity"]]
> = []`
>
> On Mon, Jul 27, 2015 at 5:28 PM, Lasse Schuirmann <
> notifications@github.com> wrote:
>
>> I usually give more hints than full solutions ;) I meant
>>
>> outlines[result["severity"]] = outlines.get(result["severity"], [])
>> outlines[result["severity"]].append(...)
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/coala-analyzer/coala-sublime/commit/fc9e0e54d13580e4daa323e5d2b773c71ec58e62#commitcomment-12370329>
>> .
>>
>
>


Pretty okish


It should be remember that from the next release onwards the Realmd code will be a in a shared git submodule rather than part of this repo.

but in the meantime, this code will allow the sharing of a realmd across the cores


it is necessary that the client is not to get in idle and after disconnect


Read about: varchar vs nvarchar


reack


really messy way of doing this


(Was the name of the variable changed? It used to be filePrefix, now it says currentFilePrefix. That names doesn't make sense from the caller's point of view.)


Just put variable, it should be the correct thing.


Name : UserId


Container should be fine.


you can do this without specifically storing everything in an array and just reversing the number and storing its reverse form in a string or something


Идентацията трябва да е навътре


По един метод на ред в такива дълги вериги


add new h1 tag on new line


why are you not uploading `.java` files?


Weird spacing here. You really need to set your editor to autoindent.


Still a long line.


Yes, Keghani told me to change the name of the variable.


read up on method naming conventions


ack


what is `nAll` and `mAll` ..?


Oui, ça revient à ce que je disais:

- changement d'état complexe via REST: action simplifie l'API
- pas de changement d'état: action avec résultat non observable via l'API

Je suis pas certain que le use case "enable" justifie une action cela étant dit, mais c'est un autre débat.
Idem, pour translate, j'aurais imaginé une ressource /items/4242/translations/en_US par ex.


LGTM.


way too messy!!


@kares I see you have done this on master.  I wanted to test it with 1-3-stable as well.  OK to merge if it is green?

I see this has been released already.  Any reason it is done from master and not 1-3-stable?

Btw:  Any need for the ```History.txt```file?


Согласно этому посту http://stackoverflow.com/questions/9183321/how-to-use-jndi-datasource-provided-by-tomcat-in-spring
В этой строке у нас должно указываться имя базы данных. То есть логично написать:
<jee:jndi-lookup jndi-name="jdbc/flowerexpert" id="dataSource"


This only test proves round-tripping, assuming that .NET's default serialization for `DateTime` never changes, and is not culture dependent.

I'd suggest a test which actually verifies the string result too. Whatever the encoding is right now essentially becomes part of the API contract for Neo4jClient.

Imagine this scenario:
1. You add this change
1. You consume this update in your app, and start writing `DateTime` values to your DB
1. Something in Neo4jClient or .NET changes, which doesn't affect round-tripping (so the test still passes), but changes the encoding mechanism
1. You'd never know that the library no longer matches what is in your DB

Best case: it's read-compatible, and doesn't affect queries. (For example, if the number of decimal places changed?)
Worst case: it's not read-compatible, and it starts throwing errors at runtime, or silently corrupting data


You do not have a `Polynomial` class which models a `Polynomial`. Read the description


You should work more on your design


They are the lenghts of the polynoms
On Jul 27, 2015 3:11 PM, "Cosma Alex" <notifications@github.com> wrote:

> what is nAll and mAll ..?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gohonel/JavaMainRepo/commit/23412b82751484f7197c6d4ac545f5f0d47a3ced#commitcomment-12370581>
> .
>


Непонятна цель этого действия


Here for example, you should pass as a parameter two objects of type `Polynomial` and return another `Polynomial`!


As dates are so very culture-dependent, I'd suggest explicitly testing different cultures. See the `DeserializeShouldPreserveOffsetValuesUsingIso8601Format` test above for an approach.


Write some text about posthtml. Ex: posthtml can be used for add or clean html attrs, update classes, simplify structures and etc. More more and more about it. Aslo you can write about talk about the similarity with postcss in ideas.


I'd suggest making every component of the date/time unique, to flush out any issues with formatting macros getting mixed up. With this current test scenario, you'd never know if somebody mixed up the month and minutes.

Safest option would be something like `2001, 2, 3, 4, 5, 6`.


It's not. The compiler will generate the same code.


or vunit-hdl if hyphens are allowed


We should update `_check_result_status` to do error logging when the status check fails, specifically logging the result status, reason, and message. That way test runners will have access to all of the information on why an operation failed with their backend.


Formatting here is inconsistent with rest of file (should be a space between the parameters), and thus likely to cause unnecessary diff noise in future changes to this test/file. Better to keep the format consistent early.


не почистил после дебага - по сути на тест это никак не влияет


сделал в @9674e84e7c50f0f0514e1ab32f7c82fde6ef2c10


Any plugins which have a working version will now be located at github.com/KyleTheHack3r/PM-Plugins/releases. The 1v1 plugin isn't going to be worked on directly within GitHub, but I will push a commit whenever there's a workin version.


While you code is much simplier, it doesn't account for a person modifiying one of the yml files into include an effect that doesn't exist. I'm going to add the commit, but I will have to modify it to do the necessary checks.


This model supports `null`, but there's no test for how that will be serialized and deserialized.


This changes the default port on the machine where the integration tests are run.


I forked your project and I would like to generate proper license and copyright header with all authors mentioned.

PS: This project is really great and I appreciate all your work.


@lindamartin old license isn't removed


Looks good


You can directly use the rotation from the pose. P.rotation()
You can directly use the center from the pose. => P.center(), P.translation() depending of your need


and why would a `Polynomial` class contain the "lengths" (degrees) of 2 polynomials instead of one, and having two instances of that class?


Underscores are allowed for Modules.  Hyphens, IMHO, not.

https://www.python.org/dev/peps/pep-0008/#package-and-module-names
--
Modules should have short, all-lowercase names. Underscores can be used in
the module name if it improves readability.
---

On Mon, Jul 27, 2015 at 2:17 PM, kraigher <notifications@github.com> wrote:

> or vunit-hdl if hyphens are allowed
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LarsAsplund/vunit/commit/6c64f9cf02f07f8c6a271ba770118eb3e95c35d8#commitcomment-12370645>
> .
>

--
Jan Vermaete
"For every complex problem there is an answer that is clear, simple, and
wrong." - H.L. Mencken


Ah, I see we already have debug logging statements that do this in `_check_result_status`. Let's update those to be `self.logger.error` statements instead. That way test runners won't have to mess with the logging configuration to figure out what's going on.


Fine, you can keep these static, however, all your functions everywhere are static, and that is really bad. You are working on object oriented programming paradigm thinking, not the procedural one.


a `if/else` should do it no ?
````
var frameObject;
if (type is number or string)
   then frameObject = {...}
else
   frameObject = index[i];
````

that's what i would do, but I'm not sure what the JIT compiler would do with it :P


@shapiromatron Yup, sorry for the error emails. The script runs against gunicorn directly now, but still times out on the NC State Highway Patrol. I'm thinking of using a different caching method until #38 comes in.


Yes, home button is useless. you don't have to click on it.


because of we require it now in formservice and ecwidservice nginx snippets: eg: https://github.com/yola/formservice/commit/db632b095e04a22e4259284a1c7eb9f4bc0a32ea


@lindamartin Why do we need this line?


I would prefer vunit_hdl or vunit_eda. Seems strange to add a py prefix to differentiate with the other vunit package which is also implemented in python. I think the usage should govern the differentiating prefix/suffix not the implementation.


Подумайте о более оптимальных для производительности методах удаления записей из таблицы


Students/Sporis A. Ioan Catalin/Polynomials/src/Polynomial.java

You should not keep your coefficients in a `String` but in an array of integers, for example. You can however create a constructor for this class which takes a string as a parameter and creates such an array of integers from that string, however, the Polynomial object should not have a `String` representation of the coefficients.


also, make your fields private and create getters/setters


Students/Sporis A. Ioan Catalin/Polynomials/src/Functions.java

Careful on naming conventions


Not yet but we should! For now directories are tested one by one (see the check install page)


@vilchik-elena indeed thanks. Fixed in 0b7c3e63b5d73e8121c130d77cff7e88abaf30f8


We will need a test with just rank_2,
and possibly another one with rank_2/3/4.
(and might indulge a const to encode those 3 ranks)

Should we use param tweak bounds for these rank tests.
What exact bounds would you use ?

Later, I planned another simplification: remove the first PawnSafePush
in evaluate_threats

Another question: when patches are not so equivalent, SPRT is clearly the way to go,
but often people use numgames to compare very similar green patches
before choosing the one to submit to LTC.
Not clear when numgames is correct way.


s/indipendent/independent/


So should I review anything at all?


As discuessed (use clean_user_info()), Done


Also, why are you not passing 2 `Polynomial` objects as parameters and returning a third? you are in this way modifying one of them but that is not ok


Your Matrix assignment is quite alright


I think there is the same mistake in frenchtorrentdb and xthor.


you don't need an else if you use return


+1 vunit_hdl

On Mon, Jul 27, 2015 at 2:26 PM, kraigher <notifications@github.com> wrote:

> I would prefer vunit_hdl or vunit_eda. Seems strange to add a py prefix to
> differentiate with the other vunit package which is also implemented in
> python. I think the usage should govern the differentiating prefix/suffix
> not the implementation.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LarsAsplund/vunit/commit/6c64f9cf02f07f8c6a271ba770118eb3e95c35d8#commitcomment-12370771>
> .
>

--
Jan Vermaete
"For every complex problem there is an answer that is clear, simple, and
wrong." - H.L. Mencken


When hanging, there is no need to power off the machine.
Just open Task Manager, kill `msvsmon.exe`
Same for VS2012, VS2013 and VS2015.


Why not use an `inline` function?


Nope, `with` is right here. It's being used to describe "data-rich web applications *with* ambitious interfaces".


@alexeagle, are you the one to ask? :)


I don't understand.


Что еще за хрень? Зачем приводить Storage к пустой строке? Костыль?


Done


Errr ... what ?


#279 should fix this problem. Thanks @Simage ! Please don't hesitate to comment and/or open tickets if you see some bugs.


hhhhhhhh


"it's" → "its"


typo :p


Много повторяющихся тестов с различием лишь в значении одного и того же параметра.
Для этих целей следует использовать [@dataProvider](https://phpunit.de/manual/current/en/writing-tests-for-phpunit.html#writing-tests-for-phpunit.data-providers)


USE SQLTEST1
GO

SELECT
	MAX(u.Name) AS Name
   ,c.BrandName
   ,c.Number
   ,DATEDIFF(MINUTE,b.PickupTime,b.DropTime) AS TravelTimeInMinutes
FROM
	Cab c
INNER JOIN
	Bookings b
ON
	c.CabID = b.CabID
INNER JOIN
	[User] u
ON
	b.UserID = u.UserID
GROUP BY
	u.UserID
   ,c.BrandName
   ,c.Number
   ,DATEDIFF(MINUTE,b.PickupTime,b.DropTime)
HAVING SUM(b.Fare) BETWEEN 500 AND 1000


CVE requested in oss-security mailing list: http://openwall.com/lists/oss-security/2015/07/27/4

Thanks Alex for fixing this issue.


@vilchik-elena it was required for deployment of the plugin in maven central repository but with parent pom 23 it is not required anymore. But indeed here there is obviously a copy/paste mistake.
It is removed in fead4b8e42a5d1801b4bd784abacd6926255b082.


@pdion891 ping...

I'm checking the dependencies on Debian packages website: https://packages.debian.org/wheezy-backports/keepalived and will add the libnl1 to the wheezy-backports as well and see if there is any other package it might depend on.

Before I proceed, could you please  change the Job in a way that we keep the last 2 successful builds?

As a matter of fact, I'm still puzzled about how the builds from 17th and 22nd July worked.

Cheers,
Wilder


Oh, OK, I got it now, thank you. :+1:


Source Contribution of Traffic to PM2.5 by regions at urban locations (1990-2014)


silight optimisation by doing:
field = r.component.table[f]
field.writable = field.readable = False


Maybe this should be "Building" instead?


Hell ya


@al-martinez DWMR. Good stuff here. Press on!


This comment is outdated now?


厉害


Done


TeamCity xBIM :: Xbim Essentials :: Release [Build 63](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2138&buildTypeId=XBIM_XbimEssentials_Release) is now running


LOL


Can we please leave 0.10? I'm still using 0.10 in production.


I believe this is incorrect -- isn't the default version 1.4.2?


This seems a very peculiar place to do an onaccept routine...what exactly is this doing?


Hm the issue is, the patch i got there is actually really good. However, i will look to find a solution that this is not compiled under windows.


To test, download the following file and save it under: /usr/local/share/locale/fr_FR/LC_MESSAGES/OPNsense.mo

https://pkg.opnsense.org/snapshots/OPNsense.mo

After that, in file /usr/local/www/system_general.php in get_locale_list() add:

    $locales['fr_FR'] = gettext('French');

After that you can change the language to French under "System: Settings: General".

I'll take 75 into the upstream repository if you want. Languages will be enabled in the GUI when they reach 30% complete.

Let me know if you need anything else and sorry about the delay. :)


TeamCity xBIM :: Xbim Essentials :: Release [Build 63](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2138&buildTypeId=XBIM_XbimEssentials_Release) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:00


see: https://github.com/alibaba/tengine/pull/629


my preference is NOT to move .jars around branches, SQLite is a little special and 1-3-stable in particular should support older JDBC SQlite versions (e.g. due to Ruboto) actually can be easily switch-ed from ENV: https://github.com/jruby/activerecord-jdbc-adapter/blob/master/Gemfile#L58-L60


lol... come up with it on your own? or have some help?


Wygląda na zapytanie wykonywane dla contracts...
Co Ty mi tu podpuchy robisz? :D


Hmmm, to prevent naming collisions in the future, maybe this should include some sort of timestamp or random nonce to provide uniqueness? Just a thought, I wouldn't worry about it too much right now.


I think this is misleading in that readers may infer that they can configure the version of nginx used, when they cannot. I believe we can omit everything except the link to the releases page.


Editing README.md


I've taken a look at what this fn does...I still doesn't like it being called in a postp...perhaps you want to set an onaccept hook here?
s3db.configure(tablename, onaccept=xxxx)


ага, чтобы можно было валидатор на это поле повесить.


I'm right, these docs are wrong. From `bin/compile`:

```
    ver=go${GOVERSION:-1.4.2}
```


The 'if r.interactive' could be done just by indeting under the pevious check for that?


label = T() ?


If this line is commented out it seems to be more stable. Haven't figured out why.


"it's" → "its"


The same problem on my Nexus. But for WinPhone there is an app which can "cut" the melody and you're able to set it as a ring tone. I've used http://www.mobyss.com to find it. Maybe the same could be found also for android...


Never use nested queries.

* Use Joins


incorrect indentation


Issue #255 describes problem. For reference, it has been copied below:

>With word-wrap on, END moves to the furthest right location on the current display line. Pressing END again moves to the end of the actual line. However, with SHIFT+END, the END only does the latter: extend selection to end of the actual line. Similarly with HOME wrt the beginning of the display/actual line. It would be nice if the behavior was consistent for movement and for selection.

>This can be fixed with the following:

>    Remove Shift+End shortcut for SCI_LINEENDEXTEND, add it to SCI_LINEENDWRAPEXTEND.
>    Remove Shift+Home shortcut for SCI_VCHOMEEXTEND, add it to SCI_VCHOMEWRAPEXTEND.

>Should these changes be made to the default mappings?

This commit changes exactly what the issue suggests, and the behavior when word wrapping is enabled is much more consistent.


What is the point of the intermediate variable? Better to remove I think


Great, that fixed it.


Same here


fix #8


I think String.format is a more appropriate way of string concatenation


@mbernat It's just this part, `FindReviewIssueByCommitItem` -> `ParseReviewIssue` -> `AddReviewBlocker` -> `Edit`. It's not only about looking for problems, it's also about knowing about this change ;-)


SELECT TOP 1 s.* FROM
(SELECT
	MAX(c.BrandName) AS BrandName
   ,MAX(c.Number) AS Number
   ,COUNT(*) AS MaxCount
FROM
	Bookings b
INNER JOIN
	Cab c
ON
	b.CabID = c.CabID
GROUP BY
	c.CabID)s
ORDER BY MaxCount DESC


Please strike "as normal" -- Strunk #13, ["omit needless words"](http://www.bartleby.com/141/strunk5.html).

What's the default jruby version used?


Nice catch.


Uruchamiam php lms-notify.php -d i tyle, Tak samo uruchamiam starą wersję i leci debts/deadline ;-)


Major credit goes to: http://neuralnetworksanddeeplearning.com/index.html


Need to check above query.

It may work directly with one select.


Can only figure out "en-US", "it-IT", "es-ES", "fr-FR". "Vla", "Nor", "Hrv": unknown to me.


Now, if you could please fix the commit message to add a Signed-off-by: message, I'd appreciate it. I'm not merging this PR until you do that.


Is there a role which would have one and not the other?
: the GIS guys can create a cap_area but not update the alert
- the Alert Editor/Approver/Admin can do both
If not then no need for both checks...we can just check for ability to update this alert?
Even if so, then useful to note the relevant roles here (I'm aware these are in a template but I don't see this module being used outside that template for now)


BTW, locale files, and corresponding font bitmap setting seems better than just include those in source code.


Add an Area


`-o` is needed for `lcov`: `lcov -c -d . -o coverage.info`


this is 0.4-only, may need to add it to Compat


I don't really like this class. It has two separate implementation files, but the header is the same for all architecture, where half the flags are now irrelevant for a given arch. I'd much rather this not be a class (but just a plain struct), remove the member functions (or at least the Detect function, moving it to a static or a standalone function, and get rid of the constructor) and that it #ifdefs out the irrelevant stuff.

Any code that checks any of these flags is platform specific in the first place, and thus should be conditionally compiled, so this shouldn't be an issue.


Hm, so source maps are only supported when the assets are compressed — which isn’t exactly what I expected. I expected that if I were using, say, CoffeeScript and I enabled source maps, I’d get source maps.

Additionally — assets aren’t compressed by default in development, so even though source maps are enabled by default in development, they’ll most likely never be used, right?


You did not make a general implementation. I wanted a general one. And also a way to move backwards :smile_cat:.


USE SQLTEST1
GO

SELECT TOP 3 s.* FROM
(SELECT
	 MAX(u.Name) AS Name
	,COUNT(*) AS BookingCount
FROM
	Bookings b
INNER JOIN
	[User] u
ON
	b.UserID = u.UserID
GROUP BY
	b.UserID)s
ORDER BY
	BookingCount DESC


Sinjce we don't modify these lists then using a tuple is a little more efficient: (x,y) not [x, y]


Yes, but you say nothing changed (except for this one line). So I'm not sure what I'm supposed to review...


you open a pipe but never use nor close it.


We should differentiate the 3 as it looks odd with 3 'Load Template'
So
# Load Area Template
etc


Hi again @pdion891

That's my SSVM built upon the template 17th July

* SSVM Linux distro:
  * root@s-2-VM:~# uname -a
  * Linux s-2-VM 3.2.0-4-amd64 #1 SMP Debian 3.2.68-1+deb7u2 x86_64 GNU/Linux

* QCOW downloaded on 20th July:
  * -rw-r--r--. 1 root root 325160960 Jul 20 13:26 f2f43a13-77d0-432e-82c2-49f585a8b5a7.qcow2

According to Debian Packages page, the lbnl1 is equivalent to lbnl-3-200 package. You cans ee it here:

* https://packages.debian.org/wheezy-backports/libnl-3-200

When I tried to install the lbnl1 from wheezy-backports, I got the following:

root@s-2-VM:~# apt-get install -t wheezy-backports libnl1
Reading package lists... Done
Building dependency tree
Reading state information... Done
libnl1 is already the newest version.
0 upgraded, 0 newly installed, 0 to remove and 33 not upgraded.

I then tried to install the package lbnl-3-200 and got the following:

root@s-2-VM:~# apt-get install -t wheezy-backports libnl-3-200
Reading package lists... Done
Building dependency tree
Reading state information... Done
libnl-3-200 is already the newest version.
libnl-3-200 set to manually installed.
0 upgraded, 0 newly installed, 0 to remove and 33 not upgraded.

So, did the job configuration changed?

Cheers,
Wilder


Thank you!!


Would it be an idea to use a shell command instead of Perl for which ? This would remove the need for File::Which (which isn't included by default) as it seems...


Nice addition :)


this is interesting.... why do you have to check item.indexOf(source) and source.indexOf(item) ?


You aren't using the `afterAll` and `afterEach` methods right? That's the ones I would remove


This should be `return errno;`


On a second look, line 28 is useless


hmm the area_location isn't really a separate template is it?
So perhaps this is:
\# Add Area Location components of Template


I fixed PgSQL alter column syntax.


Right your are !
Same for the tempfile - I need to close it after reading it.

I'll do them both in the next revision of changes.


Actually this does more than 'load' so perhaps:
\# Create Area record from Template


ingnore the first commit it's only line endings


The dialog is started in a different thread as we don't want to block the demuxer when the dialog is open.
'bDialogForceStart' is used so that the demuxer thread can read the answer from the dialog.

I'm not sure where to put this dialog stuff, also do I need to lock the GUI? Need some help with this.


This should probably also not return something negative, but rather an existing error code such as `EGENERIC`. Also, I think you had a comment about this not getting reached here before? Best would be to add a `/* NOTREACHED */` comment above it as per the KNF style.


but why
```js
block('b1').elem('inner').def()(function() {
    return applyCtx({
        elem: 'wrapper',
        content: this.ctx
    });
});
```
in 1.x doesn't results with infinite recursion? my vote for make it same in 2.x — is it cause any problems? performance?


@mbernat Actually I was right, the rest was already there :D


Whoops - edited a wrong file. Please discard.


I wonder, should you check the JTA transaction status to make sure it is not marked for rollback already somehow?


I know I'm anal, but I prefer these 2 statements the other way around, so they grow in size:
T = current.T
db = current.db
s3db = current.s3db

All the same in every piece of code helps things appear smooth & clear (this currently jars with me)


```
// set player.voted to true after submitting a vote
Players.update(currentPlayer._id, { $set: {voted: true}});
```

Is still needed for the if statement to check for **one vote** per player **per voting round**.
`if(!curretPlayer.voted) {...}

Also, `Session.set("playerID", null)` is in the wrong spot. At the current spot all players would have there ID wiped from the Session and not the player that is voted out. It should be in the if statement check of your next commit.


Just a few minor issues flagged above. The update looks great overall though.


base.apply!


Should this be done before it’s merged, or not? [This StackOverflow answer](http://stackoverflow.com/a/19912684) suggests `application/octet-stream` or `application/json`, but also that it doesn’t really matter.


Michael, it most certainly should support optional parameters. And in 3.0, it certainly does.


speaking about `once()` and context comparing problems I suggest to slow down porting to 1.x — we need to figure out how it should work


in ruby 2.2 seems the 'names' object is being optimized and is always the same object resulting in the query object having only one entry. this fix that problem


We have a problem here. Module descriptors will be stored multiple times along with all of their dependencies. Consider storing only path of dependency modules.

Here is an example of what you have stored upon serialization:
```xml
    <ErlangModuleBuildOrderDescriptor>
      <module>boss_db</module>
      <erlangModules>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/boss_db_pt.erl</path>
        </ErlangModuleDescriptor>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/inflector.erl</path>
        </ErlangModuleDescriptor>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/boss_cache_sup.erl</path>
        </ErlangModuleDescriptor>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/boss_db_cache.erl</path>
        </ErlangModuleDescriptor>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/boss_db_test.erl</path>
        </ErlangModuleDescriptor>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/boss_db_adapter.erl</path>
        </ErlangModuleDescriptor>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/db_adapters/boss_db_adapter_dynamodb.erl</path>
          <dependencies>
            <dependency value="/home/user/projects/ChicagoBoss/deps/boss_db/src/boss_db_adapter.erl" />
          </dependencies>
        </ErlangModuleDescriptor>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/db_adapters/boss_db_adapter_mnesia.erl</path>
          <dependencies>
            <dependency value="/home/user/projects/ChicagoBoss/deps/boss_db/src/boss_db_adapter.erl" />
          </dependencies>
        </ErlangModuleDescriptor>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/db_adapters/boss_db_adapter_mongodb.erl</path>
          <dependencies>
            <dependency value="/home/user/projects/ChicagoBoss/deps/boss_db/src/boss_db_adapter.erl" />
          </dependencies>
        </ErlangModuleDescriptor>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/db_adapters/boss_db_adapter_riak.erl</path>
          <dependencies>
            <dependency value="/home/user/projects/ChicagoBoss/deps/boss_db/src/boss_db_adapter.erl" />
          </dependencies>
        </ErlangModuleDescriptor>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/db_adapters/boss_db_adapter_mysql.erl</path>
          <dependencies>
            <dependency value="/home/user/projects/ChicagoBoss/deps/boss_db/src/boss_db_adapter.erl" />
          </dependencies>
        </ErlangModuleDescriptor>
        <ErlangModuleDescriptor>
          <path>/home/user/projects/ChicagoBoss/deps/boss_db/src/db_adapters/boss_db_adapter_pgsql.erl</path>
          <dependencies>
            <dependency value="/home/user/projects/ChicagoBoss/deps/boss_db/src/boss_db_adapter.erl" />
          </dependencies>
        </ErlangModuleDescriptor>
...
```


I think the issue is more todo with the type of calls
On Mon, Jul 27, 2015 at 8:24 AM Olivier Biot <notifications@github.com>
wrote:

> a if/else should do it no ?
>
> var frameObject;
> if (type is number or string)
>    then frameObject = {...}
> else
>    frameObject = index[i];
>
> that's what i would do, but I'm not sure what the JIT compiler would do
> with it :P
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/melonjs/melonJS/commit/8f4aa972711649ff8cd734bbd2694d2fbfe50f0a#commitcomment-12370746>
> .
>


Nesting is wrong.


I also ran into this issue when running tests on a fresh installation of `node` — I think my last laptop was on 0.10, so it might be a `0.12` thing? /shrug thanks for the fix!


Just reread the next commit, and it seems the last point is addressed, so just delete the `Session.set("playerID", null)` line from line 784.


Добавление обработчиков js_inited без вызовы родительского метода ломает очень много. При наличии тестов это было бы сразу видно.


SELECT
	 MAX(u.Name) AS UserName
	,MAX(v.Name) AS VendorName
	,COUNT(*) AS BookingCount
FROM
	Bookings b
INNER JOIN
	[User] u
ON
	b.UserID = u.UserID
INNER JOIN
	Cab c
ON
	b.CabID = c.CabID
INNER JOIN
	Vendor v
ON
	v.VendorID = c.VendorID
GROUP BY
	 b.UserID
	,b.CabID
	,c.VendorID


Jak nie dasz --type to lecą wszystkie typy i pewnie dlatego złapałeś akurat zapytanie dla --type=contracts


**#bug**
code duplication
all if statements below answers a question hasSubprojects() ?


Of course! I still have to go through some stuff. My apologies for the delayed response, as I have been busy the past few weeks.


This seems good to me.


@vilchik-elena thanks. Amend in d67cba5362ef548867770a8920aef1606551dcd2.


And also, what's the point of having both ```ErlangModuleBuildOrderDescriptor``` and ```ErlangModuleDescriptor```?


git status


These are always hard-coded in the workflow?
I confess this appears odd to me...I would envisage a dynamic filter here - although with a pre-selection perhaps.
I also confess that currently I don't understand why areas are different for different priorities...is this basically scale?
e.g. Large Earthquake, Medium Earthquake, Small Earthquake?


This is going to pull all requests to locationtrivia.herokuapp.com. So if they are passing in the incorrect path or passing in the incorrect http verb, it will still hit this stub. Make sure you filter for the right path and the right http verb.


Scratch that, I see you’ve addressed this in a follow-up commit. #tooearly


hello world


better error message.


see previous problem on stubs. Not specific enough


and other thing - if user search in empty collection, should it get exception (is it error or unexpected behaviour)?
Are standard algorithms doing it?


Of course the code is the same - it's just a style thing - easier to see that there is always a return value. But I will just remove the style stuff.


It replaced scg with scglive because the icon is named after scglive while the organizer in the tournament table was called SCG


oh~ I forgot that. :)


This has also been addressed in a follow-up commit. :100:


Can you create issue or pr about this?


And this)


unused variable?


Кто спорит)


Hi @wilderrodrigues,
I saw the job success on 22th, I'm wondering if dependencies for keepalived package would have change recently on the repo.

I've changed the build keep to 2.


whats up :dancer:


USE SQLTEST1
GO

SELECT
	 MAX(c.Number) As Number
	,MAX(c.BrandName) AS BrandName
	,COUNT(*) As Count
	,u.Gender
FROM
	Cab c
INNER JOIN
	Bookings b
ON
	c.CabID = b.CabID
INNER JOIN
	[User] u
ON
	b.UserID = u.UserID
Group By
	c.CabID,u.Gender


To be checked with Left Join


Awesome, good job


If `get` fails, we should still try to `destroy` the key. Otherwise, we'll orphan it on the backend. If `destroy` fails, then we're out of luck.

We can either change this here, or go back in a later patch and fix it across all integration tests. I'm fine either way.


Nie, to jedyne co leci.
Jak daje debtors to w ogóle nic nie ma:

150727 14:57:52	   16 Connect	lms@localhost on lms
		   16 Query	SET NAMES 'UTF8'
		   16 Query	SELECT section, var, value, description FROM uiconfig WHERE disabled = 0


Template? I tried it, but it produces errors when I'm trying to compare float and btScalar values (wrongly identifying btScalar as double).


Now why would you go and trim trailing space like that???!


One thing I'm not sure about and would like advice: Copy the items or not? It appears that sorting them in-place wouldn't hurt anything, but it seems inherently wrong that a printer function would modify it's input. However, few lines down at https://github.com/openshift/origin/blob/master/pkg/cmd/cli/describe/printer.go#L298 we're doing just that for project lists.


look like this line is not working on when building .deb  the ${VERSION} remain empty.
why not change it for something like this:

```
cat pom.xml |sed '22!d'| cut -d'>' -f2 |cut -d'<' -f1
```


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/18341/testReport/) for this commit**


Do a pull request. ;)


I workaround this in the next commit. Don't really need it.


This looks like its been duplicated from a prior commit. I'm assuming you'll rebase this patch to remove this before we merge it in.


I thought so too, but there was another organizer that had different name for icon and the tournament table; so I changed the icon's name to match the organizers in the tournament table and changed the related code accordingly.

Sorry about changing it without telling you :(


We probably shouldn't check for the exact `ResultReason` here. I can definitely see this changing from appliance to appliance. The only thing that really matters here is that the operation failed.


ack


Yes, I think it makes sense


Calls to class methods like `before_action` should be placed at the top of the class, FYI.


Does this return a new object or update the `result` object in place? If it's the former then line 55 below, where you `result.delete(:_raw_result)` the previously set key in the hash, wouldn't be needed if you used two separate variable names.


赞


according to the API, delete should return a `204` and no content not an empty array


Non observable par l'API m'a un peu chagriné mais on est ok finalement ^^

Détails mais j'aurais plutôt fait /items/4242/translation?lang=en_US: c'est plus une action de transformation de la représentation dans mon esprit.


Две лишних пустых строки получилось.


Hi @kenrussell,

I am adding the support for dx10 textures.. I saw you wrote the main part of jogl texture, may I ask you something?

Regards


In the build that failed it mentioned the libnl-genl-3-200, which is already installed in my SSVM, based on the systemvm from 17th July.

But it also mentions the libnl-genl-3-200, which I did not try to install before. I just gave it a try and also go a successful response:

root@s-2-VM:~# apt-get install -t wheezy-backports  libnl-genl-3-200
Reading package lists... Done
Building dependency tree
Reading state information... Done
libnl-genl-3-200 is already the newest version.
libnl-genl-3-200 set to manually installed.
0 upgraded, 0 newly installed, 0 to remove and 33 not upgraded.

Checking the logs of the build from the 22nd I got the following:

+ apt-get --no-install-recommends -q -y --force-yes -t wheezy-backports install keepalived irqbalance open-vm-tools
Reading package lists...
Building dependency tree...
Reading state information...
The following extra packages will be installed:
  libcap-ng0 libdumbnet1 libicu48 libnl-3-200 libnl-genl-3-200 libnuma1
  libperl5.14 libsnmp-base libsnmp15
Suggested packages:
  snmp-mibs-downloader open-vm-tools-desktop
Recommended packages:
  zerofree
The following NEW packages will be installed:
  irqbalance keepalived libcap-ng0 libdumbnet1 libicu48 libnl-3-200
  libnl-genl-3-200 libnuma1 libperl5.14 libsnmp-base libsnmp15 open-vm-tools
0 upgraded, 12 newly installed, 0 to remove and 33 not upgraded.

So, it installed all the new packages.

Perhaps starting the build manually messed up something. It's very weird. :(

cheers,
Wilder


No worries, you can change whatever you want ^^
Just wanted to tell you why that line was there, if you changed the root cause all the better :)


W dniu 27.07.2015 14:59, paszczus napisał(a):
> Nie, to jedyne co leci.
> Jak daje debtors to w ogóle nic nie ma:
>
> 150727 14:57:52 16 Connect lms@localhost on lms
> 16 Query SET NAMES 'UTF8'
> 16 Query SELECT section, var, value, description FROM uiconfig WHERE
> disabled = 0
>
> --
> Reply to this email directly or view it on GitHub [1].

Jak dokładnie próbujesz uruchomić skrypt?

--
Pozdrawiam
Tomasz Chiliński, Chilan


```cpp
template<typename T, typename U, typename V>
T clamp(T v, U h, V l)
{
  assert(h <= v);
  return std::max<T>(std::min<T>(v,h), l);
}
```


Could you please squash this into the commit where they were forgotten?


Kak, coba dong yang ditampilkan oleh perintah print bukan hanya luas-nya saja, tapi juga kata2 ini,
"Luas segiempatnya adalah ????"

bisaaa? lihat contoh sebelumnya.


The difference is the Task vs Setting? I am moderately surprised by that, but my sbt skills aren't exactly up to date, so maybe I shouldn't be.


please can you document a little bit @makevoid thanks


php lms-notify.php --type=debtors -d lub samo php lms-notify.php -d , bez -d to samo


I'll do soon as possible, sorry by the delay :(

Regards


SELECT Name, AVG(rating) FROM
Vendor
JOIN
Cab
ON
Vendor.VendorID = Cab.VendorID
JOIN
Bookings
ON
Bookings.CabID = Cab.CabID
GROUP BY Name


Ok, I will test tomorrow at night :) Ty


@chandralaxmi please indent


I explained the how but not the why:
`<div><p>Text</p><p><a>Read more</a></p></div>`

How do I get ` Text` and `Read more` on the same line? This commit is a css-ignorant way.


Done.


@swdevbali maksudnya apa luas segi empat?????


W której sekcji konfiguracyjnej i jakie ustawienia posiadasz?
Jeśli nie dasz --section=bleble to skrypt bierze pod uwagę sekcję konfiguracyjną notify.
Ustawienia niezależnie od sekcji to:
debtors_....


Ok, fair enough :)


damn, I forgot to add `CURSOR_COLOR`


@saherneklawy I want to verify that this commit is live somewhere (mainly the heroku showcase).


Itu lho, baris ke 5 kan perintahnya:
print luas

jadi misalkan nilai luas itu 10, yang muncul:
10

gitu aja.

Nah, coba ubah baris ke 5 supaya yang muncul adalah
Luas segiempatnya adalah 5

gitu lho. Bisa ???


please change blacklist dvd_usb_rtl28xxu to blacklist_dvb_usb_rtl28xxu

That d makes a big difference.  :)


Fix it with this commit:

https://github.com/t4dek/short_url/commit/78d7d4eccca049d6e80f02be2d95202a79dd61c2


lgtm (ack)


Could the default be 'all' here?


ack


tbh I haven't -- my IDE now automatically wraps at 120 chars so that checkstyle will not complain


A small comment would be welcome to tell what the function is doing and what it returns.


USE SQLTEST1
GO

SELECT
	 MAX(c.BrandName) AS BrandName
	,MAX(v.Name) AS VendorName
	,SUM(b.Distance) As TotalDist
	,SUM(b.Distance)/SUM(DATEDIFF(MINUTE,b.PickupTime,b.DropTime))*60 AS AverageSpeed
FROM
	Bookings b
INNER JOIN
	Cab c
ON
	b.CabID = c.CabID
INNER JOIN
	Vendor v
ON
	c.VendorID = v.VendorID
GROUP BY c.CabID


This test is not needed after changes in this commit:

https://github.com/t4dek/short_url/commit/77690256f3253c33999a0ed7cfc15d06b3883750


Should be nl-NL, no-NO, hr_HR for Netherlands, Norway and Croatia.
Actualy we can use only the language part, like en,fr unless country/dialect needed to specify like with pt/pt-PT/pt-BR or zh-Hans/zh-Hant
Here is the full language list: http://www.localeplanet.com/icu/iso639.html
The existing one 3-digit looks more like ISO-3 alphabetic country codes (not language!)https://en.wikipedia.org/wiki/ISO_3166-1
Autodetecting the system language (or at least detecting the region and switching to jp/ko/zh when Unicode font will be implemented) will be a good feature too


O to przy okazji dobrze wiedziec - fajne! Ale ja ustawienia mam w sekcji notify, np. debtors_limit, debtors_message itp. Tak jak pisałem wyżej


Line 151: also comment </MediaCodec> out


It may better set "-1" as default


I forgot to add `CURSOR_COLOR`, @Mayccoll you fix it, please? I'm to lazy to create a new PR because of this tiny thing ;)


Edits the tabs at the top of the website


Line 172: also comment MediaCodec out


where does that variable come from?


Deleted within changes in this commit

https://github.com/t4dek/short_url/commit/78d7d4eccca049d6e80f02be2d95202a79dd61c2


I love this!


Adds the backgroung image


1: M111 unrecognized component name: recordext


Actually, I think the old way is better style to see that there is a return value in all cases. The change you made kind of visually decouples the value returned from the cases when it happens, because you break the if/else structure.


USE SQLTEST1
GO

SELECT TOP 1 * FROM
(SELECT
MAX(BrandName) AS BrandName,
SUM(Fare)/SUM(Distance) AS AverageFare
FROM
Cab INNER JOIN
Bookings ON
Cab.CabID = Bookings.CabID
GROUP BY Cab.CabID
) S
ORDER BY S.AverageFare


Fix via this commit:

https://github.com/t4dek/short_url/commit/7661e5a987e59d01f358f3c8bdb11bdfeddb41de


This seems like a type I didnt realize. Will change


`if (filter)` doesn't read well, but `if (filter() ==  true)` (if filter returns true) does


I thought it wasn't based on a glance at `to_dict`, but it's better to be safe.


Fix via this commit:

https://github.com/t4dek/short_url/commit/6983c7437028a3097fd6b31f24a694c6bfb0d0ad


CVE requested in oss-security mailing list: http://openwall.com/lists/oss-security/2015/07/27/5

Thanks Alex for fixing this issue.


> This branch is 800 commits ahead, 2 commits behind rubyspec:master

Yeah, definitely not dead.


testing it.


Either have braces or don't, but be consistent


Maybe assign the result of this expression to a clearly named variable, to help readability, eg:
```python
aol_is_name = aol != slugified_aol and AreaOfLaw.objects.filter(slug=slugified_aol).exists()
if aol_is_name:
```


This doesn't neat to exist. Use Floats/Doubles.concat from guava.


hmm htsp specs: subscriptionChangeWeight -> Reply message fields: None
Strange as we get a reply with the predictive tuning stuff..


Daj na wszelki wypadek uruchomienie:
--section=notify --type=debtors -d
i pokaż jakie zapytanie sql wykonuje?


A hint on why this will be needed may please reviewers (although it's rather easy to guess if you know `git branch` well enough).


CREATE DATABASE SQLTEST1
GO

USE SQLTest1
GO

CREATE TABLE Vendor
(
	VendorID INT PRIMARY KEY,
	Name VARCHAR(100)
)

GO

INSERT INTO Vendor
VALUES(101,'Sai Travels'),
(102, 'Meru Cabs'),
(103,'Miracle Cabs')

GO

CREATE TABLE Cab
(
	CabID INT PRIMARY KEY,
	VendorID INT,
	Number INT,
	BrandName VARCHAR(20)
)

GO

INSERT INTO Cab
VALUES (201,101,8529,'Mercedes'),
(202,103,5764,'Jaguar'),
(203,101,1967,'Lamborghini'),
(204,102,7359,'Mercedes'),
(205,103,1992,'Audi'),
(206,103,0786,'BMW'),
(207,101,0007,'Audi'),
(208,102,8541,'Fiat')

GO

CREATE TABLE [User]
(
	UserID	INT PRIMARY KEY,
	Name	VARCHAR(20),
	Gender	CHAR(1),
)

GO

INSERT INTO [User] VALUES
(301,'Ravi','M'),
(302,'Kavi','F'),
(303,'Abhi','M'),
(304,'Savita','F'),
(305,'Gopal','M'),
(306,'Bhopal','M'),
(307,'Dolly','F'),
(308,'Tanu','F'),
(309,'Prince','M'),
(310,'Raj Kishore','M')

GO

CREATE TABLE Bookings
(
	BookingID	INT PRIMARY KEY,
	CabID	INT,
	UserID	INT,
	Fare	DECIMAL,
	Distance DECIMAL,
	PickUpTime DATETIME,
	DropTime DATETIME,
	Rating	INT
)

GO

INSERT INTO Bookings VALUES
(401,204,309,101,13,'2015-04-07 19:00:00','2015-04-07 19:30:00',5),
(402,205,301,105,15.2,'2015-05-11 09:15:00','2015-05-11 10:00:00',3),
(403,204,309,2000,190,'2015-03-19 20:45:00','2015-03-20 01:00:00',2),
(404,201,302,1995,150,'2015-07-07 11:00:00','2015-07-07 15:00:00',5),
(405,204,303,553,50,'2015-09-12 19:00:00','2015-09-12 22:15:00',2),
(406,202,302,465,45,'2015-01-07 09:00:00','2015-01-07 09:40:00',1),
(407,205,304,258,20,'2015-07-02 03:00:00','2015-07-02 03:15:00',4),
(408,202,309,125,15,'2015-06-23 09:00:00','2015-06-23 10:00:00',5),
(409,204,310,1462,30,'2015-02-05 06:00:00','2015-02-05 08:00:00',4),
(410,207,306,1876,60,'2015-01-29 15:00:00','2015-01-29 18:00:00',1),
(411,203,308,1145,100,'2015-06-04 20:00:00','2015-06-05 06:00:00',0),
(412,206,309,1358,90,'2015-01-19 02:00:00','2015-01-19 08:00:00',1),
(413,208,301,102,5,'2015-03-21 11:00:00','2015-03-21 11:15:00',5),
(414,206,309,503,50,'2015-02-28 08:00:00','2015-02-28 10:00:00',4),
(415,204,304,786,62,'2015-03-09 16:00:00','2015-03-09 19:00:00',3),
(416,208,306,143,3,'2015-04-09 11:30:00','2015-04-09 11:45:00',2),
(417,203,309,658,12,'2015-05-04 01:00:00','2015-005-04 01:40:00',0),
(418,206,308,852,17,'2015-02-18 15:00:00','2015-02-18 16:00:00',1),
(419,208,301,450,22,'2015-03-11 18:00:00','2015-03-12 10:00:00',4),
(420,204,309,420,29,'2015-02-17 11:00:00','2015-02-17 21:00:00',1)


What do you expect to see in an Area Profile page?
A Profile page basically has components of a resource.
An Area has the Area Location component, but I'm not sure there's much value to having this be a Profile page section rather than an Inline component?
A profile page does have a map section which I can see being very relevant, although this could also be put into an rheader


No i wrzuć tu zawartość sekcji notify


oops, that did not work, next commit sorts it


No, I'm just working on the Angular typings. Are you looking for a core contributor?


Good news. I'm surprised the MRI team uses, as I remember Brian cited lack of involvement from it as the main reason to kill the project. Someone should take over the site, btw - http://rubyspec.org/


it's not there  because some views do not require an 'aol' parameter. and there's no model for 'all' in the DB.


Or a function


the point of defining `to_string ()` is so you don't need to call it in `@"this type of strings"`


Sine this is only touching dev dependencies, I see no objection to switching to io.js (althuogh it would require me to roll out io on a couple of machines :D).

On the other side, this would mean, that our dev stack would be inaccessible to node users for the time being... But maybe that's okay for the moment


@ssut ?


Why i18ncmp?


This `transformed_get` code could be a function, to reduce complexity here


Actually no,that was added because in Junits that switch did not work correct.


Like before, this is duplicated.


Will be fixed


Now groups by residue/nucleotide type AND atom type - since main chain atoms not expected to behave the same as side chain atoms, so seems illogical to group all as one..


`CryptographicAlgorithm` and `CryptographicLength` actually don't apply to `SecretData`. Yeah, I know it's weird, since `SecretData` uses a `KeyBlock` and the `KeyBlock` has space for those two attributes. That's just how the KMIP spec works. You can take these out, since their presence can actually cause errors on more finicky backends.


Still having problems with my Dev keys, there is an error, I have no access to my company file , not even to the app itself.


@pierre-rouanet  why adding the config of ergo robot here ?


The type for `SecretData` `KeyBlocks` should be `OPAQUE`. At least, that's what the spec claims.


页面


You can set an interface as red only if you have nethserver-firewall-base installed.
It doesn't depend in any way on the user name. :)


    return 0, 0, err

should be enough, os.Stat() should add exactly the same prefix, see http://play.golang.org/p/Q1mIaBK903 - so here I'd expect you'd have it duplicated.

Same for similar patterns below (os.Open, not sure about dir.Readdirnames)


navbar name


this is a good reference but it would be even better if you referenced the equation number in the SAG paper. and then explicitly state that "E below is a copy of equation XXX in paper YYY." As it is, I'm not sure if we need to look in the Schmidt or Bishop reference, and where exactly we would need to look, and which equation below is taken from there.


Now the app won't use text configurations files. It will save and load settings from Windows registries.
Created new module: Settings.vb
Form3 re-designed without "Default mail configuration". Default mail configuration has desappeared.
Fixes #2 .


Will be fix


Are you sure about that?
Doesn't this mean that the positions of the non-current manuscripts are *not* restored from the session?

Maybe I'm missing something? But what I thought was:

* Retrieve the position from the current documents
* Iterate over all docs and save all positions (where available from the weakrefs)
  and save all except the current one to the "manuscripts" setting.


Hmm... I don't really think a bash script to run git commands belongs in an educational boilerplate project like this. Perhaps this is common practice and I've just failed to notice it, but it doesn't seem that valuable.


@chandralaxmi  : method should not be long. It should be max 6-7 lines. make it more modular and use more functions.


How is this patch relevant to the rest of this PR? If it isn't, revert it and provide it in a separate PR; if it is, make it more clear how so.


Well done! Thanks.


So we should just change the install  of  libnl to use  the backports ?


Thanks a lot for all your testing!
Can you check the last two commits?

- 8aab1481a66f0cb0db1ec5f13cb228e8277de3de: to address your first comment about the dimension mismatch during the kernel rotation. Indeed, the problem seems to be that some arrays are 1D, eg *(91,)*, and some are 2D, e.g. *(1,91)*. I tried to solve this by reshaping the rotation arrays `Ylm_rot` to 1D as well.

- 92484388ccf978e73c70f941efe4861f2740ed7b: To address your last comment in case *DIPY* was never used before and so the settings folder was missing, i.e. *.dipy*.

Do these modifications solve the issues on your system?


np, i am closing it so...


Yeah. Someone who can make a decision about what to do with this pull request, as it has been hanging for some time now.


Ca se tient (d'ailleurs, dans cette veine, pourquoi pas utiliser Accept-Language? la transformation de la représentation est en général gérée directement via les headers non? et si tu voulais appliquer le principe d'affordance, tu mettrais directement /items/4242?lang=en_US?)


The same formatting should be applied to the other options.


Use memcpy for these instead.


The changes look good to me!
(finally found some time to review them)


FYI, a "commit" is a patch which some metadata (like a timestamp or git's revision hash) is being tacked on. A patch is a group of related changes, bundled together into an entity (e.g. a file). Hence "change" would be the proper term to use here. (Not that it really matters, though.)


Oops, sorry. Of course you are right ...


We should be using OS routines for this instead of consulting the cpuid flags.


Should have a comment explaining the purpose of this struct, in particular what kind of "unit" you're referring to.


shwing


You should explain why you need this GIT_COLOR_RESET. I guess it's about `git branch -vv` that displays the remote in color, and resets afterwards?

I think what you're doing here conflicts with my remark about 'color' in the tag thread: I still think that changing the behavior of `%(color)` is bad, but this piece of code depends on the patch I'm suggesting to drop.

Perhaps it would be better to introduce a new `%(colornext:...)` atom that would act like your `%(color)`, and keep the existing `%(color)` unchanged? But then, `%(colornext:...)` should be the one printing this GIT_COLOR_RESET: it should mean "next atom is this color", not "starting from next atom, use this color".


"Reply message fields: None" does not mean that you don't get a reply. It just says that there is no additional data in the reply.

What do you mean with "tvh doesn't seems to respond for dead subscription". You should get a response containing an "error" field in case something goes wrong (e.g. subscription not found). => https://github.com/tvheadend/tvheadend/blob/master/src/htsp_server.c#L2153 Are you absolutely sure that you get no reply from tvh?

I suggest that we're going to find the actual root cause for this.


@drudge yeah, sure. Also I face the problem, that coveralls can't parse lcov report. It says 0% and I don't know why :worried: Trying to fix...


Where is the PR? github doesn't link there from commits.

@vvakame can make decisions. Seems to me that if the backbone library permits this usage, then the typings should as well. In the PR can you include a code snippet which is correct backbone usage but the typings give compiler warnings?


Hi @melissarebooted, is this file necessary for being committed to the repo?

Do you think you could remove it?


I think in order to take over that site we'd have to get @brixen involved, since he seems to be the owner of the domain AFAIK.

Source is here: https://github.com/rubyspec/rubyspec.github.io

If he doesn't respond on GitHub, his contact info is available through a whois lookup on the rubyspec.org domain.


Dokładnie to samo:
150727 15:28:59	   15 Connect	lms@localhost on lms
		   15 Query	SET NAMES 'UTF8'
		   15 Query	SELECT section, var, value, description FROM uiconfig WHERE disabled = 0
		   15 Quit

15:30:17 mysql{7}> select * from uiconfig WHERE section='notify';
+-----+---------+-----------------+---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-------------+----------+
| id  | section | var             | value                                                                                | description | disabled |
+-----+---------+-----------------+---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-------------+----------+
| 175 | notify  | debtors_days    | 0                                                                                |             |        0 |
| 166 | notify  | debtors_limit   | 0                                                                                |             |        0 |
| 173 | notify  | debtors_message | Szanowni Państwo,
Uprzejmie informujemy, że do dnia dzisiejszego nie otrzymaliśmy płatności za poniższą fakturę:

- Faktura %number, pozostało do zapłaty %value zł.

|             |        0 |
| 174 | notify  | debtors_subject | Przypomnienie o płatności za usługę                                                                                |             |        0 |
| 163 | notify  | debug_email     | foo@bar.com
|             |        1 |
| 162 | notify  | mailfname       | Foo-Net                                                                                |             |        0 |
| 161 | notify  | mailfrom        | noreply@foo.bar.pl                                                                                |             |        0 |
| 169 | notify  | notes_message   | test                                                                                |             |        1 |
| 170 | notify  | notes_subject   | notes test                                                                                |             |        1 |
| 164 | notify  | smtp_host       | foo.bar.pl                                                                                |             |        0 |
| 165 | notify  | smtp_port       | 25                                                                                |             |        0 |
+-----+---------+-----------------+---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+-------------+----------+
11 rows in set (0.00 sec)


Forgot to remove this.


Maybe it would be better to set $separator from PMA_URL_getArgSeparator() above this loop (instead of below) and use $separator instead of "&".


Can we place these and related files inside a subdir?


Why is this change good and/or needed?


Need test on windows environment.


Decide how to handle cases in terms of what to return/throw when split doesn't return a list > 1. I haven't thought through that yet, but on first reaction, I don't like letting it blow up like this.

I'm also wondering about the use cases that has you working the url rather than having a json element from interacting with the richard api, thus having an id already available to you.


heroku was out of sync.
i ran ` git push heroku staging:master` to update


2 blanks


Didn't review this. I'll assumed it doesn't have any bugs :P


But why did you change from v4l2_subdev_call -> v4l2_device_call_all?
Didn't the driver work without those changes?

The original version of this driver was using v4l2_device_call_all
everywhere, but the Hans Verkuil (linux-media developer)
said that i should change these to v4l2_subdev_call [1].

There is only one subdev on the board, so there shouldn't be any need to
call v4l2_subdev_call_all.

Also, the initialization you removed is not handled by the i2c sub module
as you claim.
The i2c subdev will __only__ initialize the saa7113c chip.
The initializations you removed are for initializing the smi2021 chip, and
this is not done via the I2C bus.

The reason for the question is that I don't have the 4 input device, and
you claim that your version works with the 4 input version with these fixes.

It might be that your device is different from mine, and don't need those
initializations?

[1]: http://thread.gmane.org/gmane.linux.kernel/1554414

On Sun, Jul 26, 2015 at 11:40 PM, Philip Bordado <notifications@github.com>
wrote:

> Hi Jon,
>
> Some changes are required on newer kernel and some are cleanups.
> The chip initialization is now part of the i2c sub module on newer kernel
> so I removed it.
>
> kerpz
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kerpz/smi2021/commit/23f1053cf3ca2d63b1037f2f2271143ee927962f#commitcomment-12363106>
> .
>

--
Jonarne
http://jonarne.no


The issue is not completely about the docs. May I ask a question @jareguo, are we able to access to the runtime(cocos+pixi) globally?


Also note that identifiers starting with an underscore followed by an upperscore letter are reserved by the compiler. As far as I know, some compiler(s) at least throw a warning about this (albeit probably not when passed as a program switch).


@Tuuleh by moving `app/services/search/search.rb` to `app/services/search.rb` and removing your `require_relative` statements will fix this problem by passing on the job of loading files to Rails' magical autoloader. This totally contradicts what I said on Friday.


@airlessproject
I ran several bus benches. At first it seemed like this improved a lot, but after several testing over some days it seemed more like the speed did not really improve. Just that you know.


I agree with LordHoto, which is why I did it that way in the first place.. I think it's clearer that the final returns are the "else" fallback of the previous if statements.


This is the PR, with a code snippet showing the desired functionality, which now is not possible:
https://github.com/borisyankov/DefinitelyTyped/pull/4931


"Heroku Elements"


Presumably then we can remove `size_impl`


zoom--


NOPE. Put something that'll complain loudly here.


Thanks for your Feedback Martin!


@flavorjones There is no default jruby version. You must be explicit about any jrubies in your Gemfile 'as normal'


Guess you know where I work now eh? Wrote this code over a year ago and keep using it on apps.


Disons que là j'aurais l'impression de récupérer la version en_US.
Dans mon exemple j'exprime la volonter de traduire (à la google translate en live) en passant par une action.

Voilà c'est tout le sel de REST un besoin mais pas vraiment/assez de cadre pour savoir comment bien faire.

Ceci dit oui tout ce qui est header Accept-* c'est carrément à utiliser.


what is yougov??

On Mon, Jul 27, 2015 at 1:07 PM, Mark Liversedge <notifications@github.com>
wrote:

> Haha! You're going up against YouGov!
> Good luck :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TuxedoFish/YRS2015/commit/00a8a0d57412eeb58d42e402ad7422c4e1d2bd58#commitcomment-12370486>
> .
>


Where is `brand_string` even defined? oO

Consider at least adding a `g_` prefix.


Can we use variables to avoid duplication of version numbers for slf4j and junit?


double quotes here, because we use them below.

If you wanted, you could use `for a in mbrec.get("artist-credit", [])`


* (without DAZ)


Is this from the stuff we were doing with the `[anons]` tracks? We should have it in a different commit


I'll try to do it with python 'requests' (I did my tests with curl).
Worse case we could use 'pycurl'.


Is this used?


I unintentionally changed the line QUEST_LV_0100_20150717_007634. I'm not sure what it means.


This file looks highly specific to x86/x64, and hence the filename should indicate this with a suffix.


funny comment


Sure thing.
Just note it might need some fixing to work that way (I had some errors).


There should be a comment explaining the purpose of this whole thing.


I'll fix it as suggested :)


Usually when writing a function, you give the function what it needs as parameters and it will give you the result as a return value. Here, you are achieving this via having global variables that the function mutates. This makes it less clear what the function uses and affects.

In this case, you change the x and y value outside the function, and then use it inside. It would be better make a list of coordinates outside the function and then pass it to the function. And then rather than mutate the external movable variable, use the value the function returns.

So, in the case of move down, try:
```javascript
var shiftedCoordinates = block.coordinates.map(function(coordinate) {
   return [
    coordinate[0],
    coordinate[1] + 1
  ];
}

if(willFill(shiftedCoordinates) {
 //Do stuff
}

```
NB. ```javascript map ``` is a easy of making a new array out of another array https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Global_Objects/Array/map, you could also do it by building a new array by looping through the other


I wasn't sure exactly how to implement this so I tried a few different things and this was the last thing I tried.


YouGov, like Populus, Ipsos Mori, Comres, ICM etc are all organisations that ring and interview people to find out what they think about things.

They are pollsters.

They then tell us how many people might vote one way or another, or what they've said about things that might be in the news; e.g. EU referendum, benefit cap, fox hunting.

There is a website that tracks all of them to show what the current polling is for political parties; http://ukpollingreport.co.uk/ but if you look on YouGov's website you will see they poll on lots of other things too....

See https://yougov.co.uk/#


Awesome, this worked.


I suggest renaming to emitter_x64.cpp for discoverability reasons (when looking for an emitter, people would look for files starting with "emitter" rather than "x64").


Typo, `offest` -> `offset`


OK! :+1:


Here i cahnged nothing


rgerger


The class doesn't actually have type parameters. Were they removed?


bug?


This file has bits of MurmurHash3 (https://code.google.com/p/smhasher/source/browse/trunk/MurmurHash3.cpp) randomly mixed with other code. I'd prefer simply taking the original mmh3 code from the link and use it directly. The variants and dispatching here are afaik mostly useful for texture hashing (where the tradeoffs have a large impact.) I think we should just always use the 128-bit variant of mmh3, which also gives us more reassurance against collisions which Subv asked about in another comment.


Functions in this file need proper interface documentation. Also, consider changing these functions names to something more specific than "Hash64".


@maimoonak please confirm whether the parameters you have removed isn't causing build issues, the class on my end still expects the extra params you have ommited


The timeslot gets a value between 0 and 10. Shouldn't it get a value between 0 and SLOTFRAME_LENGTH?

Also this function does not guarantee that the available offtsets are found if they exists, causing sixtop_addCells to fail even when there are free cells. Is this the intended behaviour?


After adding "MISSING" status, when I rename a file then click "Amend Last Commit" checkbox, it longer prompts "Do you want to remove it from the index?".

Steps to reproduce:
Create a new file `1.txt`
`git add 1.txt`
`git commit`

Rename the file directly (e.g. Explorer rename) as `2.txt`

Open TortoiseGit commit dialog
Click "Amend Last Commit" checkbox
The old TortoiseGit should show "Do you want to remove it from the index?".
But after this commit, it does not.


 (*) assumes float24 is implemented as a 32-bit float.

(at least that's what I took away from IRC discussions)


No it is not. To LTC should go the simplest / most readable one.


Hi,

what is the advantage of macb over emacps?

Thank you


@danmar I don't see any problems with this code when compiling with Visual Studio 2012:

    std::string s;
    if( s[0] ) {}


Test


Hmmm, we'll have to pass 'mc' so it won't be a 'utility' function, but after all we only use it to detect NTLM...


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.828.0b5b67669ef18b5532df1de30e2f7ce407e5c872](http://build.palaso.org/viewLog.html?buildId=60056&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:03:53


We should consider adding some helpers for `hashCode`, `equals` and `toString` (if they don't already exist) to reduce the boilerplate for classes such as this.


Set during one call and don't do it in a loop.


Taky dobre :monkey:


Do you mean globally in Fireball Scripts? Yes, we can.


I wonder if this is necessary and OK. When I run the 'manpath' command, it prints a warning and I had nix path listed twice:

manpath: warning: $MANPATH set, appending /etc/manpath.config
/home/bohan/.nix-profile/share/man:/home/bohan/.nix-profile/share/man:/home/bohan/local/share/man:/usr/local/man:/usr/local/share/man:/usr/share/man

The /etc/manpath.config machinery seems to be able to automatically infer the man paths from the the PATH environment variable.
Also, I remember I used to have problems when I had some profile scripts modifying  MANPATH: the 'man' command would not find system-wide paths anymore. I don't understand why because from what we can see in the warning above, it does seems that 'manpath' still appends system-wide paths despite the MANPATH "override".


haha you can contribute and write a funny comment, too :smile:


Ah, wrong comment - sorry.
Yea, I'll add it.


I'd prefer a static factory method called `parse`.


I have pyton installated  and I don't have problems  with these lines


Sounds good.

On Mon, Jul 27, 2015 at 9:24 AM, Toby Dylan Hocking <
notifications@github.com> wrote:

> this is a good reference but it would be even better if you referenced the
> equation number in the SAG paper. and then explicitly state that "E below
> is a copy of equation XXX in paper YYY." As it is, I'm not sure if we need
> to look in the Schmidt or Bishop reference, and where exactly we would need
> to look, and which equation below is taken from there.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IshmaelBelghazi/bigoptim/commit/5f8ab9f7ab614095d46f28507619dd00123e5033#commitcomment-12371988>
> .
>


You should not manually append file extension like this, because the OS will not detect conflicting filenames. If you save as 'myfile' and 'myfile.ext' already exists, it will be overwritten without asking for confirmation. If you save as 'myfile' and 'myfile' exists, it will ask if you want to overwrite 'myfile', even if the result will be to create 'myfile.ext', and 'myfile' will not be touched. You should really use the none static version of the FileDialog to specify the default extension, like I did in my pull request.


Here I meant to say:

I'll try to do it with python 'requests' (I did my tests with curl).
Worse case we could use 'pycurl'.


Yeah, the assert message isn't making much sense to me either.


Does it? I don't think it does.


Do we want to allow this? Additional information tends to be useful. `CopycatException` doesn't include a no-args constructor.


Also, chances are that even if float24 is changed in the future, it'll wrap a u32, so this check will still pass anyway.


Are bounds checks used to make sure you indeed don't exceed that size? (I get that you basically leave room for 1024 kb of host code for each instruction here, but this regardless seems like quite the fragile construction.)


Beklager stor commit, men det er altså ingen ny funksjonalitet her, kun flyttet rundt på ting


They are dead headers, they arent used.


If you or anyone you know would like to give a talk, please feel free to reach out to me through the meet up site or on twitter (@aapwild). Or if you have other questions or suggestions, I'd love to hear them. This is brand new so I’m open to any ideas to make this a fun and useful meet up.

^^ suggested edit


This release is OP bots.


Sounds good!!! :shipit:


call_function will call 'hiera' with the closure scope of this function ('params_lookup'). If you want to make it appear as if the call to 'hiera' comes from the scope that called this function ('params_lookup'), you should use the function 'call_function_with_scope' instead, and pass the scope you got  from the declared 'scope_param()' - i.e. the 'scope' parameter in this method. (You are probably ok here, since 'hiera' only looks in top-scope anyway, but if you were to call say hira_include, include, realize, contain etc. it would matter).


purple stars now lel


Missing Newline.


Are you guys referring to the webrtc ringer? What build are you using?


Yeah, it doesn't unless you guarantee that all UnitState instances are 16-byte aligned.

You'll probably want to use http://en.cppreference.com/w/cpp/language/alignas .


As far as I understand proxy aren't suppose to set cookies (probably because it's not clear whose cookie is it and may cause collisions).

I found RFC 2109 stating:
Proxies must not introduce Set-Cookie (Cookie) headers of their own in proxy responses (requests).

If we decide to implement it we'll need to be careful not to forward it to the origin server (mod_proxy will probably just forward).


*type="device"* doesn't exist.


Thanks - will fix.


gettext around strings, also `%w` instead `[``]`


Funkfeuer*


I don't think that would be the solution because I still don't understand what happened. The build from 22nd, which was triggered by github, installed the deps automatically. However, the one you triggered manually did not.

Perhaps try to trigger another one or wait for a day and see if it gets built automatically once something gets merged?

How is the job configuration setup in terms of how often it builds?

Cheers,
Wilder


Actually I will just run all HTML of the Config pages through the HTML validator and fix those things (separate from re-styling ideas for Config)


`%w` instead `[]`


This *might* be off-topic for this PR, but this is redundant. If you look at shader_bytecode.h, the only thing `GetRegisterType` does is check if the register id is in some range. `GetIndex` does the same, subtracting the base. And then the only thing this does is add back the same base (in the form of the separate register arrays.) These arrays should be unified in a single one with all registers, and then the id can just be used directly.


Fix #58


Hi Bart,

When you add a field or make another database-related change to the model, don't forget to create a new migration to change the database with `python manage.py makemigrations`. (I've done it for this one and a previous one).

Thanks a lot!


`%w`


Are you have any example of mis-detection that solved with this commit?


@hhaidar maybe handy to also give solutions for if mongodb is running and you cannot connect through the app? like #562


lout?


This reportedly does not compile on gcc. Do `offsetof(UnitState, input_registers[0]) + index*sizeof(input_registers[0])`.


Same comments from the other function apply.


The superclass does this check too.


Was this a bug?


A "message," both for errors and results, seems implicit for every method in this class. I'm wondering if we can extract them or use another attr_reader to set them. There's also the AR pattern of calling .errors. Maybe not ideal, but there's something funky here and I can't put my finger on it.


Great debug ! :)


I think that in Kafka, we don't use braces for single-line statements right?


`%w` here


Hi Loic,

yes, I actually had done that, but still had not pushed yet:
commit 7233cc444f05e50a75f67cb67b1045dc3f6bb288
Author: Pietro Vischia <pietro.vischia@gmail.com>
Date:   Thu Jul 23 17:33:04 2015 +0200

I am creating a new pull request :)


Ok. I wasn't aware that this was a common practice. I'm used to dropping the entire database and creating a new one but that's obviously not a good idea once we're in production :-)


Question about where these backoff parameters are coming from; in particular, I'm curious about kTransmissionDelay


The modification it is right.


I'm a bit uneasy at the fact that we have a concrete class inheriting from another concrete class.


Other than one small question above, this commit LGTM.


Actually, I notice now that the unification has already been picked up by your merge, so.. :)


gettext missing


MSVC doesn't support alignas properly, iirc, so we might have to resort to adding a per-compiler macro. (2015 does support it, so the timing is a bit unfortunate.)


Thanks for reverting it.
This commit has not yet landed on my install, and when I run the 'manpath' command, it prints a warning and I had nix path listed twice:

manpath: warning: $MANPATH set, appending /etc/manpath.config
/home/bohan/.nix-profile/share/man:/home/bohan/.nix-profile/share/man:/home/bohan/local/share/man:/usr/local/man:/usr/local/share/man:/usr/share/man

The /etc/manpath.config machinery seems to be able to automatically infer the man paths from the the PATH environment variable.
Also, I remember I used to have problems when I had some profile scripts modifying MANPATH: the 'man' command would not find system-wide paths anymore. I don't understand why because from what we can see in the warning above, it does seems that 'manpath' still appends system-wide paths despite the MANPATH "override".


I don't understand what this is preventing. It looks like I can't delete the email if I'm not in the first two participants. Under what circumstances would that not be the case?


TeamCity xBIM :: Xbim Essentials :: Release [Build 64](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2139&buildTypeId=XBIM_XbimEssentials_Release) is now running


I found 64DD CIC photo, here http://n64.trashfolder.org/i/64dd-cic.jpg


`offsets = new HashMap<>();`


Store this in a variable...


linux/lcd_notify.h is missing in source


assign the post for the comment by using the variable you stored in line 12.


Hmm how would this differ from opaque objects then?
On Jul 27, 2015 9:23 AM, "Peter Hamilton" <notifications@github.com> wrote:

> The type for SecretData KeyBlocks should be OPAQUE. At least, that's what
> the spec claims.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hadesto/PyKMIP/commit/65ce92d2a42b0e491d835ca640033a6f62ab2aa8#commitcomment-12371971>
> .
>


TeamCity xBIM :: Xbim Essentials :: Release [Build 64](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2139&buildTypeId=XBIM_XbimEssentials_Release) outcome was **SUCCESS**
Summary: Running Build time: 00:01:23


As a non-mskcc user potentially interested in LDAP support at my local institution:
hard-coding your personal user-ID for LDAP querying seems to be a bit restrictive?


Yeah, `offsetof` requires compile-time evaluable expressions.


@syhily
主要还是懒...所以搞回Hexo了...


So, you are trying to link to a specific user's profile. where does ```users_path``` link to?


对啊，这不是原来没有，然后commit加上了zhprs的前缀么。。。


If this function stays around any longer, it should at least have some comments explaining what it does and what it's intended to be used for.


Typically this would be `Converter<S, T>` or something like that. Has it been established that there is no benefit in having type parameters here?


Why does this not handle uniform inputs?


Modify this Test to be Nunit Framework


:ok:


<!DOCTPE HTML>

		<link rel="stylesheet" type="text/css" href="css.css" />
		<title>html5自学</title>

	<button onclick="playPause()" >播放/暂停</button>
	<button onclick="makeBig()" >大</button>
	<button onclick="makeNormal()" >中</button>
	<button onclick="makeSmall()" >小</button>

	<video  id="video1" controls="controls" class="vid" >
 	 <source src="abcd.mp3" type="video/mp4">
 	 <source src="abcd.ogg" type="video/ogg">

	Your browser does not support the video tag.
	</video>


Oh dear, that's what happens if you rely too much on search/replace after copy/paste...
That or just a blackout :)


And merge the tests.


Should probably use `===` here.


(commit reviewed)


main purpose of the PR was using the constants in place of the magic values. when @sebastianbergmann agrees I am willing to change the comparison operator though.


(Edit, I saw this commit was eventually reverted in the main repository, so you can forget about my comment here)

I wonder if this is necessary and OK. When I run the 'manpath' command, it prints a warning and I have nix path listed twice:

manpath: warning: $MANPATH set, appending /etc/manpath.config
/home/bohan/.nix-profile/share/man:/home/bohan/.nix-profile/share/man:/home/bohan/local/share/man:/usr/local/man:/usr/local/share/man:/usr/share/man

The /etc/manpath.config machinery seems to be able to automatically infer the man paths from the the PATH environment variable.
Also, I remember I used to have problems when I had some profile scripts modifying MANPATH: the 'man' command would not find system-wide paths anymore. I don't understand why because from what we can see in the warning above, it does seem that 'manpath' still appends system-wide paths despite the MANPATH "override".


Ist noch was zu fixen?


Nein, war nur von den Release-Notes im Appstore und der Commit-Message irritiert. Das Changeset ist in Ordnung.


create a read me file


My understanding is that this code is trying to prevent user X to delete a message in which user X didn't participate but users Y and Z did. Not sure how or when this could happen but the code was already there. I just made sure we had a test for it.


Typo "an set" -> "a set".


@arturoc if you log in with github at travis, as you have push access, you should see a refresh button on a build page to retrigger a build.


Works well. Cheers!

I'm not really convinced by the location of the project files. why have you put them in subfolders, rather than in the project file directory?


Is there any reason to be calling this if you're calling `configure_java`? From source at end of `jdk.bzl` it looks like no?


Может всё же
```php
parent::tearDown();


确实，但是我看了scws的源码，发现这些操作都是相当轻量级的。从概念完整和代码可读性的角度来考虑，如果没有可观的内存和性能提高，我觉得并不需要使用这些优化措施而去破坏可读性。


Additionally file name can and often do have dots prior to the extension delimiter (if any).


This should not really become part of sparql client as Franz' implementation of the JSON response violates the W3C standard. Nevertheless, I need this working for RepMine, hence, this fork will contain this ugly fix...


remove space.


haha great commit message


Oops


:ok: :1234:


第一次使用！！！


Front end assignment 4 updated


Right, I think that makes sense. Probably worth adding to CODING_STYLE (together with how we allow treating pointers as booleans (rather than explicitly comparing them to NULL), but not ints, etc).


Also, maybe all occurrences of `set` should be replaced with `list`.


(which is useful for testing presto for example)


It's "JIT", not "Jit" ...


*core/src/main/java/org/jruby/embed/IsolatedScriptingContainer.java-* seem like a left over or Java-- source :)


to use a table which is created in one database (e.g. hive, psql) while sending test query to other database (e.g. presto)


@springmeyer not sure what "all" references here, but given that [this commit](https://github.com/mapnik/mapnik/commit/461163d8830697bea85192321a1e8fb0789b931b) removes OSM plugins files and we still have [OSM plugins in control](https://github.com/mapnik/debian/blob/master/master/debian/control#L91) the build fails.
It may be just me missing some step though :)


:+1: looks great, incomparable with the previous version!


Take a look a the example below


Scala has a similar method and it's called `grouped`. The difference is that it takes the number of elements per group instead of the number of groups.


What about this one? Which one are you guys preferring? The deep nesting? Or the reduced one?


Theoretically, acquiring and releasing connections should be faster. Personally I would go with HikariCP because, quoting [Joshua Bloch's Effective Java, Item 47](http://jtechies.blogspot.de/2012/07/item-47-know-and-use-libraries.html):
 >"Generally speaking, library code is likely to be better than code that you’d write yourself and is likely to improve over time. This is no reflection on your abilities as a programmer. Economies of scale dictate that library code receives far more attention than most developers could afford to devote to the same functionality."

For example, the second log I posted [here](https://github.com/bob7l/HawkReloaded/issues/48#issuecomment-123074056) seems to indicate that something within the `ConnectionManager` goes wring to some extend.

I agree that focussing on indexes is the most important point when looking at the performance. From my point of view, the problem is that you are somewhat limited by the current database layout. There are already indexes on every column except `data` (witch currently is impossible to index properly). Unless there is some way to normalize whatever is stored in `data` into multiple tables or columns, I fear, there is little that can be done.


below


database name


FWIW, we've given up on old Python support. Let's move to dict comprehensions when added/changing things.


missing gettext


I suggest using new-style typedefs for function pointer aliases, i.e.

    using CompiledShader = (*)(void* state);


it appears that they conform to the RFC for escaping control characters http://www.ietf.org/rfc/rfc4627.txt


nice!


i know i pushed this to my repo to test on osx before merging in the main repo but i forgot to enable travis there first so i needed to actually push


tnx, fixed.


This would be better done as a `dict.update()` instead of allocating a new dict entirely.

These two nitpicks brought to you courtesy of this changeset being linked from a PR. :)


Kinda meh on the whole "look up the code again every time" thing. We can detect when the shader code is modified and update the code pointer then. This should allow for better code generation (swizzles are constant, etc.) too.


This fix introduces new (undesired?) behaviour in the following scenario:
Imagine a webpage where the meta tag required for a native smart app banner is not yet hardcoded onto the page, but added through javascript.

Safari on iOS will not re-evaluate the html and add the native banner again
(see: http://stackoverflow.com/questions/31289530/load-ios-smart-app-banner-through-javascript/

A workaround was to use this cool javascript plugin, since that would just generate the banner in the theme that was required for that device.
Now the logic that should only look at the theming, suddenly decides whether the banner is loaded or not.

For me, this 'fix' is breaking existing functionality.


No reason. That's a  test / showcase.


Hi @maimoonak, though  I think the following error is not relevant with this changes but i just take a fresh checkout from master branch and while trying to build opensrp-form with mvn clean install it is giving the following building error, do you have any clue with this?

pensrp-server/opensrp-form/target/surefire/surefire6491999996705892563tmp
Running org.opensrp.form.repository.it.AllSubmissionsIntegrationTest
Tests run: 4, Failures: 0, Errors: 4, Skipped: 0, Time elapsed: 1.557 sec <<< FAILURE!
Results :

Tests in error:
  shouldFetchAllFormSubmissionsAfterServerVersion(org.opensrp.form.repository.it.AllSubmissionsIntegrationTest): Failed to load ApplicationContext
  shouldFetchAllFormSubmissions(org.opensrp.form.repository.it.AllSubmissionsIntegrationTest): Failed to load ApplicationContext
  shouldFetchFormSubmissionsBasedOnANMIDTimeStampAndBatchSize(org.opensrp.form.repository.it.AllSubmissionsIntegrationTest): Failed to load ApplicationContext
  shouldCheckIfFormSubmissionExistsByInstanceId(org.opensrp.form.repository.it.AllSubmissionsIntegrationTest): Failed to load ApplicationContext

Tests run: 4, Failures: 0, Errors: 4, Skipped: 0


Here you need to clarify more. You do not say anything about type_manager type. Which is crucial for the matching. And please read it before sending to review ;)


Thanks! BitHub has sent payment of  $10.34USD for this commit.


Also, I needed this change:
```diff
diff --git a/master/debian/mapnik-input-plugin-postgis.install b/master/debian/mapnik-input-plugin-postgis.install
index 21b1d4e..0ab4fe1 100644
--- a/master/debian/mapnik-input-plugin-postgis.install
+++ b/master/debian/mapnik-input-plugin-postgis.install
@@ -1 +1 @@
-usr/lib/mapnik/input/postgis*
+plugins/input/postgis*
```


remove this space.


Also, c.f. my MurmurHash3 comments in the other commit.


remove that weird space.


dont need nil in quotes


always a space before the expect statement.


keep the scopes up top of file


Looks like working, thanks a lot. #433


@arushgupta why we use invoice_number here


yes a left over from manual merging things - thanx